From dd0c5a94e53154ed09b2f55809a9df09c52f26d9 Mon Sep 17 00:00:00 2001
From: Kevin Hahn <kevin_hahn@sil.org>
Date: Fri, 10 Jan 2025 10:39:46 +0700
Subject: [PATCH] only exclude Sense.PartOfSpeech.Predefined from equality test

---
 backend/FwLite/MiniLcm.Tests/CreateEntryTestsBase.cs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/backend/FwLite/MiniLcm.Tests/CreateEntryTestsBase.cs b/backend/FwLite/MiniLcm.Tests/CreateEntryTestsBase.cs
index ceb32c1a29..86fcb0a912 100644
--- a/backend/FwLite/MiniLcm.Tests/CreateEntryTestsBase.cs
+++ b/backend/FwLite/MiniLcm.Tests/CreateEntryTestsBase.cs
@@ -29,10 +29,8 @@ public async Task CanCreateEntry_AutoFaker()
             .For(e => e.ComplexForms).Exclude(e => e.Id)
             //predefined is always true in fwdata bridge, so we need to exclude it for now
             .For(e => e.Senses).For(s => s.SemanticDomains).Exclude(s => s.Predefined)
-            // TODO: Figure out how to exclude just PartOfSpeech.Predefined
-            // .For(e => e.Senses).Exclude(s => s.PartOfSpeech.Predefined)); // rejected because PartOfSpeech could be null
-            // .For(e => e.Senses).Exclude(s => s.PartOfSpeech?.Predefined)); // not allowed because "An expression tree lambda may not contain a null propagating operator."
-            .For(e => e.Senses).Exclude(s => s.PartOfSpeech)); // Not ideal but best we can do for now
+            .For(e => e.Senses).Exclude(s => s.PartOfSpeech!.Predefined)
+        );
     }
 
     [Fact]