Workaround for crash in cibuild

Bart Koelman · Bart Koelman · commit cc45e154e730 · 2021-11-01T11:05:50.000+01:00
diff --git a/src/JsonApiDotNetCore/Serialization/Request/Adapters/AtomicOperationObjectAdapter.cs b/src/JsonApiDotNetCore/Serialization/Request/Adapters/AtomicOperationObjectAdapter.cs
@@ -69,7 +69,10 @@ private WriteOperationKind ConvertOperationCode(AtomicOperationObject atomicOper
             {
                 case AtomicOperationCode.Add:
                 {
-                    if (atomicOperationObject.Ref is { Relationship: null })
+                    // ReSharper disable once MergeIntoPattern
+                    // Justification: Merging this into a pattern crashes the command-line versions of CleanupCode/InspectCode.
+                    // Tracked at: https://youtrack.jetbrains.com/issue/RSRP-486717
+                    if (atomicOperationObject.Ref != null && atomicOperationObject.Ref.Relationship == null)
                     {
                         using IDisposable _ = state.Position.PushElement("ref");
                         throw new ModelConversionException(state.Position, "The 'relationship' element is required.", null);

Original file line number	Diff line number	Diff line change
`@@ -69,7 +69,10 @@ private WriteOperationKind ConvertOperationCode(AtomicOperationObject atomicOper`
`69`	`69`	`{`
`70`	`70`	`case AtomicOperationCode.Add:`
`71`	`71`	`{`
`72`		`- if (atomicOperationObject.Ref is { Relationship: null })`
	`72`	`+ // ReSharper disable once MergeIntoPattern`
	`73`	`+ // Justification: Merging this into a pattern crashes the command-line versions of CleanupCode/InspectCode.`
	`74`	`+ // Tracked at: https://youtrack.jetbrains.com/issue/RSRP-486717`
	`75`	`+ if (atomicOperationObject.Ref != null && atomicOperationObject.Ref.Relationship == null)`
`73`	`76`	`{`
`74`	`77`	`using IDisposable _ = state.Position.PushElement("ref");`
`75`	`78`	`throw new ModelConversionException(state.Position, "The 'relationship' element is required.", null);`