Fixes, added remaining project tests

Author: Arlodotexe

src/Nomad/ModifiableEntity.cs

@@ -1,4 +1,5 @@
 using System.Collections.Generic;
+using System.Diagnostics;
 using CommunityToolkit.Diagnostics;
 using Ipfs;
 using OwlCore.ComponentModel;

src/Nomad/ModifiableImagesCollection.cs

@@ -116,15 +116,15 @@ public async Task ApplyEntryUpdateAsync(EventStreamEntry<DagCid> eventStreamEntr
         {
             case nameof(AddImageAsync):
                 {
-                    var imageFile = await Inner.GetAsync(image.Id, cancellationToken);
                     Inner.Inner.Images = [.. Inner.Inner.Images, image];
+                    var imageFile = await Inner.GetAsync(image.Id, cancellationToken);
                     ImagesAdded?.Invoke(this, [imageFile]);
                     break;
                 }
             case nameof(RemoveImageAsync):
                 {
-                    var imageFile = await Inner.GetAsync(image.Id, cancellationToken);
                     Inner.Inner.Images = [.. Inner.Inner.Images.Except([image])];
+                    var imageFile = await Inner.GetAsync(image.Id, cancellationToken);
                     ImagesRemoved?.Invoke(this, [imageFile]);
                     break;
                 }

src/Nomad/ModifiableProject.cs

@@ -116,7 +116,6 @@ public static ModifiableProject FromHandlerConfig(NomadKuboEventStreamHandlerCon
         {
             Id = handlerConfig.RoamingKey.Id,
             EventStreamHandlerId = handlerConfig.RoamingKey.Id,
-            Inner = handlerConfig.RoamingValue,
             InnerProject = readOnlyProject,
             InnerEntity = modifiableEntity,
             InnerAccentColor = modifiableAccentColor,
@@ -158,7 +157,7 @@ public static ModifiableProject FromHandlerConfig(NomadKuboEventStreamHandlerCon
     /// <summary>
     /// The roaming project data that this handler modifies.
     /// </summary>
-    public required Project Inner { get; init; }
+    public Project Inner => InnerProject.Inner;
 
     /// <inheritdoc/>
     public required IModifiableProjectCollection<IReadOnlyProject> Dependencies { get; init; }
@@ -175,7 +174,7 @@ public static ModifiableProject FromHandlerConfig(NomadKuboEventStreamHandlerCon
     public string Category => InnerProject.Category;
 
     /// <inheritdoc/>
-    public string Name => InnerProject.Category;
+    public string Name => InnerProject.Name;
 
     /// <inheritdoc/>
     public string Description => InnerProject.Description;
@@ -201,10 +200,10 @@ public static ModifiableProject FromHandlerConfig(NomadKuboEventStreamHandlerCon
     /// <inheritdoc/>
     public string[] Features => InnerProject.Features;
 
-    /// <inheritdoc/>
+    /// <inheritdoc cref="WindowsAppCommunity.Sdk.IReadOnlyProject.CategoryUpdated" />
     public event EventHandler<string>? CategoryUpdated;
 
-    /// <inheritdoc/>
+    /// <inheritdoc cref="WindowsAppCommunity.Sdk.IReadOnlyProject.PublisherUpdated" />
     public event EventHandler<IReadOnlyPublisher>? PublisherUpdated;
 
     /// <inheritdoc/>
@@ -252,7 +251,7 @@ public static ModifiableProject FromHandlerConfig(NomadKuboEventStreamHandlerCon
     /// <inheritdoc/>
     public IAsyncEnumerable<IFile> GetImageFilesAsync(CancellationToken cancellationToken) => InnerEntity.GetImageFilesAsync(cancellationToken);
 
-    /// <inheritdoc/>
+    /// <inheritdoc cref="IReadOnlyProject{TDependencyCollection}.GetPublisherAsync" />
     public Task<IReadOnlyPublisher?> GetPublisherAsync(CancellationToken cancellationToken) => InnerProject.GetPublisherAsync(cancellationToken);
 
     /// <inheritdoc/>
@@ -267,15 +266,23 @@ public async Task UpdatePublisherAsync(IReadOnlyPublisher publisher, Cancellatio
         var updateEvent = new ValueUpdateEvent(null, (DagCid)valueCid, false);
 
         var appendedEntry = await AppendNewEntryAsync(targetId: EventStreamHandlerId, eventId: nameof(UpdatePublisherAsync), updateEvent, DateTime.UtcNow, cancellationToken);
-        await ApplyPublisherUpdateEntryUpdateAsync(appendedEntry, updateEvent, publisher.Id, cancellationToken);
+        await ApplyPublisherUpdateEntryUpdateAsync(appendedEntry, updateEvent, publisher.Id, publisher, cancellationToken);
 
         EventStreamPosition = appendedEntry;
     }
 
     /// <inheritdoc/>
-    public Task UpdateCategoryAsync(string category, CancellationToken cancellationToken)
+    public async Task UpdateCategoryAsync(string category, CancellationToken cancellationToken)
     {
-        throw new NotImplementedException();
+        cancellationToken.ThrowIfCancellationRequested();
+
+        var valueCid = await Client.Dag.PutAsync(category, pin: KuboOptions.ShouldPin, cancel: cancellationToken);
+        var updateEvent = new ValueUpdateEvent(null, (DagCid)valueCid, false);
+
+        var appendedEntry = await AppendNewEntryAsync(targetId: EventStreamHandlerId, eventId: nameof(UpdateCategoryAsync), updateEvent, DateTime.UtcNow, cancellationToken);
+        await ApplyCategoryEntryUpdateAsync(appendedEntry, updateEvent, category, cancellationToken);
+
+        EventStreamPosition = appendedEntry;
     }
 
     /// <inheritdoc/>
@@ -415,7 +422,6 @@ public override async Task ApplyEntryUpdateAsync(EventStreamEntry<DagCid> stream
         ;
     }
 
-    /// <inheritdoc />
     internal Task ApplyFeatureAddEntryUpdateAsync(EventStreamEntry<DagCid> streamEntry, ValueUpdateEvent updateEvent, string addedFeature, CancellationToken cancellationToken)
     {
         cancellationToken.ThrowIfCancellationRequested();
@@ -425,8 +431,7 @@ internal Task ApplyFeatureAddEntryUpdateAsync(EventStreamEntry<DagCid> streamEnt
         FeaturesAdded?.Invoke(this, [addedFeature]);
         return Task.CompletedTask;
     }
-
-    /// <inheritdoc />
+    
     internal Task ApplyFeatureRemoveEntryUpdateAsync(EventStreamEntry<DagCid> streamEntry, ValueUpdateEvent updateEvent, string removedFeature, CancellationToken cancellationToken)
     {
         cancellationToken.ThrowIfCancellationRequested();
@@ -437,7 +442,6 @@ internal Task ApplyFeatureRemoveEntryUpdateAsync(EventStreamEntry<DagCid> stream
         return Task.CompletedTask;
     }
 
-    /// <inheritdoc />
     internal async Task ApplyPublisherUpdateEntryUpdateAsync(EventStreamEntry<DagCid> streamEntry, ValueUpdateEvent updateEvent, string publisherId, CancellationToken cancellationToken)
     {
         cancellationToken.ThrowIfCancellationRequested();
@@ -447,7 +451,6 @@ internal async Task ApplyPublisherUpdateEntryUpdateAsync(EventStreamEntry<DagCid
         await ApplyPublisherUpdateEntryUpdateAsync(streamEntry, updateEvent, publisherId, publisher, cancellationToken);
     }
 
-    /// <inheritdoc />
     internal Task ApplyPublisherUpdateEntryUpdateAsync(EventStreamEntry<DagCid> streamEntry, ValueUpdateEvent updateEvent, string publisherId, IReadOnlyPublisher updatedPublisher, CancellationToken cancellationToken)
     {
         cancellationToken.ThrowIfCancellationRequested();

src/Nomad/ModifiablePublisher.cs

@@ -149,7 +149,6 @@ public static ModifiablePublisher FromHandlerConfig(NomadKuboEventStreamHandlerC
             ChildPublishers = childPublishers,
             Client = client,
             KuboOptions = kuboOptions,
-            Inner = handlerConfig.RoamingValue,
             RoamingKey = handlerConfig.RoamingKey,
             LocalEventStream = handlerConfig.LocalValue,
             LocalEventStreamKey = handlerConfig.LocalKey,
@@ -189,7 +188,7 @@ public static ModifiablePublisher FromHandlerConfig(NomadKuboEventStreamHandlerC
     /// <summary>
     /// The roaming project data that this handler modifies.
     /// </summary>
-    public required Publisher Inner { get; init; }
+    public Publisher Inner => InnerPublisher.Inner;
 
     /// <inheritdoc/>
     public required IModifiablePublisherCollection<IReadOnlyPublisher> ParentPublishers { get; init; }

src/Nomad/ModifiableUser.cs

@@ -2,6 +2,7 @@
 using CommunityToolkit.Diagnostics;
 using Ipfs;
 using Ipfs.CoreApi;
+using OwlCore.ComponentModel;
 using OwlCore.Nomad;
 using OwlCore.Nomad.Kubo;
 using OwlCore.Nomad.Kubo.Events;
@@ -13,7 +14,7 @@ namespace WindowsAppCommunity.Sdk.Nomad;
 /// <summary>
 /// Represents a user that can be modified.
 /// </summary>
-public class ModifiableUser : NomadKuboEventStreamHandler<ValueUpdateEvent>, IModifiableUser
+public class ModifiableUser : NomadKuboEventStreamHandler<ValueUpdateEvent>, IModifiableUser, IDelegable<User>
 {
     /// <summary>
     /// Creates a new instance of the <see cref="ModifiableUser"/> class from the given handler configuration.
@@ -116,6 +117,11 @@ public static ModifiableUser FromHandlerConfig(NomadKuboEventStreamHandlerConfig
     /// <inheritdoc/>
     public required string Id { get; init; }
 
+    /// <summary>
+    /// The roaming user data that this handler modifies.
+    /// </summary>
+    public User Inner => InnerUser.Inner;
+
     /// <summary>
     /// The handler for reading user data.
     /// </summary>

src/Nomad/ReadOnlyProject.cs

@@ -212,8 +212,7 @@ public static ReadOnlyProject FromHandlerConfig(NomadKuboEventStreamHandlerConfi
         if (Inner.Publisher is null)
             return null;
 
-        var publisherId = await Client.Dag.GetAsync<Cid>(Inner.Publisher, cancel: cancellationToken);
-        return await PublisherRepository.GetAsync(publisherId, cancellationToken);
+        return await PublisherRepository.GetAsync(Inner.Publisher, cancellationToken);
     }
 
     /// <inheritdoc/>