test: verify video review is restored and prevent archive timestamp overwrite

Author: NazarTymoshchuk

VictoryCenter/VictoryCenter.BLL/Commands/Admin/VideoReviews/Delete/DeleteVideoReviewHandler.cs

@@ -39,6 +39,11 @@ public async Task<Result<long>> Handle(DeleteVideoReviewCommand request, Cancell
             return Result.Fail<long>(ErrorMessagesConstants.NotFound(request.Id, typeof(VideoReview)));
         }
 
+        if (entity.IsArchived)
+        {
+            return Result.Ok(entity.Id);
+        }
+
         entity.IsArchived = true;
         entity.ArchivedAt = _timeProvider.GetUtcNow();
 

VictoryCenter/VictoryCenter.IntegrationTests/ControllerTests/VideoReviews/VideoReviewManagementTests.cs

@@ -84,9 +84,11 @@ public async Task VideoReviewManagement_ShouldSupportAuthorizedCrud()
         Assert.Equal(HttpStatusCode.OK, restoreResponse.StatusCode);
         var restoredId = await restoreResponse.Content.ReadFromJsonAsync<long>();
         Assert.Equal(created.Id, restoredId);
-        Assert.False(await Fixture.DbContext.VideoReviews
+        var restoredEntity = await Fixture.DbContext.VideoReviews
             .AsNoTracking()
-            .AnyAsync(item => item.Id == created.Id && item.IsArchived));
+            .SingleAsync(item => item.Id == created.Id);
+        Assert.False(restoredEntity.IsArchived);
+        Assert.Null(restoredEntity.ArchivedAt);
     }
 
     [Fact]