Skip to content

Commit

Permalink
Fix SenseDiffShouldUpdateAllFields test
Browse files Browse the repository at this point in the history
The SenseDiffToUpdate method specifically excludes the PartOfSpeech
object now, so the test shouldn't expect PartOfSpeech updated.
  • Loading branch information
rmunn committed Jan 10, 2025
1 parent b333bc1 commit e4521c5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion backend/FwLite/FwLiteProjectSync.Tests/UpdateDiffTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public void SenseDiffShouldUpdateAllFields()
var senseDiffToUpdate = SenseSync.SenseDiffToUpdate(before, after);
ArgumentNullException.ThrowIfNull(senseDiffToUpdate);
senseDiffToUpdate.Apply(before);
before.Should().BeEquivalentTo(after, options => options.Excluding(x => x.Id).Excluding(x => x.EntryId).Excluding(x => x.DeletedAt).Excluding(x => x.ExampleSentences).Excluding(x => x.SemanticDomains));
before.Should().BeEquivalentTo(after, options => options.Excluding(x => x.Id).Excluding(x => x.EntryId).Excluding(x => x.DeletedAt).Excluding(x => x.ExampleSentences).Excluding(x => x.SemanticDomains).Excluding(x => x.PartOfSpeech));
}

[Fact]
Expand Down

0 comments on commit e4521c5

Please sign in to comment.