Fixes for the rework

Nexus-Mods · Dec 19, 2023 · 98cc224 · 98cc224
1 parent 03b6679
commit 98cc224
Show file tree

Hide file tree

Showing 8 changed files with 20 additions and 11 deletions.
diff --git a/.gitignore b/.gitignore
@@ -4,6 +4,7 @@
 ## Get latest from https://github.com/github/gitignore/blob/main/VisualStudio.gitignore
 
 .idea/
+*/Artifacts.*/*
 
 # User-specific files
 *.rsuser

diff --git a/benchmarks/NexusMods.EventSourcing.Benchmarks/NexusMods.EventSourcing.Benchmarks.csproj b/benchmarks/NexusMods.EventSourcing.Benchmarks/NexusMods.EventSourcing.Benchmarks.csproj
@@ -19,4 +19,16 @@
       <PackageReference Include="BenchmarkDotNet" Version="0.13.11" />
     </ItemGroup>
 
+    <ItemGroup>
+      <Compile Remove="BenchmarkDotNet.Artifacts\**" />
+    </ItemGroup>
+
+    <ItemGroup>
+      <EmbeddedResource Remove="BenchmarkDotNet.Artifacts\**" />
+    </ItemGroup>
+
+    <ItemGroup>
+      <None Remove="BenchmarkDotNet.Artifacts\**" />
+    </ItemGroup>
+
 </Project>
diff --git a/src/NexusMods.EventSourcing/NexusMods.EventSourcing.csproj b/src/NexusMods.EventSourcing/NexusMods.EventSourcing.csproj
@@ -4,6 +4,7 @@
     </PropertyGroup>
     <ItemGroup>
       <PackageReference Include="DynamicData" Version="8.3.27" />
+      <PackageReference Include="MemoryPack" Version="1.10.0" />
       <PackageReference Include="System.Reactive" Version="6.0.0" />
       <PackageReference Include="TransparentValueObjects" PrivateAssets="all" OutputItemType="Analyzer" ReferenceOutputAssembly="false"/>
     </ItemGroup>

diff --git a/tests/NexusMods.EventSourcing.TestModel/FodyWeavers.xml b/tests/NexusMods.EventSourcing.TestModel/FodyWeavers.xml
diff --git a/tests/NexusMods.EventSourcing.TestModel/Model/Mod.cs b/tests/NexusMods.EventSourcing.TestModel/Model/Mod.cs
@@ -1,5 +1,4 @@
 using NexusMods.EventSourcing.Abstractions;
-using ReactiveUI.Fody.Helpers;
 
 namespace NexusMods.EventSourcing.TestModel.Model;
 

diff --git a/tests/NexusMods.EventSourcing.TestModel/NexusMods.EventSourcing.TestModel.csproj b/tests/NexusMods.EventSourcing.TestModel/NexusMods.EventSourcing.TestModel.csproj
@@ -12,9 +12,7 @@
     </ItemGroup>
 
     <ItemGroup>
-      <PackageReference Include="DynamicData" Version="8.3.27" />
       <PackageReference Include="MemoryPack" Version="1.10.0" />
-      <PackageReference Include="ReactiveUI.Fody" Version="19.5.1" />
     </ItemGroup>
 
 </Project>
diff --git a/tests/NexusMods.EventSourcing.TestModel/Services.cs b/tests/NexusMods.EventSourcing.TestModel/Services.cs
@@ -1,5 +1,6 @@
 using Microsoft.Extensions.DependencyInjection;
 using NexusMods.EventSourcing.Abstractions;
+using NexusMods.EventSourcing.Events;
 
 namespace NexusMods.EventSourcing.TestModel;
 

diff --git a/tests/NexusMods.EventSourcing.Tests/BasicFunctionalityTests.cs b/tests/NexusMods.EventSourcing.Tests/BasicFunctionalityTests.cs
@@ -125,23 +125,23 @@ public void EntityCollectionsAreObservable()
         var modId = AddMod.Create(tx, "First Mod", loadoutId);
         tx.Commit();
 
-        var loadout = _ctx.Get<LoadoutRegistry>();
-        loadout.Loadouts.Should().NotBeEmpty();
+        var loadoutRegistry = _ctx.Get<LoadoutRegistry>();
+        loadoutRegistry.Loadouts.Should().NotBeEmpty();
 
         var called = false;
-        ((INotifyCollectionChanged)loadout.Loadouts).CollectionChanged += (sender, args) =>
+        ((INotifyCollectionChanged)loadoutRegistry.Loadouts).CollectionChanged += (sender, args) =>
         {
             called = true;
             args.Action.Should().Be(NotifyCollectionChangedAction.Add);
             args.NewItems.Should().NotBeNull();
         };
 
         using var tx2 = _ctx.Begin();
-        var modId2 = AddMod.Create(tx2, "Second Mod", loadoutId);
+        var loadout2Id = CreateLoadout.Create(tx2, "Test 2");
         tx2.Commit();
 
         called.Should().BeTrue();
-        loadout.Loadouts.Should().NotBeEmpty();
+        loadoutRegistry.Loadouts.Should().NotBeEmpty();
     }
 
     [Fact]