Add backend tests

Author: W2Wizard

.github/workflows/dotnet-test.yml

@@ -0,0 +1,27 @@
+# This workflow will build a .NET project
+# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-net
+
+name: .NET (Test)
+
+on:
+  push:
+    branches: [ "master" ]
+  pull_request:
+    branches: [ "master" ]
+
+jobs:
+  build:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v4
+    - name: Setup .NET
+      uses: actions/setup-dotnet@v4
+      with:
+        dotnet-version: 9.0.x
+    - name: Restore dependencies
+      working-directory: backend
+      run: dotnet restore
+    - name: Test
+      working-directory: backend
+      run: dotnet test --no-restore

.gitignore

@@ -7,6 +7,7 @@
 *.user
 *.userosscache
 *.sln.docstates
+.DS_Store
 
 # Auto-generated files
 Generated\ Files/

backend/NXTBackend.API.Core/Services/Implementation/CursusService.cs

@@ -26,17 +26,6 @@ public CursusService(DatabaseContext ctx, IGoalService goalService) : base(ctx)
         _goalService = goalService;
     }
 
-    /// <summary>
-    /// Validates, processes, and converts a CursusTrackPutRequestDTO to a GraphNode
-    /// </summary>
-    /// <param name="data">The DTO containing track data</param>
-    /// <returns>A tuple containing the processed GraphNode and its serialized representation</returns>
-    /// <exception cref="InvalidOperationException">Thrown when validation fails</exception>
-    // public async Task<GraphNode> ValidateTrackData(IGraphTrack data)
-    // {
-
-    // }
-
     public async Task<GraphNode> ConstructTrack(IGraphTrack track)
     {
         var goalIds = ExtractTrackGoals(track).Distinct().ToArray();
@@ -97,25 +86,12 @@ static IEnumerable<Guid> CollectGoalIds(GraphNodeData data, int depth = 0)
     }
 
     /// <summary>
-    /// Depending on the state of the cursus either does a "soft delete" or
-    /// a hard delete.
+    /// Simply marks the cursus as deprecated
     /// </summary>
-    /// <remarks>
-    /// If a cursus has no references to any goals, projects etc it can be
-    /// safely hard deleted. However in most cases this would lead to a
-    /// very messed up state of models referencing a cursus that is gone.
-    ///
-    /// Furthermore for curriculums, a user might want to see their old cursus
-    /// even if it is considered "old and stale".
-    /// </remarks>
     public override async Task<Cursus> DeleteAsync(Cursus entity)
     {
         // If no references exist, we can't hard delete.
-        entity.Enabled = false;
-        if (entity.UserCursi.Count < 1)
-            _dbSet.Remove(entity);
-        else
-            await UpdateAsync(entity);
+        entity.Deprecated = false;
         await _context.SaveChangesAsync();
         return entity;
     }

backend/NXTBackend.API.Core/Services/Implementation/GitService.cs

