Skip to content

Conversation

@danielmpetrov
Copy link
Contributor

@danielmpetrov danielmpetrov commented Sep 18, 2019

This PR is an attempt to close out #989.

Changelog since the original PR:

I came to think of two inherent properties of empty/single element collections.
They are:

  • always ordered, and
  • both in ascending and descending order at the same time.
  • Organized the tests for Be In Ascending/Descending Order in GenericCollectionAssertionsSpecs.cs into sub-regions for better navigation, since it was hard to find my way around. I kept the original tests, and added on a few more as stated earlier, but please check them that they make sense. 👍

Closes #296 and Resolves #989

@dennisdoomen dennisdoomen requested a review from jnyrup September 18, 2019 19:24
@jnyrup jnyrup merged commit 9a3e049 into fluentassertions:master Sep 21, 2019
@jnyrup
Copy link
Member

jnyrup commented Sep 21, 2019

Thanks @david-a-jetter and @danielmpetrov for contributing this improvement

@danielmpetrov danielmpetrov deleted the gh_296 branch September 21, 2019 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Collection Assertion NotBeInAscendingOrder with lambda not available but documentation says it is

4 participants