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
Changes Add(IExecutionResult) to AddResult for Snapshot tests #5320
Conversation
@michaelstaib @PascalSenn swapping the SnapshotExtensions.Add calls to snapshot.AddResult using method chaining, Rider has applied this not just to this usage but all situations where the static method call can be swapped. Should I wind this back? |
var empty = TypeConverterExtensions.Convert<object, Guid>( | ||
DefaultTypeConverter.Default, | ||
null); | ||
var empty = DefaultTypeConverter.Default.Convert<object, Guid>(null); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In unit tests we need to be explicit. So this needs to be reverted.
@@ -11,31 +11,27 @@ public class PagingObjectFieldDescriptorExtensionsTests | |||
public static void ObjectFieldDescriptor_UseOffsetPaging_Descriptor_Is_Null() | |||
{ | |||
Assert.Throws<ArgumentNullException>( | |||
() => PagingObjectFieldDescriptorExtensions.UsePaging( | |||
default(IObjectFieldDescriptor)!)); | |||
() => default(IObjectFieldDescriptor)!.UsePaging()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same goes here.
e3347b5
to
e05bad5
Compare
Renames Snapshot Add extension method to AddResult