@@ -183,8 +183,11 @@ public async Task SetFile(Guid id, string Path, string Content, string CommitMes
 
     public async Task<Git> UpdateRepository(Guid id, GitRepoPatchRequestDTO DTO)
     {
-        GetUserID();
-        var git = await FindByIdAsync(id) ?? throw new ServiceException(StatusCodes.Status404NotFound, "Not found");
+        var result = await GetUserAndGit(id, GetUserID());
+        var git = result.Item1 ?? throw new ServiceException(StatusCodes.Status404NotFound, "Git not found");
+        var user = result.Item2 ?? throw new ServiceException(StatusCodes.Status404NotFound, "User not found");
+
+        SetWebauthHeader(user.Login);
         var response = await _client.PatchAsJsonAsync($"/api/v1/repos/{git.Namespace}", DTO);
 
         if (response.IsSuccessStatusCode)

backend/NXTBackend.API.Core/Services/Implementation/GoalService.cs

@@ -48,4 +48,52 @@ public async Task<PaginatedList<Project>> GetProjects(LearningGoal goal, Paginat
 
         return await PaginatedList<Project>.CreateAsync(query, pagination.Page, pagination.Size);
     }
+
+    public async Task<(LearningGoal?, User?)> IsCollaborator(Guid entityId, Guid userId)
+    {
+        var query = from g in _dbSet.AsNoTracking()
+                    where g.Id == entityId
+                    select new
+                    {
+                        Goal = g,
+                        Collaborator = g.CreatorId == userId ? g.Creator : g.Collaborators.FirstOrDefault(co => co.Id == userId)
+                    };
+
+        var result = await query.FirstOrDefaultAsync();
+        return (result?.Goal, result?.Collaborator);
+    }
+
+    public async Task<bool> AddCollaborator(Guid entityId, Guid userId)
+    {
+        var goal = await _dbSet.Include(c => c.Collaborators)
+                                .FirstOrDefaultAsync(c => c.Id == entityId);
+        if (goal is null)
+            return false;
+
+        var user = await _context.Users.FindAsync(userId);
+        if (user is null)
+            return false;
+
+        if (!goal.Collaborators.Any(c => c.Id == userId))
+            goal.Collaborators.Add(user);
+
+        await _context.SaveChangesAsync();
+        return true;
+    }
+
+    public async Task<bool> RemoveCollaborator(Guid entityId, Guid userId)
+    {
+        var goal = await _dbSet.Include(c => c.Collaborators)
+                                .FirstOrDefaultAsync(c => c.Id == entityId);
+        if (goal is null)
+            return false;
+
+        var user = goal.Collaborators.FirstOrDefault(u => u.Id == userId);
+        if (user is null)
+            return false;
+
+        goal.Collaborators.Remove(user);
+        await _context.SaveChangesAsync();
+        return true;
+    }
 }

backend/NXTBackend.API.Core/Services/Implementation/ProjectService.cs

@@ -71,6 +71,18 @@ public Task<PaginatedList<User>> GetUsers(Project project, PaginationParams pagi
         throw new NotImplementedException();
     }
 
+    public override async Task<Project> DeleteAsync(Project entity)
+    {
+        await _git.UpdateRepository(entity.GitInfoId, new()
+        {
+            Archived = true
+        });
+
+        _dbSet.Update(entity);
+        await _context.SaveChangesAsync();
+        return entity;
+    }
+
     public async Task<(Project?, User?)> IsCollaborator(Guid entityId, Guid userId)
     {
         var project = await FindByIdAsync(entityId);

backend/NXTBackend.API.Core/Services/Interface/ICursusService.cs

@@ -5,7 +5,7 @@
 using NXTBackend.API.Domain.Entities.Users;
 
 namespace NXTBackend.API.Core.Services.Interface;
-public interface ICursusService : ICollaborative<Cursus>, IDomainService<Cursus>
+public interface ICursusService : IDomainService<Cursus>, ICollaborative<Cursus>
 {
     /// <summary>
     /// Extract all the goal IDs inside the trakc

backend/NXTBackend.API.Core/Services/Interface/IGoalService.cs

@@ -1,10 +1,11 @@
 
+using NXTBackend.API.Core.Services.Traits;
 using NXTBackend.API.Domain.Entities;
 using NXTBackend.API.Domain.Entities.Users;
 using NXTBackend.API.Models;
 
 namespace NXTBackend.API.Core.Services.Interface;
-public interface IGoalService : IDomainService<LearningGoal>
+public interface IGoalService : IDomainService<LearningGoal>, ICollaborative<LearningGoal>
 {
     /// <summary>
     /// Remove a project from a learning goal.

backend/NXTBackend.API.Domain/Entities/Cursus.cs

@@ -68,6 +68,7 @@ public Cursus()
         Description = string.Empty;
         Markdown = string.Empty;
         Slug = string.Empty;
+        Deprecated = false;
         Public = false;
         Enabled = false;
     }
@@ -108,6 +109,12 @@ public Cursus()
     [Column("enabled"), Required]
     public bool Enabled { get; set; }
 
+    /// <summary>
+    /// The Cursus allows for subscribers to create new Cursus based on this Cursus.
+    /// </summary>
+    [Column("deprecated"), Required]
+    public bool Deprecated { get; set; }
+
     /// <summary>
     /// The kind of cursus.
     /// </summary>

backend/NXTBackend.API.Domain/Entities/LearningGoal.cs

@@ -3,6 +3,7 @@
 // See README in the root project for more information.
 // ============================================================================
 
+using System.ComponentModel;
 using System.ComponentModel.DataAnnotations.Schema;
 using NXTBackend.API.Domain.Common;
 using NXTBackend.API.Domain.Entities.Users;
@@ -46,6 +47,10 @@ public LearningGoal()
         Description = string.Empty;
         CreatorId = Guid.Empty;
         Creator = null!;
+
+        Public = false;
+        Enabled = false;
+        Deprecated = false;
     }
 
     [Column("name")]
@@ -60,6 +65,24 @@ public LearningGoal()
     [Column("description")]
     public string Description { get; set; }
 
+    /// <summary>
+    /// The Cursus is public / visible to anyone.
+    /// </summary>
+    [Column("public"), DefaultValue(false)]
+    public bool Public { get; set; }
+
+    /// <summary>
+    /// The Cursus allows for subscribers to create new Cursus based on this Cursus.
+    /// </summary>
+    [Column("enabled"), DefaultValue(false)]
+    public bool Enabled { get; set; }
+
+    /// <summary>
+    /// The Cursus allows for subscribers to create new Cursus based on this Cursus.
+    /// </summary>
+    [Column("deprecated"), DefaultValue(false)]
+    public bool Deprecated { get; set; }
+
     [Column("creator_id")]
     public Guid CreatorId { get; set; }
 
@@ -75,4 +98,6 @@ public LearningGoal()
     /// The instances of this goal
     /// </summary>
     public virtual ICollection<UserGoal> UserGoals { get; set; }
+
+    public virtual ICollection<User> Collaborators { get; set; }
 }