Replies: 1 comment 1 reply
-
Indeed, this was changed in 6.0. But is it happening in 8.0.0-rc.2 or in 6.12?
Given this is more of an extensibility feature, do you think it is worthwhile to add that to migration guide? It is mentioned in the breaking changes.
That one is mentioned here https://github.com/fluentassertions/fluentassertions/blob/release-8.0/docs/_pages/upgradingtov8.md#from-executeassertion-to-assertionchain
It's part of the
What's the specific error you're getting. Because indeed, that was adjusted in 6.11.
No, you need to use |
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
-
I started testing migration from 6.12.0 to 8.0.0-rc2 in my major projects and found some issues. I'm unsure whether they should be in the migration guide.
BeEquivalentTo
get failing because they tested internal properties. I've fixed it with addingIncludingInternalProperties
. But, this feature was already added in 6.0.0. I'm wondering why this problem now starts after 6.12.0.AssertionOptions.CloneDefaults<T>()
to get default options. I did not find helpful information pointing to the new sourceAssertionConfiguration.Current.Equivalency.CloneDefaults<T>()
.Execute.Assertion
to add some specific assertions. I did not find any hint on "removing"Execute
in the public API. Only indirectly I found how to get access.AssertionScope.AddReportable
has been made internal (was added in 6.0.0 to public API). I did not find any reference toAddReportable
in migration or release notes.AssertionScope.FailWith
. IsAddPreFormattedFailure
the correct replacement? I did not find any hint on this change.Beta Was this translation helpful? Give feedback.
All reactions