Skip to content

Commit

Permalink
[skip ci] Fixed: Fix xUnit missing annotation warnings (Kareadita#2324)
Browse files Browse the repository at this point in the history
  • Loading branch information
senpai-notices authored Oct 18, 2023
1 parent e1c9db0 commit 961520a
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
6 changes: 3 additions & 3 deletions API.Tests/Services/ArchiveServiceTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -156,7 +156,7 @@ public void FindFirstEntry(string[] files, string expected)
}


//[Theory]
[Theory(Skip = "No specific reason")]
//[InlineData("v10.cbz", "v10.expected.png")] // Commented out as these break usually when NetVips is updated
//[InlineData("v10 - with folder.cbz", "v10 - with folder.expected.png")]
//[InlineData("v10 - nested folder.cbz", "v10 - nested folder.expected.png")]
Expand Down Expand Up @@ -189,7 +189,7 @@ public void GetCoverImage_Default_Test(string inputFile, string expectedOutputFi
}


//[Theory]
[Theory(Skip = "No specific reason")]
//[InlineData("v10.cbz", "v10.expected.png")] // Commented out as these break usually when NetVips is updated
//[InlineData("v10 - with folder.cbz", "v10 - with folder.expected.png")]
//[InlineData("v10 - nested folder.cbz", "v10 - nested folder.expected.png")]
Expand Down Expand Up @@ -375,7 +375,7 @@ public void FindCoverImageFilename(string[] filenames, string archiveName, strin

#region CreateZipForDownload

//[Fact]
[Fact(Skip = "No specific reason")]
public void CreateZipForDownloadTest()
{
var fileSystem = new MockFileSystem();
Expand Down
5 changes: 3 additions & 2 deletions API.Tests/Services/ReadingListServiceTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -876,6 +876,7 @@ public async Task DeleteReadingList_ShouldDelete()

#region UserHasReadingListAccess
// TODO: UserHasReadingListAccess tests are unavailable because I can't mock UserManager<AppUser>
[Fact(Skip = "Unable to mock UserManager<AppUser>")]
public async Task UserHasReadingListAccess_ShouldWorkIfTheirList()
{
await ResetDb();
Expand All @@ -889,7 +890,7 @@ public async Task UserHasReadingListAccess_ShouldWorkIfTheirList()
Assert.Single(userWithList.ReadingLists);
}


[Fact(Skip = "Unable to mock UserManager<AppUser>")]
public async Task UserHasReadingListAccess_ShouldNotWork_IfNotTheirList()
{
await ResetDb();
Expand All @@ -902,7 +903,7 @@ public async Task UserHasReadingListAccess_ShouldNotWork_IfNotTheirList()
Assert.Null(userWithList);
}


[Fact(Skip = "Unable to mock UserManager<AppUser>")]
public async Task UserHasReadingListAccess_ShouldWork_IfNotTheirList_ButUserIsAdmin()
{
await ResetDb();
Expand Down

0 comments on commit 961520a

Please sign in to comment.