New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Simplify inheritance of SubsequentOrderingAssertions
#2439
base: develop
Are you sure you want to change the base?
Conversation
Pull Request Test Coverage Report for Build 7398640344
💛 - Coveralls |
Qodana for .NETIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked View the detailed Qodana reportTo be able to view the detailed Qodana report, you can either:
To get - name: 'Qodana Scan'
uses: JetBrains/qodana-action@v2023.2.8
with:
upload-result: true Contact Qodana teamContact us at qodana-support@jetbrains.com
|
Src/FluentAssertions/Collections/SubsequentOrderingAssertions.cs
Outdated
Show resolved
Hide resolved
SubsequentOrderingAssertions
SubsequentOrderingAssertions
I don't know or recall why #1416 used Could we use |
I don't have any preference regarding the naming. I was simply wondering why it needed so many levels of inheritance and could find no reason for it. Should I rename |
Not sure I get you. |
That would practically be the same. All methods were in one class and all other classes were empty. I chose I expected the first change to have less chance of breaking things, as I don't change the return type of existing methods. That was my reason for choosing this way... |
Head branch was pushed to by a user without write access
3bc5754
to
3f5b06a
Compare
Qodana for .NETIt seems all right 👌 No new problems were found according to the checks applied 💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at qodana-support@jetbrains.com
|
@jnyrup I think this is waiting for you, isn't it? |
The
SubsequentOrderingGenericCollectionAssertions<TCollection, T>
is only used as a base class forSubsequentOrderingAssertions
. By moving the functionality into the (previously empty)SubsequentOrderingAssertions
the inheritance tree can be simplified.This should also increase the code coverage according to #1823.
IMPORTANT
./build.sh --target spellcheck
or.\build.ps1 --target spellcheck
before pushing and check the good outcome