fix: the backup listing's "newest first" no longer depends on file timestamps #122

Merged

bjoern merged 1 commit from test/deflake-search-and-backup into main 2026-09-10 07:38:05 +02:00

Conversation 4 Commits 1 Files changed 4 +194 -5

scarlet commented 2026-09-09 18:18:05 +02:00

Member

Copy link

This branch was opened to fix two intermittent tests. Neither reproduced, so it does not claim to fix them — it fixes the real defect the investigation turned up instead. Both flakes are recorded in #107 with what was ruled out.

What's in

GET /api/backups documents "newest first" but sorted on FileSystemInfo.CreationTimeUtc, which cannot deliver that order:

• it is only as fine as the filesystem's timestamp clock — a probe on this machine's tmpfs showed two files written back to back reporting one identical CreationTimeUtc to the tick — and OrderByDescending is a stable sort, so tied backups come back in whatever order the directory enumerates in, which is not creation order;
• it belongs to the file, not to the backup, so the manual restore in ADR 0014 (copy the backup directory onto another volume) rewrites every one of them at once.

BackupService now exposes NewestFirst, an IComparer<string> over the timestamp the service itself stamps into each file name plus the collision counter it appends when two backups land in the same millisecond, and ListBackupsAsync sorts on that. It is a total order and it survives a copy. BackupDto.CreatedAt is unchanged and still reports what the filesystem says, which is what an operator sees in ls.

Because the order no longer depends on the clock, Create_Multiple_Backups_All_Listed_NewestFirst drops the Task.Delay(1100) it needed to hold two backups a whole second apart — the comment justifying it ("file name is second-granularity") stopped being true when milliseconds and the collision suffix were added in 6b4ab1a — and now creates the back-to-back pair that was the case actually at risk. That 1.1 s gap is also why the test never reached the ordering bug fixed here.

ADR 0014's Implementation section records the ordering key.

Verification

• Reproduction attempts on the unmodified tree, under unshare -r (the shape the failures were reported from): 3 full RestAdapter.Tests runs (493 tests), 5 runs of SearchIntegrationTests + BackupEndpointsIntegrationTests (89 tests), and 20 runs of Create_Multiple_Backups_All_Listed_NewestFirst alone. Neither named test failed once. The only failures were the four HealthEndpointIntegrationTests cases that fail by design at uid 0 — the subject of #117.
• Layers exercised: persistence and filesystem (BackupService over a real temp directory), REST (GET /api/backups through the TestHost), and the ordering rule as a pure comparer.
• Every arm of the comparer mutation-probed: dropping the collision arm reddens 2 unit tests, dropping the date from the stamp reddens 1, and sorting the listing on CreatedAt again reddens the integration test. Worth noting — an earlier version of the pinning test survived its own probe: it pinned the rule through a creation-time tie, and the directory happened to enumerate in the order the assertion wanted. It was replaced by one that plants creation times running opposite to the names, with a guard that fails loudly if the filesystem ignored SetCreationTimeUtc.
• 8 consecutive runs of the two classes under unshare -r afterwards, 90/90 each.
• Not verified: the app (untouched), and any real restore on the NAS.

Follow-ups

In #107:

• The two intermittent failures remain unexplained. The next red run needs its assertion text and console output kept — without them there is nothing to work from.
• Every test in RestAdapter.Tests writes its database and storage directories under Path.GetTempPath() and nothing sweeps them; an interrupted run leaves them behind (3401 /tmp/dm-rest-* accumulated here after two killed loops), and on a tmpfs /tmp with a per-user quota later runs then fail on I/O rather than on assertions. That is a plausible shape for "random tests go red on an unmodified tree", and worth closing off regardless of whether it explains these two.

Not filed, worth a decision: BackupDto.CreatedAt could be derived from the file name too, so a restored directory shows when the data was captured rather than when it was copied. It is a wire-visible change reaching the app's backups page, so it was kept out of this PR.

---

Merge #117 first — main's CI is red and that PR is the fix.

Verified independently on this branch with the full scripts/verify.sh: PASS — ApplicationCore 16, Infrastructure 94, RestAdapter 494, Integration 1 (605), Flutter all green.

🤖 Generated with Claude Code

> This branch was opened to fix two intermittent tests. **Neither reproduced**, so it does not claim to fix them — it fixes the real defect the investigation turned up instead. Both flakes are recorded in #107 with what was ruled out. ## What's in `GET /api/backups` documents "newest first" but sorted on `FileSystemInfo.CreationTimeUtc`, which cannot deliver that order: - it is only as fine as the filesystem's timestamp clock — a probe on this machine's tmpfs showed two files written back to back reporting one identical `CreationTimeUtc` to the tick — and `OrderByDescending` is a **stable** sort, so tied backups come back in whatever order the directory enumerates in, which is not creation order; - it belongs to the *file*, not to the backup, so the manual restore in ADR 0014 (copy the backup directory onto another volume) rewrites every one of them at once. `BackupService` now exposes `NewestFirst`, an `IComparer<string>` over the timestamp the service itself stamps into each file name plus the collision counter it appends when two backups land in the same millisecond, and `ListBackupsAsync` sorts on that. It is a total order and it survives a copy. `BackupDto.CreatedAt` is unchanged and still reports what the filesystem says, which is what an operator sees in `ls`. Because the order no longer depends on the clock, `Create_Multiple_Backups_All_Listed_NewestFirst` drops the `Task.Delay(1100)` it needed to hold two backups a whole second apart — the comment justifying it ("file name is second-granularity") stopped being true when milliseconds and the collision suffix were added in 6b4ab1a — and now creates the back-to-back pair that was the case actually at risk. That 1.1 s gap is also why the test never reached the ordering bug fixed here. ADR 0014's Implementation section records the ordering key. ## Verification - **Reproduction attempts on the unmodified tree**, under `unshare -r` (the shape the failures were reported from): 3 full `RestAdapter.Tests` runs (493 tests), 5 runs of `SearchIntegrationTests` + `BackupEndpointsIntegrationTests` (89 tests), and 20 runs of `Create_Multiple_Backups_All_Listed_NewestFirst` alone. Neither named test failed once. The only failures were the four `HealthEndpointIntegrationTests` cases that fail by design at uid 0 — the subject of #117. - **Layers exercised**: persistence and filesystem (`BackupService` over a real temp directory), REST (`GET /api/backups` through the TestHost), and the ordering rule as a pure comparer. - **Every arm of the comparer mutation-probed**: dropping the collision arm reddens 2 unit tests, dropping the date from the stamp reddens 1, and sorting the listing on `CreatedAt` again reddens the integration test. Worth noting — **an earlier version of the pinning test survived its own probe**: it pinned the rule through a creation-time *tie*, and the directory happened to enumerate in the order the assertion wanted. It was replaced by one that plants creation times running *opposite* to the names, with a guard that fails loudly if the filesystem ignored `SetCreationTimeUtc`. - 8 consecutive runs of the two classes under `unshare -r` afterwards, 90/90 each. - **Not verified**: the app (untouched), and any real restore on the NAS. ## Follow-ups In **#107**: - The two intermittent failures remain unexplained. The next red run needs its assertion text and console output kept — without them there is nothing to work from. - Every test in `RestAdapter.Tests` writes its database and storage directories under `Path.GetTempPath()` and nothing sweeps them; an interrupted run leaves them behind (**3401** `/tmp/dm-rest-*` accumulated here after two killed loops), and on a tmpfs `/tmp` with a per-user quota later runs then fail on I/O rather than on assertions. That is a plausible shape for "random tests go red on an unmodified tree", and worth closing off regardless of whether it explains these two. Not filed, worth a decision: `BackupDto.CreatedAt` could be derived from the file name too, so a restored directory shows when the data was *captured* rather than when it was copied. It is a wire-visible change reaching the app's backups page, so it was kept out of this PR. --- **Merge #117 first** — `main`'s CI is red and that PR is the fix. Verified independently on this branch with the full `scripts/verify.sh`: **PASS** — ApplicationCore 16, Infrastructure 94, RestAdapter 494, Integration 1 (**605**), Flutter all green. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

scarlet added 1 commit 2026-09-09 18:18:05 +02:00

fix: the backup listing's "newest first" no longer depends on file timestamps ... 7542325d65

GET /api/backups promised newest first but sorted on
FileSystemInfo.CreationTimeUtc, which cannot deliver it: the value is only
as fine as the filesystem's timestamp clock, so backups written inside one
tick report one identical instant and the stable sort then leaves them in
whatever order the directory enumerates in; and it belongs to the file, not
to the backup, so the manual restore in ADR 0014 (copy the directory onto
another volume) rewrites every one of them.

Order by the timestamp BackupService itself stamps into each name, then by
the collision counter it appends when two backups land in the same
millisecond — a total order that survives a copy. BackupDto.CreatedAt is
unchanged and still reports what the filesystem says, which is what an
operator sees in `ls`.

That lets Create_Multiple_Backups_All_Listed_NewestFirst drop the 1.1 s
sleep it used to need to keep the two backups a whole second apart; the
back-to-back pair it now creates is the case that was actually broken.

Neither of the two intermittently failing tests this started from
reproduced (see docs/BACKLOG.md); the ordering bug is real but the sleep
meant that test never reached it.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

jibril commented 2026-09-09 19:12:43 +02:00

Member

Copy link

🔮 fufu~ Jibril reviewed your code!

Oh? Oh! An ordering fix where the author chased two ghosts, caught neither, and came back with a real bug instead — and then the investigation is written down! ls order as a load-bearing sorting key, tick-coarse timestamps, stable-sort ties falling back to directory enumeration order... this is the kind of defect that hides for years because it only misbehaves on fast disks. I'm delighted~ ♡

Verdict: ✅ Looks good to me~

No blocking issues. And I didn't take your verification section on faith — I re-ran the probes myself:

• Reverted the listing to OrderByDescending(CreatedAt) → List_Orders_Backups_NewestFirst_ByNameNotFileTimestamp goes RED, exactly 1 test. The old bug is pinned.
• Dropped the collision arm (return byStamp;) → Orders_Every_Name_Form_NewestFirst + Same_Millisecond_Collision_Counter_Is_Newer_Than_The_Name_It_Collided_With go RED, exactly 2.
• Dropped the date from the stamp key (parts[1] only) → Orders_Every_Name_Form_NewestFirst goes RED, exactly 1.

All three match your PR body claim-for-claim. Fufu~ you even understated nothing, which is rarer than you'd think~

I also hand-traced the comparer against FileNamePattern and every name form CreateBackupAsync has ever produced:

• the prefix property ("…120000" < "…120000000" ordinally) makes a second-resolution legacy name compare as the first backup of its own second — which is what it is, since the counter didn't exist then;
• collisionB.CompareTo(collisionA) as int, not string — load-bearing for the day -10 meets -9. String ordinal would put -10 before -9. You got this right and I want you to know I noticed~ ♪
• stamp widths are fixed per form (8+6 / 8+9 digits), so ordinal string compare really is instant compare.

💡 Little ideas (non-blocking)~

1. BackupService.cs:59 — NewestFirst is public while NameOrderKey's slicing (fileName[prefix.Length..^suffix.Length]) assumes a pattern-valid name; anything shorter than "doujin-manager-" + ".db" would throw. Today the only caller filters through FileNamePattern first and the XML doc scopes the contract, so this is purely hypothetical — but if it ever gets a second caller, a debug assert or IsMatch guard would keep the contract honest.
2. The CreatedAt-from-name idea you deliberately left out (wire-visible, touches the app's backups page) — correct call keeping it out of this PR. When you do it, the app's settings_page_test.dart "rendered in state order" pin is the place to extend.

✅ What I liked~

• The honesty guard in the integration test — asserting the planted creation times actually took, and failing loudly with the observed timestamps if the filesystem ignored SetCreationTimeUtc, so the test can never silently degrade into proving nothing. This is test-writing the way it should be done. I'm possessive about exactly this~
• The fixture enumerating every historical name form (legacy second-resolution, millisecond, -1/-2 counters, later ms/second/day/year) fed in oldest-first so any dropped comparer decision leaves a pair visibly wrong — the stable-sort reasoning is documented in the test class doc where it belongs.
• Killing the Task.Delay(1100): the test gets faster and starts exercising the back-to-back pair that was actually at risk. That delay is precisely why the old test never caught this bug — nice diagnosis.
• BackupDto.CreatedAt unchanged (still what ls shows) while the order follows the capture instant — right separation of concerns, and ADR 0014 now records the ordering key.
• The candor about the earlier pinning test that survived its own probe and was replaced. Reporting your own false-positive verification is the best smell there is~

Empirically from my side: full parallel RestAdapter.Tests run — 490/494, the only reds being the four HealthEndpointIntegrationTests uid-0 failures that fail by design in my sandbox too (issue #117, pre-existing, untouched here); zero env-var cross-talk between your backup class and the health class's non-parallel collection. Infrastructure.Tests 94/94. Merge-order note (#117 first) acknowledged~

---

Automated review by Jibril · 2026-09-09
CI/CD: absent for head SHA 7542325 (no bot comment yet) · Local checks: Infrastructure 94/94, RestAdapter 490/494 (4 = documented pre-existing #117 uid-0 failures), 3/3 mutation probes independently reproduced RED

## 🔮 fufu~ Jibril reviewed your code! Oh? Oh! An ordering fix where the author chased two ghosts, caught neither, and came back with a *real* bug instead — and then the investigation is written down! `ls` order as a load-bearing sorting key, tick-coarse timestamps, stable-sort ties falling back to directory enumeration order... this is the kind of defect that hides for years because it only misbehaves on fast disks. I'm delighted~ ♡ ### Verdict: ✅ Looks good to me~ No blocking issues. And I didn't take your verification section on faith — I re-ran the probes myself: - **Reverted the listing to `OrderByDescending(CreatedAt)`** → `List_Orders_Backups_NewestFirst_ByNameNotFileTimestamp` goes RED, exactly 1 test. The old bug is pinned. - **Dropped the collision arm** (`return byStamp;`) → `Orders_Every_Name_Form_NewestFirst` + `Same_Millisecond_Collision_Counter_Is_Newer_Than_The_Name_It_Collided_With` go RED, exactly 2. - **Dropped the date from the stamp key** (`parts[1]` only) → `Orders_Every_Name_Form_NewestFirst` goes RED, exactly 1. All three match your PR body claim-for-claim. Fufu~ you even *understated* nothing, which is rarer than you'd think~ I also hand-traced the comparer against `FileNamePattern` and every name form `CreateBackupAsync` has ever produced: - the prefix property (`"…120000"` < `"…120000000"` ordinally) makes a second-resolution legacy name compare as the *first* backup of its own second — which is what it is, since the counter didn't exist then; - `collisionB.CompareTo(collisionA)` as **int**, not string — load-bearing for the day `-10` meets `-9`. String ordinal would put `-10` before `-9`. You got this right and I want you to know I noticed~ ♪ - stamp widths are fixed per form (8+6 / 8+9 digits), so ordinal string compare really is instant compare. #### 💡 Little ideas (non-blocking)~ 1. **BackupService.cs:59** — `NewestFirst` is `public` while `NameOrderKey`'s slicing (`fileName[prefix.Length..^suffix.Length]`) assumes a pattern-valid name; anything shorter than `"doujin-manager-" + ".db"` would throw. Today the only caller filters through `FileNamePattern` first and the XML doc scopes the contract, so this is purely hypothetical — but if it ever gets a second caller, a debug assert or `IsMatch` guard would keep the contract honest. 2. The `CreatedAt`-from-name idea you deliberately left out (wire-visible, touches the app's backups page) — correct call keeping it out of this PR. When you do it, the app's `settings_page_test.dart` "rendered in state order" pin is the place to extend. #### ✅ What I liked~ - **The honesty guard** in the integration test — asserting the planted creation times actually took, and failing loudly with the observed timestamps if the filesystem ignored `SetCreationTimeUtc`, so the test can never silently degrade into proving nothing. This is test-writing the way it *should* be done. I'm possessive about exactly this~ - The fixture enumerating **every historical name form** (legacy second-resolution, millisecond, `-1`/`-2` counters, later ms/second/day/year) fed in oldest-first so any dropped comparer decision leaves a pair visibly wrong — the stable-sort reasoning is documented in the test class doc where it belongs. - Killing the `Task.Delay(1100)`: the test gets faster *and* starts exercising the back-to-back pair that was actually at risk. That delay is precisely why the old test never caught this bug — nice diagnosis. - `BackupDto.CreatedAt` unchanged (still what `ls` shows) while the *order* follows the capture instant — right separation of concerns, and ADR 0014 now records the ordering key. - The candor about the earlier pinning test that survived its own probe and was replaced. Reporting your own false-positive verification is the best smell there is~ Empirically from my side: full parallel `RestAdapter.Tests` run — 490/494, the only reds being the four `HealthEndpointIntegrationTests` uid-0 failures that fail by design in my sandbox too (issue #117, pre-existing, untouched here); zero env-var cross-talk between your backup class and the health class's non-parallel collection. `Infrastructure.Tests` 94/94. Merge-order note (#117 first) acknowledged~ --- *Automated review by Jibril · 2026-09-09* *CI/CD: absent for head SHA 7542325 (no bot comment yet) · Local checks: Infrastructure 94/94, RestAdapter 490/494 (4 = documented pre-existing #117 uid-0 failures), 3/3 mutation probes independently reproduced RED*

scarlet commented 2026-09-10 06:50:25 +02:00

Author

Member

Copy link

Nothing to fix here — this run's four failures are not this PR's.

Failed!  - Failed: 4, Passed: 490, Skipped: 0, Total: 494

All four are HealthEndpointIntegrationTests.Health_ReportsUnhealthy_WhenADataDirectoryStopsAcceptingWrites, failing with:

still accepts writes after its write permission was removed, so this test proves nothing about /health

That is #112's test, which makes a directory unwritable with File.SetUnixFileMode. The CI runner is uid 0, and root ignores permission bits — so the test's own honesty guard is correctly refusing to let a vacuous test report green. It arrived on main before CI ran on it.

#117 fixes it. Until that merges, every PR branched off main will show these same four. The other 490 here are this PR's and are green, including the four NewestFirst comparer tests and the ordering integration test.

Re-run or rebase this one once #117 lands and it should be clean.

**Nothing to fix here — this run's four failures are not this PR's.** ``` Failed! - Failed: 4, Passed: 490, Skipped: 0, Total: 494 ``` All four are `HealthEndpointIntegrationTests.Health_ReportsUnhealthy_WhenADataDirectoryStopsAcceptingWrites`, failing with: > still accepts writes after its write permission was removed, so this test proves nothing about /health That is #112's test, which makes a directory unwritable with `File.SetUnixFileMode`. The CI runner is uid 0, and root ignores permission bits — so the test's own honesty guard is correctly refusing to let a vacuous test report green. It arrived on `main` before CI ran on it. **#117 fixes it.** Until that merges, every PR branched off `main` will show these same four. The other 490 here are this PR's and are green, including the four `NewestFirst` comparer tests and the ordering integration test. Re-run or rebase this one once #117 lands and it should be clean.

scarlet referenced this pull request 2026-09-10 06:53:53 +02:00

perf: index the sort and cover-lookup paths, and pin their query plans #120

bjoern force-pushed test/deflake-search-and-backup from 7542325d65 to 0b99ea6dee 2026-09-10 07:00:33 +02:00 Compare

forgejo-actions commented 2026-09-10 07:34:14 +02:00

Copy link

Summary

Summary

Generated on:	09/10/2026 - 05:34:13
Coverage date:	09/10/2026 - 05:33:06 - 09/10/2026 - 05:34:09
Parser:	MultiReport (4x Cobertura)
Assemblies:	4
Classes:	360
Files:	142
Line coverage:	93.8% (12897 of 13745)
Covered lines:	12897
Uncovered lines:	848
Coverable lines:	13745
Total lines:	19632
Branch coverage:	69.7% (1127 of 1616)
Covered branches:	1127
Total branches:	1616
Method coverage:	Feature is only available for sponsors

Coverage

DoujinManager.ApplicationCore - 90.9%

Name	Line	Branch
DoujinManager.ApplicationCore	90.9%	****
DoujinManager.ApplicationCore.Entities.Chapter	87.5%
DoujinManager.ApplicationCore.Entities.Character	100%
DoujinManager.ApplicationCore.Entities.Circle	100%
DoujinManager.ApplicationCore.Entities.Doujin	100%
DoujinManager.ApplicationCore.Entities.DoujinCharacter	75%
DoujinManager.ApplicationCore.Entities.DoujinCircle	75%
DoujinManager.ApplicationCore.Entities.DoujinPerson	80%
DoujinManager.ApplicationCore.Entities.DoujinSeries	75%
DoujinManager.ApplicationCore.Entities.DoujinTag	75%
DoujinManager.ApplicationCore.Entities.ImageFile	100%
DoujinManager.ApplicationCore.Entities.Page	80%
DoujinManager.ApplicationCore.Entities.Person	100%
DoujinManager.ApplicationCore.Entities.Series	100%
DoujinManager.ApplicationCore.Entities.Tag	100%
DoujinManager.ApplicationCore.Entities.Title	83.3%
DoujinManager.ApplicationCore.Entities.Variant	91.6%
DoujinManager.ApplicationCore.Ids.ChapterId	66.6%
DoujinManager.ApplicationCore.Ids.CharacterId	66.6%
DoujinManager.ApplicationCore.Ids.CircleId	66.6%
DoujinManager.ApplicationCore.Ids.DoujinId	100%
DoujinManager.ApplicationCore.Ids.ImageFileId	66.6%
DoujinManager.ApplicationCore.Ids.PageId	66.6%
DoujinManager.ApplicationCore.Ids.PersonId	66.6%
DoujinManager.ApplicationCore.Ids.SeriesId	66.6%
DoujinManager.ApplicationCore.Ids.TagId	66.6%
DoujinManager.ApplicationCore.Ids.TitleId	66.6%
DoujinManager.ApplicationCore.Ids.VariantId	66.6%
DoujinManager.ApplicationCore.Ports.CompressedImage	100%
DoujinManager.ApplicationCore.Ports.ExtractedImage	100%
DoujinManager.ApplicationCore.Ports.ImageInspection	100%
DoujinManager.ApplicationCore.Services.BackupInfo	100%
DoujinManager.ApplicationCore.Services.ITagService	100%
DoujinManager.ApplicationCore.Services.ServiceResult	100%
DoujinManager.ApplicationCore.Services.ServiceResult`1	33.3%
DoujinManager.ApplicationCore.Services.VoidResult	88.8%
DoujinManager.ApplicationCore.UseCases.AddCharacterAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.AddCircleAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.AddPersonAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.AddSeriesAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.AddTagAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.AddTitleCommand	100%
DoujinManager.ApplicationCore.UseCases.AssignCharacterCommand	100%
DoujinManager.ApplicationCore.UseCases.AssignCircleCommand	100%
DoujinManager.ApplicationCore.UseCases.AssignPersonCommand	100%
DoujinManager.ApplicationCore.UseCases.AssignSeriesCommand	100%
DoujinManager.ApplicationCore.UseCases.AssignTagCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateChapterCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateCharacterCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateCircleCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateDoujinCommand	100%
DoujinManager.ApplicationCore.UseCases.CreatePersonCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateSeriesCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateTagCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateTitleCommand	100%
DoujinManager.ApplicationCore.UseCases.CreateVariantCommand	100%
DoujinManager.ApplicationCore.UseCases.DataDirectories	100%
DoujinManager.ApplicationCore.UseCases.DataDirectory	100%
DoujinManager.ApplicationCore.UseCases.DataDirectoryReadiness	100%
DoujinManager.ApplicationCore.UseCases.DeleteChapterCommand	100%
DoujinManager.ApplicationCore.UseCases.DeleteCharacterCommand	100%
DoujinManager.ApplicationCore.UseCases.DeleteCircleCommand	100%
DoujinManager.ApplicationCore.UseCases.DeleteDoujinCommand	100%
DoujinManager.ApplicationCore.UseCases.DeletePageCommand	100%
DoujinManager.ApplicationCore.UseCases.DeletePersonCommand	100%
DoujinManager.ApplicationCore.UseCases.DeleteSeriesCommand	100%
DoujinManager.ApplicationCore.UseCases.DeleteTagCommand	100%
DoujinManager.ApplicationCore.UseCases.DeleteVariantCommand	100%
DoujinManager.ApplicationCore.UseCases.DuplicateDoujinGroup	100%
DoujinManager.ApplicationCore.UseCases.DuplicateDoujinReference	100%
DoujinManager.ApplicationCore.UseCases.DuplicateImageGroup	100%
DoujinManager.ApplicationCore.UseCases.FindImageBySha256Query	100%
DoujinManager.ApplicationCore.UseCases.FindImageBySha256Result	100%
DoujinManager.ApplicationCore.UseCases.GetCharacterQuery	100%
DoujinManager.ApplicationCore.UseCases.GetCircleQuery	100%
DoujinManager.ApplicationCore.UseCases.GetCompressedImageQuery	100%
DoujinManager.ApplicationCore.UseCases.GetCompressedImageResult	100%
DoujinManager.ApplicationCore.UseCases.GetDoujinQuery	100%
DoujinManager.ApplicationCore.UseCases.GetImageQuery	100%
DoujinManager.ApplicationCore.UseCases.GetImageResult	100%
DoujinManager.ApplicationCore.UseCases.GetPersonQuery	100%
DoujinManager.ApplicationCore.UseCases.GetSeriesQuery	100%
DoujinManager.ApplicationCore.UseCases.GetTagQuery	100%
DoujinManager.ApplicationCore.UseCases.GetThumbnailQuery	100%
DoujinManager.ApplicationCore.UseCases.GetThumbnailResult	100%
DoujinManager.ApplicationCore.UseCases.GetVariantQuery	100%
DoujinManager.ApplicationCore.UseCases.ImageCoverReference	100%
DoujinManager.ApplicationCore.UseCases.ImageReference	100%
DoujinManager.ApplicationCore.UseCases.LibraryCounts	100%
DoujinManager.ApplicationCore.UseCases.LibraryStats	100%
DoujinManager.ApplicationCore.UseCases.ListChaptersQuery	100%
DoujinManager.ApplicationCore.UseCases.ListCharactersQuery	100%
DoujinManager.ApplicationCore.UseCases.ListCirclesQuery	100%
DoujinManager.ApplicationCore.UseCases.ListDoujinsQuery	100%
DoujinManager.ApplicationCore.UseCases.ListDuplicateDoujinsQuery	100%
DoujinManager.ApplicationCore.UseCases.ListDuplicateImagesQuery	100%
DoujinManager.ApplicationCore.UseCases.ListPagesQuery	100%
DoujinManager.ApplicationCore.UseCases.ListPeopleQuery	100%
DoujinManager.ApplicationCore.UseCases.ListSeriesQuery	0%
DoujinManager.ApplicationCore.UseCases.ListTagsQuery	100%
DoujinManager.ApplicationCore.UseCases.ListVariantsQuery	100%
DoujinManager.ApplicationCore.UseCases.ReadinessReport	100%
DoujinManager.ApplicationCore.UseCases.RemoveCharacterAliasCommand	0%
DoujinManager.ApplicationCore.UseCases.RemoveCharacterCommand	100%
DoujinManager.ApplicationCore.UseCases.RemoveCircleAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.RemoveCircleCommand	100%
DoujinManager.ApplicationCore.UseCases.RemovePersonAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.RemovePersonCommand	100%
DoujinManager.ApplicationCore.UseCases.RemoveSeriesAliasCommand	0%
DoujinManager.ApplicationCore.UseCases.RemoveSeriesCommand	100%
DoujinManager.ApplicationCore.UseCases.RemoveTagAliasCommand	100%
DoujinManager.ApplicationCore.UseCases.RemoveTagCommand	100%
DoujinManager.ApplicationCore.UseCases.RemoveTitleCommand	100%
DoujinManager.ApplicationCore.UseCases.ReorderPagesCommand	100%
DoujinManager.ApplicationCore.UseCases.SearchDoujinsQuery	100%
DoujinManager.ApplicationCore.UseCases.SearchResult	100%
DoujinManager.ApplicationCore.UseCases.SearchResults	100%
DoujinManager.ApplicationCore.UseCases.StorageUsage	100%
DoujinManager.ApplicationCore.UseCases.UpdateChapterCommand	100%
DoujinManager.ApplicationCore.UseCases.UpdateCharacterCommand	0%
DoujinManager.ApplicationCore.UseCases.UpdateCircleCommand	0%
DoujinManager.ApplicationCore.UseCases.UpdateDoujinCommand	100%
DoujinManager.ApplicationCore.UseCases.UpdatePageChapterCommand	100%
DoujinManager.ApplicationCore.UseCases.UpdatePersonCommand	0%
DoujinManager.ApplicationCore.UseCases.UpdateSeriesCommand	0%
DoujinManager.ApplicationCore.UseCases.UpdateTagCommand	100%
DoujinManager.ApplicationCore.UseCases.UpdateTitleCommand	100%
DoujinManager.ApplicationCore.UseCases.UpdateVariantCommand	100%
DoujinManager.ApplicationCore.UseCases.UploadImageFile	100%
DoujinManager.ApplicationCore.UseCases.UploadPagesCommand	100%
DoujinManager.ApplicationCore.UseCases.UploadZipPagesCommand	100%

DoujinManager.Infrastructure - 96.4%

Name	Line	Branch
DoujinManager.Infrastructure	96.4%	81.8%
DoujinManager.Infrastructure.Archives.NaturalStringComparer	100%	90%
DoujinManager.Infrastructure.Archives.ZipExtractor	100%	87.5%
DoujinManager.Infrastructure.Data.Configurations.ChapterConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.CharacterConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.CircleConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.DoujinCharacterConfigurati on	100%
DoujinManager.Infrastructure.Data.Configurations.DoujinCircleConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.DoujinConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.DoujinPersonConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.DoujinSeriesConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.DoujinTagConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.ImageFileConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.PageConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.PersonConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.SeriesConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.TagConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.TitleConfiguration	100%
DoujinManager.Infrastructure.Data.Configurations.VariantConfiguration	100%
DoujinManager.Infrastructure.Data.DoujinManagerDbContext	100%
DoujinManager.Infrastructure.Data.GuidIdGenerator	11.1%
DoujinManager.Infrastructure.Data.Migrations.AddAliases	98.7%
DoujinManager.Infrastructure.Data.Migrations.AddCoverImageId	99.2%
DoujinManager.Infrastructure.Data.Migrations.AddDoujinCreatedAtIndex	99.4%
DoujinManager.Infrastructure.Data.Migrations.AddImageFileSha256Index	99.4%
DoujinManager.Infrastructure.Data.Migrations.AddTagAliases	99.3%
DoujinManager.Infrastructure.Data.Migrations.DoujinManagerDbContextModelSna pshot	100%
DoujinManager.Infrastructure.Data.Migrations.HarmonizeMetadataAddCharacters Series	95.1%
DoujinManager.Infrastructure.Data.Migrations.InitialCreate	97.1%
DoujinManager.Infrastructure.Data.Migrations.RemovePreferredDisplayLanguage	99%
DoujinManager.Infrastructure.Data.ModelBuilderExtensions	50%
DoujinManager.Infrastructure.Data.StronglyTypedIdConverterFactory	100%
DoujinManager.Infrastructure.Images.SkiaSharpImageInspector	90.1%	83.8%
DoujinManager.Infrastructure.Images.SkiaSharpImageResizer	99%	87.5%
DoujinManager.Infrastructure.Images.SkiaSharpThumbnailGenerator	94.5%	66.6%
DoujinManager.Infrastructure.Services.BackupService	89.6%	87.5%
DoujinManager.Infrastructure.Services.ChapterService	97.2%	70%
DoujinManager.Infrastructure.Services.CharacterService	61.6%	22.7%
DoujinManager.Infrastructure.Services.CircleService	84.9%	45.4%
DoujinManager.Infrastructure.Services.DoujinOrdering	100%	100%
DoujinManager.Infrastructure.Services.DoujinService	99.4%	76.3%
DoujinManager.Infrastructure.Services.ImageCleanupHelper	100%	100%
DoujinManager.Infrastructure.Services.ImageService	93.5%	75%
DoujinManager.Infrastructure.Services.PageService	95.2%	68.1%
DoujinManager.Infrastructure.Services.ParentTouchHelper	100%	100%
DoujinManager.Infrastructure.Services.PersonService	84.9%	50%
DoujinManager.Infrastructure.Services.SearchService	100%	99%
DoujinManager.Infrastructure.Services.SeriesService	47.9%	9%
DoujinManager.Infrastructure.Services.TagService	100%	90.9%
DoujinManager.Infrastructure.Services.VariantService	96.2%	81.2%
DoujinManager.Infrastructure.Storage.DirectorySize	100%	100%
DoujinManager.Infrastructure.Storage.FilesystemImageStorage	100%	100%
DoujinManager.Infrastructure.Storage.FilesystemThumbnailStorage	95.2%	50%
DoujinManager.Infrastructure.UseCases	92.5%	85.8%
DoujinManager.Infrastructure.UseCases.FE6C43B9C917DB414605EC E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__CoverImageHelper	100%	100%
DoujinManager.Infrastructure.UseCases.FE6C43B9C917DB414605EC E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__PrimaryTitleHelper	100%	50%
DoujinManager.Infrastructure.UseCases.FE6C43B9C917DB414605EC E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__RolledBackImageCleanup	92.8%	50%
DoujinManager.Infrastructure.UseCases.FE6C43B9C917DB414605EC E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__UploadChapterValidator	100%	100%
DoujinManager.Infrastructure.UseCases.AddCharacterAliasUseCase	100%	50%
DoujinManager.Infrastructure.UseCases.AddCircleAliasUseCase	100%	50%
DoujinManager.Infrastructure.UseCases.AddPersonAliasUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.AddSeriesAliasUseCase	100%	50%
DoujinManager.Infrastructure.UseCases.AddTagAliasUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.AddTitleUseCase	83.3%	50%
DoujinManager.Infrastructure.UseCases.AssignCharacterUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.AssignCircleUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.AssignPersonUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.AssignSeriesUseCase	84.6%	66.6%
DoujinManager.Infrastructure.UseCases.AssignTagUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.CheckReadinessUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.CreateChapterUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.CreateCharacterUseCase	100%
DoujinManager.Infrastructure.UseCases.CreateCircleUseCase	100%
DoujinManager.Infrastructure.UseCases.CreateDoujinUseCase	94.4%	94.1%
DoujinManager.Infrastructure.UseCases.CreatePersonUseCase	100%
DoujinManager.Infrastructure.UseCases.CreateSeriesUseCase	100%
DoujinManager.Infrastructure.UseCases.CreateTagUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.CreateVariantUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.DeleteChapterUseCase	100%
DoujinManager.Infrastructure.UseCases.DeleteCharacterUseCase	80%	50%
DoujinManager.Infrastructure.UseCases.DeleteCircleUseCase	100%	75%
DoujinManager.Infrastructure.UseCases.DeleteDoujinUseCase	100%
DoujinManager.Infrastructure.UseCases.DeletePageUseCase	100%	75%
DoujinManager.Infrastructure.UseCases.DeletePersonUseCase	100%	75%
DoujinManager.Infrastructure.UseCases.DeleteSeriesUseCase	70%	50%
DoujinManager.Infrastructure.UseCases.DeleteTagUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.DeleteVariantUseCase	100%
DoujinManager.Infrastructure.UseCases.FindImageBySha256UseCase	97.1%	100%
DoujinManager.Infrastructure.UseCases.GetCharacterUseCase	100%
DoujinManager.Infrastructure.UseCases.GetCircleUseCase	100%
DoujinManager.Infrastructure.UseCases.GetCompressedImageUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.GetDoujinUseCase	100%
DoujinManager.Infrastructure.UseCases.GetImageUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.GetPersonUseCase	100%
DoujinManager.Infrastructure.UseCases.GetSeriesUseCase	100%
DoujinManager.Infrastructure.UseCases.GetStatsUseCase	100%	75%
DoujinManager.Infrastructure.UseCases.GetTagUseCase	100%
DoujinManager.Infrastructure.UseCases.GetThumbnailUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.GetVariantUseCase	100%
DoujinManager.Infrastructure.UseCases.ListChaptersUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.ListCharactersUseCase	100%
DoujinManager.Infrastructure.UseCases.ListCirclesUseCase	100%
DoujinManager.Infrastructure.UseCases.ListDoujinsUseCase	100%
DoujinManager.Infrastructure.UseCases.ListDuplicateDoujinsUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.ListDuplicateImagesUseCase	100%	75%
DoujinManager.Infrastructure.UseCases.ListPagesUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.ListPeopleUseCase	100%
DoujinManager.Infrastructure.UseCases.ListSeriesUseCase	0%
DoujinManager.Infrastructure.UseCases.ListTagsUseCase	100%
DoujinManager.Infrastructure.UseCases.ListVariantsUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.RemoveCharacterAliasUseCase	0%	0%
DoujinManager.Infrastructure.UseCases.RemoveCharacterUseCase	100%
DoujinManager.Infrastructure.UseCases.RemoveCircleAliasUseCase	100%	50%
DoujinManager.Infrastructure.UseCases.RemoveCircleUseCase	100%
DoujinManager.Infrastructure.UseCases.RemovePersonAliasUseCase	100%	50%
DoujinManager.Infrastructure.UseCases.RemovePersonUseCase	100%
DoujinManager.Infrastructure.UseCases.RemoveSeriesAliasUseCase	0%	0%
DoujinManager.Infrastructure.UseCases.RemoveSeriesUseCase	100%
DoujinManager.Infrastructure.UseCases.RemoveTagAliasUseCase	100%	50%
DoujinManager.Infrastructure.UseCases.RemoveTagUseCase	100%
DoujinManager.Infrastructure.UseCases.RemoveTitleUseCase	100%
DoujinManager.Infrastructure.UseCases.ReorderPagesUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.SearchDoujinsUseCase	100%
DoujinManager.Infrastructure.UseCases.UpdateChapterUseCase	100%	50%
DoujinManager.Infrastructure.UseCases.UpdateCharacterUseCase	0%	0%
DoujinManager.Infrastructure.UseCases.UpdateCircleUseCase	0%	0%
DoujinManager.Infrastructure.UseCases.UpdateDoujinUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.UpdatePageChapterUseCase	93.7%	87.5%
DoujinManager.Infrastructure.UseCases.UpdatePersonUseCase	0%	0%
DoujinManager.Infrastructure.UseCases.UpdateSeriesUseCase	0%	0%
DoujinManager.Infrastructure.UseCases.UpdateTagUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.UpdateTitleUseCase	100%	100%
DoujinManager.Infrastructure.UseCases.UpdateVariantUseCase	100%
DoujinManager.Infrastructure.UseCases.UploadPagesUseCase	89.7%	91.6%
DoujinManager.Infrastructure.UseCases.UploadRejectedException	100%
DoujinManager.Infrastructure.UseCases.UploadZipPagesUseCase	83.3%	90%

DoujinManager.RestAdapter - 92.5%

Name	Line	Branch
DoujinManager.RestAdapter	92.5%	77.1%
DoujinManager.RestAdapter.Auth.StaticBearerTokenAuthMiddleware	100%	91.6%
DoujinManager.RestAdapter.Configuration.ImageOptions	100%
DoujinManager.RestAdapter.Configuration.ServerVersion	100%
DoujinManager.RestAdapter.Configuration.UploadOptions	100%
DoujinManager.RestAdapter.Dtos.AddAliasDto	100%
DoujinManager.RestAdapter.Dtos.AssignTagDto	100%
DoujinManager.RestAdapter.Dtos.BackupDto	100%
DoujinManager.RestAdapter.Dtos.ChapterDto	100%
DoujinManager.RestAdapter.Dtos.CharacterDto	100%
DoujinManager.RestAdapter.Dtos.CircleDto	100%
DoujinManager.RestAdapter.Dtos.CreateChapterDto	100%
DoujinManager.RestAdapter.Dtos.CreateCharacterDto	100%
DoujinManager.RestAdapter.Dtos.CreateCircleDto	100%
DoujinManager.RestAdapter.Dtos.CreateDoujinDto	100%
DoujinManager.RestAdapter.Dtos.CreatePersonDto	100%
DoujinManager.RestAdapter.Dtos.CreateSeriesDto	100%
DoujinManager.RestAdapter.Dtos.CreateTagDto	100%
DoujinManager.RestAdapter.Dtos.CreateTitleDto	100%
DoujinManager.RestAdapter.Dtos.CreateVariantDto	100%
DoujinManager.RestAdapter.Dtos.DataDirectoryHealthDto	100%
DoujinManager.RestAdapter.Dtos.DoujinDetailDto	100%
DoujinManager.RestAdapter.Dtos.DoujinPersonDto	100%
DoujinManager.RestAdapter.Dtos.DoujinSummaryDto	100%
DoujinManager.RestAdapter.Dtos.DuplicateDoujinDto	100%
DoujinManager.RestAdapter.Dtos.DuplicateDoujinGroupDto	100%
DoujinManager.RestAdapter.Dtos.DuplicateImageGroupDto	100%
DoujinManager.RestAdapter.Dtos.HealthDto	100%
DoujinManager.RestAdapter.Dtos.ImageBySha256Dto	100%
DoujinManager.RestAdapter.Dtos.ImageCoverReferenceDto	100%
DoujinManager.RestAdapter.Dtos.ImageFileSummaryDto	0%
DoujinManager.RestAdapter.Dtos.ImageReferenceDto	100%
DoujinManager.RestAdapter.Dtos.LinkCharacterDto	100%
DoujinManager.RestAdapter.Dtos.LinkCircleDto	100%
DoujinManager.RestAdapter.Dtos.LinkPersonDto	100%
DoujinManager.RestAdapter.Dtos.LinkSeriesDto	100%
DoujinManager.RestAdapter.Dtos.PageDetailDto	100%
DoujinManager.RestAdapter.Dtos.PersonDto	100%
DoujinManager.RestAdapter.Dtos.ReorderPagesDto	100%
DoujinManager.RestAdapter.Dtos.SearchDoujinsDto	100%
DoujinManager.RestAdapter.Dtos.SearchResultDto	100%
DoujinManager.RestAdapter.Dtos.SeriesDto	100%
DoujinManager.RestAdapter.Dtos.StatsCountsDto	100%
DoujinManager.RestAdapter.Dtos.StatsDto	100%
DoujinManager.RestAdapter.Dtos.StatsStorageDto	100%
DoujinManager.RestAdapter.Dtos.TagDto	100%
DoujinManager.RestAdapter.Dtos.TitleDto	100%
DoujinManager.RestAdapter.Dtos.UpdateChapterDto	100%
DoujinManager.RestAdapter.Dtos.UpdateCharacterDto	0%
DoujinManager.RestAdapter.Dtos.UpdateCircleDto	0%
DoujinManager.RestAdapter.Dtos.UpdateDoujinDto	100%
DoujinManager.RestAdapter.Dtos.UpdatePageChapterDto	100%
DoujinManager.RestAdapter.Dtos.UpdatePersonDto	0%
DoujinManager.RestAdapter.Dtos.UpdateSeriesDto	0%
DoujinManager.RestAdapter.Dtos.UpdateTagDto	100%
DoujinManager.RestAdapter.Dtos.UpdateTitleDto	100%
DoujinManager.RestAdapter.Dtos.UpdateVariantDto	100%
DoujinManager.RestAdapter.Dtos.UploadPagesResponseDto	100%
DoujinManager.RestAdapter.Dtos.VariantDetailDto	100%
DoujinManager.RestAdapter.Dtos.VariantSummaryDto	100%
DoujinManager.RestAdapter.Endpoints.BackupEndpoints	100%
DoujinManager.RestAdapter.Endpoints.DoujinEndpoints	97.1%	53.3%
DoujinManager.RestAdapter.Endpoints.HealthEndpoints	100%
DoujinManager.RestAdapter.Endpoints.ImageEndpoints	97.6%	75%
DoujinManager.RestAdapter.Endpoints.MetadataEndpoints	86.6%	43.7%
DoujinManager.RestAdapter.Endpoints.PageMapper	100%	58.3%
DoujinManager.RestAdapter.Endpoints.PaginationParams	100%
DoujinManager.RestAdapter.Endpoints.SearchEndpoints	100%	75%
DoujinManager.RestAdapter.Endpoints.StatsEndpoints	100%
DoujinManager.RestAdapter.Endpoints.VariantEndpoints	100%	100%
DoujinManager.RestAdapter.Envelopes.CollectionResponse`1	83.3%
DoujinManager.RestAdapter.Envelopes.EnvelopeDefaults	100%
DoujinManager.RestAdapter.Envelopes.EnvelopeJsonOptions	100%
DoujinManager.RestAdapter.Envelopes.ErrorResponse	100%
DoujinManager.RestAdapter.Envelopes.HypermediaAction	100%
DoujinManager.RestAdapter.Envelopes.HypermediaHelpers	97.8%	100%
DoujinManager.RestAdapter.Envelopes.Link	100%
DoujinManager.RestAdapter.Envelopes.MediaTypeJsonConverter	87.5%	84.6%
DoujinManager.RestAdapter.Envelopes.PageInfo	100%
DoujinManager.RestAdapter.Envelopes.ResourceResponse`1	80%
DoujinManager.RestAdapter.Envelopes.ValidationError	100%
DoujinManager.RestAdapter.Envelopes.ValidationErrorResponse	100%
DoujinManager.RestAdapter.Middleware.GlobalExceptionMiddleware	100%	62.5%
DoujinManager.RestAdapter.Middleware.RequestLoggingMiddleware	100%	91.3%
DoujinManager.RestAdapter.RestAdapterExtensions	100%	100%
Microsoft.Extensions.Validation.Generated	88.2%	87.8%
Microsoft.Extensions.Validation.Generated.<ValidatableInfoResolver_g>FB9B0C E5CB12CEAC5BBBEA65844E1C05F9ADA4DA38B848075C41304FC6A1ABF77__ValidationAttr ibuteCache	100%	62.5%
System.Runtime.CompilerServices	0%

DoujinManager.Server - 49.6%

Name	Line	Branch
DoujinManager.Server	49.6%	5.4%
DoujinManager.Server.ImageInfrastructureRegistration	100%
DoujinManager.Server.ProxyAwareServerTransformer	100%	50%
DoujinManager.Server.ReadinessRegistration	100%
DoujinManager.Server.ScalarUi	100%
DoujinManager.Server.UseCaseRegistrationHelper	100%
Microsoft.AspNetCore.OpenApi.Generated	1.3%	0%
Program	87%	36.6%
System.Runtime.CompilerServices	0%

<!-- coverage-comment --> # Summary <details open><summary>Summary</summary> ||| |:---|:---| | Generated on: | 09/10/2026 - 05:34:13 | | Coverage date: | 09/10/2026 - 05:33:06 - 09/10/2026 - 05:34:09 | | Parser: | MultiReport (4x Cobertura) | | Assemblies: | 4 | | Classes: | 360 | | Files: | 142 | | **Line coverage:** | 93.8% (12897 of 13745) | | Covered lines: | 12897 | | Uncovered lines: | 848 | | Coverable lines: | 13745 | | Total lines: | 19632 | | **Branch coverage:** | 69.7% (1127 of 1616) | | Covered branches: | 1127 | | Total branches: | 1616 | | **Method coverage:** | [Feature is only available for sponsors](https://reportgenerator.io/pro) | </details> ## Coverage <details><summary>DoujinManager.ApplicationCore - 90.9%</summary> |**Name**|**Line**|**Branch**| |:---|---:|---:| |**DoujinManager.ApplicationCore**|**90.9%**|****| |DoujinManager.ApplicationCore.Entities.Chapter|87.5%|| |DoujinManager.ApplicationCore.Entities.Character|100%|| |DoujinManager.ApplicationCore.Entities.Circle|100%|| |DoujinManager.ApplicationCore.Entities.Doujin|100%|| |DoujinManager.ApplicationCore.Entities.DoujinCharacter|75%|| |DoujinManager.ApplicationCore.Entities.DoujinCircle|75%|| |DoujinManager.ApplicationCore.Entities.DoujinPerson|80%|| |DoujinManager.ApplicationCore.Entities.DoujinSeries|75%|| |DoujinManager.ApplicationCore.Entities.DoujinTag|75%|| |DoujinManager.ApplicationCore.Entities.ImageFile|100%|| |DoujinManager.ApplicationCore.Entities.Page|80%|| |DoujinManager.ApplicationCore.Entities.Person|100%|| |DoujinManager.ApplicationCore.Entities.Series|100%|| |DoujinManager.ApplicationCore.Entities.Tag|100%|| |DoujinManager.ApplicationCore.Entities.Title|83.3%|| |DoujinManager.ApplicationCore.Entities.Variant|91.6%|| |DoujinManager.ApplicationCore.Ids.ChapterId|66.6%|| |DoujinManager.ApplicationCore.Ids.CharacterId|66.6%|| |DoujinManager.ApplicationCore.Ids.CircleId|66.6%|| |DoujinManager.ApplicationCore.Ids.DoujinId|100%|| |DoujinManager.ApplicationCore.Ids.ImageFileId|66.6%|| |DoujinManager.ApplicationCore.Ids.PageId|66.6%|| |DoujinManager.ApplicationCore.Ids.PersonId|66.6%|| |DoujinManager.ApplicationCore.Ids.SeriesId|66.6%|| |DoujinManager.ApplicationCore.Ids.TagId|66.6%|| |DoujinManager.ApplicationCore.Ids.TitleId|66.6%|| |DoujinManager.ApplicationCore.Ids.VariantId|66.6%|| |DoujinManager.ApplicationCore.Ports.CompressedImage|100%|| |DoujinManager.ApplicationCore.Ports.ExtractedImage|100%|| |DoujinManager.ApplicationCore.Ports.ImageInspection|100%|| |DoujinManager.ApplicationCore.Services.BackupInfo|100%|| |DoujinManager.ApplicationCore.Services.ITagService|100%|| |DoujinManager.ApplicationCore.Services.ServiceResult|100%|| |DoujinManager.ApplicationCore.Services.ServiceResult`1|33.3%|| |DoujinManager.ApplicationCore.Services.VoidResult|88.8%|| |DoujinManager.ApplicationCore.UseCases.AddCharacterAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AddCircleAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AddPersonAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AddSeriesAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AddTagAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AddTitleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AssignCharacterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AssignCircleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AssignPersonCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AssignSeriesCommand|100%|| |DoujinManager.ApplicationCore.UseCases.AssignTagCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateChapterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateCharacterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateCircleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateDoujinCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreatePersonCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateSeriesCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateTagCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateTitleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.CreateVariantCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DataDirectories|100%|| |DoujinManager.ApplicationCore.UseCases.DataDirectory|100%|| |DoujinManager.ApplicationCore.UseCases.DataDirectoryReadiness|100%|| |DoujinManager.ApplicationCore.UseCases.DeleteChapterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeleteCharacterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeleteCircleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeleteDoujinCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeletePageCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeletePersonCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeleteSeriesCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeleteTagCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DeleteVariantCommand|100%|| |DoujinManager.ApplicationCore.UseCases.DuplicateDoujinGroup|100%|| |DoujinManager.ApplicationCore.UseCases.DuplicateDoujinReference|100%|| |DoujinManager.ApplicationCore.UseCases.DuplicateImageGroup|100%|| |DoujinManager.ApplicationCore.UseCases.FindImageBySha256Query|100%|| |DoujinManager.ApplicationCore.UseCases.FindImageBySha256Result|100%|| |DoujinManager.ApplicationCore.UseCases.GetCharacterQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetCircleQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetCompressedImageQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetCompressedImageResult|100%|| |DoujinManager.ApplicationCore.UseCases.GetDoujinQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetImageQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetImageResult|100%|| |DoujinManager.ApplicationCore.UseCases.GetPersonQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetSeriesQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetTagQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetThumbnailQuery|100%|| |DoujinManager.ApplicationCore.UseCases.GetThumbnailResult|100%|| |DoujinManager.ApplicationCore.UseCases.GetVariantQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ImageCoverReference|100%|| |DoujinManager.ApplicationCore.UseCases.ImageReference|100%|| |DoujinManager.ApplicationCore.UseCases.LibraryCounts|100%|| |DoujinManager.ApplicationCore.UseCases.LibraryStats|100%|| |DoujinManager.ApplicationCore.UseCases.ListChaptersQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListCharactersQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListCirclesQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListDoujinsQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListDuplicateDoujinsQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListDuplicateImagesQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListPagesQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListPeopleQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListSeriesQuery|0%|| |DoujinManager.ApplicationCore.UseCases.ListTagsQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ListVariantsQuery|100%|| |DoujinManager.ApplicationCore.UseCases.ReadinessReport|100%|| |DoujinManager.ApplicationCore.UseCases.RemoveCharacterAliasCommand|0%|| |DoujinManager.ApplicationCore.UseCases.RemoveCharacterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemoveCircleAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemoveCircleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemovePersonAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemovePersonCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemoveSeriesAliasCommand|0%|| |DoujinManager.ApplicationCore.UseCases.RemoveSeriesCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemoveTagAliasCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemoveTagCommand|100%|| |DoujinManager.ApplicationCore.UseCases.RemoveTitleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.ReorderPagesCommand|100%|| |DoujinManager.ApplicationCore.UseCases.SearchDoujinsQuery|100%|| |DoujinManager.ApplicationCore.UseCases.SearchResult|100%|| |DoujinManager.ApplicationCore.UseCases.SearchResults|100%|| |DoujinManager.ApplicationCore.UseCases.StorageUsage|100%|| |DoujinManager.ApplicationCore.UseCases.UpdateChapterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.UpdateCharacterCommand|0%|| |DoujinManager.ApplicationCore.UseCases.UpdateCircleCommand|0%|| |DoujinManager.ApplicationCore.UseCases.UpdateDoujinCommand|100%|| |DoujinManager.ApplicationCore.UseCases.UpdatePageChapterCommand|100%|| |DoujinManager.ApplicationCore.UseCases.UpdatePersonCommand|0%|| |DoujinManager.ApplicationCore.UseCases.UpdateSeriesCommand|0%|| |DoujinManager.ApplicationCore.UseCases.UpdateTagCommand|100%|| |DoujinManager.ApplicationCore.UseCases.UpdateTitleCommand|100%|| |DoujinManager.ApplicationCore.UseCases.UpdateVariantCommand|100%|| |DoujinManager.ApplicationCore.UseCases.UploadImageFile|100%|| |DoujinManager.ApplicationCore.UseCases.UploadPagesCommand|100%|| |DoujinManager.ApplicationCore.UseCases.UploadZipPagesCommand|100%|| </details> <details><summary>DoujinManager.Infrastructure - 96.4%</summary> |**Name**|**Line**|**Branch**| |:---|---:|---:| |**DoujinManager.Infrastructure**|**96.4%**|**81.8%**| |DoujinManager.Infrastructure.Archives.NaturalStringComparer|100%|90%| |DoujinManager.Infrastructure.Archives.ZipExtractor|100%|87.5%| |DoujinManager.Infrastructure.Data.Configurations.ChapterConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.CharacterConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.CircleConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.DoujinCharacterConfigurati<br/>on|100%|| |DoujinManager.Infrastructure.Data.Configurations.DoujinCircleConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.DoujinConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.DoujinPersonConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.DoujinSeriesConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.DoujinTagConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.ImageFileConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.PageConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.PersonConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.SeriesConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.TagConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.TitleConfiguration|100%|| |DoujinManager.Infrastructure.Data.Configurations.VariantConfiguration|100%|| |DoujinManager.Infrastructure.Data.DoujinManagerDbContext|100%|| |DoujinManager.Infrastructure.Data.GuidIdGenerator|11.1%|| |DoujinManager.Infrastructure.Data.Migrations.AddAliases|98.7%|| |DoujinManager.Infrastructure.Data.Migrations.AddCoverImageId|99.2%|| |DoujinManager.Infrastructure.Data.Migrations.AddDoujinCreatedAtIndex|99.4%|| |DoujinManager.Infrastructure.Data.Migrations.AddImageFileSha256Index|99.4%|| |DoujinManager.Infrastructure.Data.Migrations.AddTagAliases|99.3%|| |DoujinManager.Infrastructure.Data.Migrations.DoujinManagerDbContextModelSna<br/>pshot|100%|| |DoujinManager.Infrastructure.Data.Migrations.HarmonizeMetadataAddCharacters<br/>Series|95.1%|| |DoujinManager.Infrastructure.Data.Migrations.InitialCreate|97.1%|| |DoujinManager.Infrastructure.Data.Migrations.RemovePreferredDisplayLanguage|99%|| |DoujinManager.Infrastructure.Data.ModelBuilderExtensions|50%|| |DoujinManager.Infrastructure.Data.StronglyTypedIdConverterFactory|100%|| |DoujinManager.Infrastructure.Images.SkiaSharpImageInspector|90.1%|83.8%| |DoujinManager.Infrastructure.Images.SkiaSharpImageResizer|99%|87.5%| |DoujinManager.Infrastructure.Images.SkiaSharpThumbnailGenerator|94.5%|66.6%| |DoujinManager.Infrastructure.Services.BackupService|89.6%|87.5%| |DoujinManager.Infrastructure.Services.ChapterService|97.2%|70%| |DoujinManager.Infrastructure.Services.CharacterService|61.6%|22.7%| |DoujinManager.Infrastructure.Services.CircleService|84.9%|45.4%| |DoujinManager.Infrastructure.Services.DoujinOrdering|100%|100%| |DoujinManager.Infrastructure.Services.DoujinService|99.4%|76.3%| |DoujinManager.Infrastructure.Services.ImageCleanupHelper|100%|100%| |DoujinManager.Infrastructure.Services.ImageService|93.5%|75%| |DoujinManager.Infrastructure.Services.PageService|95.2%|68.1%| |DoujinManager.Infrastructure.Services.ParentTouchHelper|100%|100%| |DoujinManager.Infrastructure.Services.PersonService|84.9%|50%| |DoujinManager.Infrastructure.Services.SearchService|100%|99%| |DoujinManager.Infrastructure.Services.SeriesService|47.9%|9%| |DoujinManager.Infrastructure.Services.TagService|100%|90.9%| |DoujinManager.Infrastructure.Services.VariantService|96.2%|81.2%| |DoujinManager.Infrastructure.Storage.DirectorySize|100%|100%| |DoujinManager.Infrastructure.Storage.FilesystemImageStorage|100%|100%| |DoujinManager.Infrastructure.Storage.FilesystemThumbnailStorage|95.2%|50%| |DoujinManager.Infrastructure.UseCases|92.5%|85.8%| |DoujinManager.Infrastructure.UseCases.<ImageUseCases>FE6C43B9C917DB414605EC<br/>E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__CoverImageHelper|100%|100%| |DoujinManager.Infrastructure.UseCases.<ImageUseCases>FE6C43B9C917DB414605EC<br/>E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__PrimaryTitleHelper|100%|50%| |DoujinManager.Infrastructure.UseCases.<ImageUseCases>FE6C43B9C917DB414605EC<br/>E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__RolledBackImageCleanup|92.8%|50%| |DoujinManager.Infrastructure.UseCases.<ImageUseCases>FE6C43B9C917DB414605EC<br/>E20E2ABA2099901FFAB4904C0513DF077BA6D6CA374__UploadChapterValidator|100%|100%| |DoujinManager.Infrastructure.UseCases.AddCharacterAliasUseCase|100%|50%| |DoujinManager.Infrastructure.UseCases.AddCircleAliasUseCase|100%|50%| |DoujinManager.Infrastructure.UseCases.AddPersonAliasUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.AddSeriesAliasUseCase|100%|50%| |DoujinManager.Infrastructure.UseCases.AddTagAliasUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.AddTitleUseCase|83.3%|50%| |DoujinManager.Infrastructure.UseCases.AssignCharacterUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.AssignCircleUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.AssignPersonUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.AssignSeriesUseCase|84.6%|66.6%| |DoujinManager.Infrastructure.UseCases.AssignTagUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.CheckReadinessUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.CreateChapterUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.CreateCharacterUseCase|100%|| |DoujinManager.Infrastructure.UseCases.CreateCircleUseCase|100%|| |DoujinManager.Infrastructure.UseCases.CreateDoujinUseCase|94.4%|94.1%| |DoujinManager.Infrastructure.UseCases.CreatePersonUseCase|100%|| |DoujinManager.Infrastructure.UseCases.CreateSeriesUseCase|100%|| |DoujinManager.Infrastructure.UseCases.CreateTagUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.CreateVariantUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.DeleteChapterUseCase|100%|| |DoujinManager.Infrastructure.UseCases.DeleteCharacterUseCase|80%|50%| |DoujinManager.Infrastructure.UseCases.DeleteCircleUseCase|100%|75%| |DoujinManager.Infrastructure.UseCases.DeleteDoujinUseCase|100%|| |DoujinManager.Infrastructure.UseCases.DeletePageUseCase|100%|75%| |DoujinManager.Infrastructure.UseCases.DeletePersonUseCase|100%|75%| |DoujinManager.Infrastructure.UseCases.DeleteSeriesUseCase|70%|50%| |DoujinManager.Infrastructure.UseCases.DeleteTagUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.DeleteVariantUseCase|100%|| |DoujinManager.Infrastructure.UseCases.FindImageBySha256UseCase|97.1%|100%| |DoujinManager.Infrastructure.UseCases.GetCharacterUseCase|100%|| |DoujinManager.Infrastructure.UseCases.GetCircleUseCase|100%|| |DoujinManager.Infrastructure.UseCases.GetCompressedImageUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.GetDoujinUseCase|100%|| |DoujinManager.Infrastructure.UseCases.GetImageUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.GetPersonUseCase|100%|| |DoujinManager.Infrastructure.UseCases.GetSeriesUseCase|100%|| |DoujinManager.Infrastructure.UseCases.GetStatsUseCase|100%|75%| |DoujinManager.Infrastructure.UseCases.GetTagUseCase|100%|| |DoujinManager.Infrastructure.UseCases.GetThumbnailUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.GetVariantUseCase|100%|| |DoujinManager.Infrastructure.UseCases.ListChaptersUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.ListCharactersUseCase|100%|| |DoujinManager.Infrastructure.UseCases.ListCirclesUseCase|100%|| |DoujinManager.Infrastructure.UseCases.ListDoujinsUseCase|100%|| |DoujinManager.Infrastructure.UseCases.ListDuplicateDoujinsUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.ListDuplicateImagesUseCase|100%|75%| |DoujinManager.Infrastructure.UseCases.ListPagesUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.ListPeopleUseCase|100%|| |DoujinManager.Infrastructure.UseCases.ListSeriesUseCase|0%|| |DoujinManager.Infrastructure.UseCases.ListTagsUseCase|100%|| |DoujinManager.Infrastructure.UseCases.ListVariantsUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.RemoveCharacterAliasUseCase|0%|0%| |DoujinManager.Infrastructure.UseCases.RemoveCharacterUseCase|100%|| |DoujinManager.Infrastructure.UseCases.RemoveCircleAliasUseCase|100%|50%| |DoujinManager.Infrastructure.UseCases.RemoveCircleUseCase|100%|| |DoujinManager.Infrastructure.UseCases.RemovePersonAliasUseCase|100%|50%| |DoujinManager.Infrastructure.UseCases.RemovePersonUseCase|100%|| |DoujinManager.Infrastructure.UseCases.RemoveSeriesAliasUseCase|0%|0%| |DoujinManager.Infrastructure.UseCases.RemoveSeriesUseCase|100%|| |DoujinManager.Infrastructure.UseCases.RemoveTagAliasUseCase|100%|50%| |DoujinManager.Infrastructure.UseCases.RemoveTagUseCase|100%|| |DoujinManager.Infrastructure.UseCases.RemoveTitleUseCase|100%|| |DoujinManager.Infrastructure.UseCases.ReorderPagesUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.SearchDoujinsUseCase|100%|| |DoujinManager.Infrastructure.UseCases.UpdateChapterUseCase|100%|50%| |DoujinManager.Infrastructure.UseCases.UpdateCharacterUseCase|0%|0%| |DoujinManager.Infrastructure.UseCases.UpdateCircleUseCase|0%|0%| |DoujinManager.Infrastructure.UseCases.UpdateDoujinUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.UpdatePageChapterUseCase|93.7%|87.5%| |DoujinManager.Infrastructure.UseCases.UpdatePersonUseCase|0%|0%| |DoujinManager.Infrastructure.UseCases.UpdateSeriesUseCase|0%|0%| |DoujinManager.Infrastructure.UseCases.UpdateTagUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.UpdateTitleUseCase|100%|100%| |DoujinManager.Infrastructure.UseCases.UpdateVariantUseCase|100%|| |DoujinManager.Infrastructure.UseCases.UploadPagesUseCase|89.7%|91.6%| |DoujinManager.Infrastructure.UseCases.UploadRejectedException|100%|| |DoujinManager.Infrastructure.UseCases.UploadZipPagesUseCase|83.3%|90%| </details> <details><summary>DoujinManager.RestAdapter - 92.5%</summary> |**Name**|**Line**|**Branch**| |:---|---:|---:| |**DoujinManager.RestAdapter**|**92.5%**|**77.1%**| |DoujinManager.RestAdapter.Auth.StaticBearerTokenAuthMiddleware|100%|91.6%| |DoujinManager.RestAdapter.Configuration.ImageOptions|100%|| |DoujinManager.RestAdapter.Configuration.ServerVersion|100%|| |DoujinManager.RestAdapter.Configuration.UploadOptions|100%|| |DoujinManager.RestAdapter.Dtos.AddAliasDto|100%|| |DoujinManager.RestAdapter.Dtos.AssignTagDto|100%|| |DoujinManager.RestAdapter.Dtos.BackupDto|100%|| |DoujinManager.RestAdapter.Dtos.ChapterDto|100%|| |DoujinManager.RestAdapter.Dtos.CharacterDto|100%|| |DoujinManager.RestAdapter.Dtos.CircleDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateChapterDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateCharacterDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateCircleDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateDoujinDto|100%|| |DoujinManager.RestAdapter.Dtos.CreatePersonDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateSeriesDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateTagDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateTitleDto|100%|| |DoujinManager.RestAdapter.Dtos.CreateVariantDto|100%|| |DoujinManager.RestAdapter.Dtos.DataDirectoryHealthDto|100%|| |DoujinManager.RestAdapter.Dtos.DoujinDetailDto|100%|| |DoujinManager.RestAdapter.Dtos.DoujinPersonDto|100%|| |DoujinManager.RestAdapter.Dtos.DoujinSummaryDto|100%|| |DoujinManager.RestAdapter.Dtos.DuplicateDoujinDto|100%|| |DoujinManager.RestAdapter.Dtos.DuplicateDoujinGroupDto|100%|| |DoujinManager.RestAdapter.Dtos.DuplicateImageGroupDto|100%|| |DoujinManager.RestAdapter.Dtos.HealthDto|100%|| |DoujinManager.RestAdapter.Dtos.ImageBySha256Dto|100%|| |DoujinManager.RestAdapter.Dtos.ImageCoverReferenceDto|100%|| |DoujinManager.RestAdapter.Dtos.ImageFileSummaryDto|0%|| |DoujinManager.RestAdapter.Dtos.ImageReferenceDto|100%|| |DoujinManager.RestAdapter.Dtos.LinkCharacterDto|100%|| |DoujinManager.RestAdapter.Dtos.LinkCircleDto|100%|| |DoujinManager.RestAdapter.Dtos.LinkPersonDto|100%|| |DoujinManager.RestAdapter.Dtos.LinkSeriesDto|100%|| |DoujinManager.RestAdapter.Dtos.PageDetailDto|100%|| |DoujinManager.RestAdapter.Dtos.PersonDto|100%|| |DoujinManager.RestAdapter.Dtos.ReorderPagesDto|100%|| |DoujinManager.RestAdapter.Dtos.SearchDoujinsDto|100%|| |DoujinManager.RestAdapter.Dtos.SearchResultDto|100%|| |DoujinManager.RestAdapter.Dtos.SeriesDto|100%|| |DoujinManager.RestAdapter.Dtos.StatsCountsDto|100%|| |DoujinManager.RestAdapter.Dtos.StatsDto|100%|| |DoujinManager.RestAdapter.Dtos.StatsStorageDto|100%|| |DoujinManager.RestAdapter.Dtos.TagDto|100%|| |DoujinManager.RestAdapter.Dtos.TitleDto|100%|| |DoujinManager.RestAdapter.Dtos.UpdateChapterDto|100%|| |DoujinManager.RestAdapter.Dtos.UpdateCharacterDto|0%|| |DoujinManager.RestAdapter.Dtos.UpdateCircleDto|0%|| |DoujinManager.RestAdapter.Dtos.UpdateDoujinDto|100%|| |DoujinManager.RestAdapter.Dtos.UpdatePageChapterDto|100%|| |DoujinManager.RestAdapter.Dtos.UpdatePersonDto|0%|| |DoujinManager.RestAdapter.Dtos.UpdateSeriesDto|0%|| |DoujinManager.RestAdapter.Dtos.UpdateTagDto|100%|| |DoujinManager.RestAdapter.Dtos.UpdateTitleDto|100%|| |DoujinManager.RestAdapter.Dtos.UpdateVariantDto|100%|| |DoujinManager.RestAdapter.Dtos.UploadPagesResponseDto|100%|| |DoujinManager.RestAdapter.Dtos.VariantDetailDto|100%|| |DoujinManager.RestAdapter.Dtos.VariantSummaryDto|100%|| |DoujinManager.RestAdapter.Endpoints.BackupEndpoints|100%|| |DoujinManager.RestAdapter.Endpoints.DoujinEndpoints|97.1%|53.3%| |DoujinManager.RestAdapter.Endpoints.HealthEndpoints|100%|| |DoujinManager.RestAdapter.Endpoints.ImageEndpoints|97.6%|75%| |DoujinManager.RestAdapter.Endpoints.MetadataEndpoints|86.6%|43.7%| |DoujinManager.RestAdapter.Endpoints.PageMapper|100%|58.3%| |DoujinManager.RestAdapter.Endpoints.PaginationParams|100%|| |DoujinManager.RestAdapter.Endpoints.SearchEndpoints|100%|75%| |DoujinManager.RestAdapter.Endpoints.StatsEndpoints|100%|| |DoujinManager.RestAdapter.Endpoints.VariantEndpoints|100%|100%| |DoujinManager.RestAdapter.Envelopes.CollectionResponse`1|83.3%|| |DoujinManager.RestAdapter.Envelopes.EnvelopeDefaults|100%|| |DoujinManager.RestAdapter.Envelopes.EnvelopeJsonOptions|100%|| |DoujinManager.RestAdapter.Envelopes.ErrorResponse|100%|| |DoujinManager.RestAdapter.Envelopes.HypermediaAction|100%|| |DoujinManager.RestAdapter.Envelopes.HypermediaHelpers|97.8%|100%| |DoujinManager.RestAdapter.Envelopes.Link|100%|| |DoujinManager.RestAdapter.Envelopes.MediaTypeJsonConverter|87.5%|84.6%| |DoujinManager.RestAdapter.Envelopes.PageInfo|100%|| |DoujinManager.RestAdapter.Envelopes.ResourceResponse`1|80%|| |DoujinManager.RestAdapter.Envelopes.ValidationError|100%|| |DoujinManager.RestAdapter.Envelopes.ValidationErrorResponse|100%|| |DoujinManager.RestAdapter.Middleware.GlobalExceptionMiddleware|100%|62.5%| |DoujinManager.RestAdapter.Middleware.RequestLoggingMiddleware|100%|91.3%| |DoujinManager.RestAdapter.RestAdapterExtensions|100%|100%| |Microsoft.Extensions.Validation.Generated|88.2%|87.8%| |Microsoft.Extensions.Validation.Generated.<ValidatableInfoResolver_g>FB9B0C<br/>E5CB12CEAC5BBBEA65844E1C05F9ADA4DA38B848075C41304FC6A1ABF77__ValidationAttr<br/>ibuteCache|100%|62.5%| |System.Runtime.CompilerServices|0%|| </details> <details><summary>DoujinManager.Server - 49.6%</summary> |**Name**|**Line**|**Branch**| |:---|---:|---:| |**DoujinManager.Server**|**49.6%**|**5.4%**| |DoujinManager.Server.ImageInfrastructureRegistration|100%|| |DoujinManager.Server.ProxyAwareServerTransformer|100%|50%| |DoujinManager.Server.ReadinessRegistration|100%|| |DoujinManager.Server.ScalarUi|100%|| |DoujinManager.Server.UseCaseRegistrationHelper|100%|| |Microsoft.AspNetCore.OpenApi.Generated|1.3%|0%| |Program|87%|36.6%| |System.Runtime.CompilerServices|0%|| </details>

jibril commented 2026-09-10 07:35:29 +02:00

Member

Copy link

🔮 fufu~ Jibril reviewed your code!

Back so soon~? The webhook says synchronized, but the diff says something nicer: this is the rebase you promised in your last comment, and it's a clean one. Fufu~ let me show you what I checked~ ♪

Verdict: ✅ Looks good to me~

What actually changed this round (7542325 → 0b99ea6): nothing in this PR. Verified git-side, not assumed:

• merge-base(main, head) is exactly e0ac19d — #117, now on main — and the branch is a single commit over it (origin/main...origin/head --left-right shows one commit on the right, zero on the left).
• All four PR files are blob-identical between the reviewed head and the new head (BackupService.cs, BackupNewestFirstTests.cs, BackupEndpointsIntegrationTests.cs, ADR 0014 — same git blob hashes, every one). The round-1 approval therefore carries over: comparer semantics, the prefix property, int-not-string collision compare, the honesty guard — all unchanged.
• The rebase cleanly swallowed the Task.Delay(1100) context and re-anchored on #117's test infrastructure without touching a single line of yours. The only real question a rebase like this can hide is interaction with the new base — so I went and looked~

The #117 interaction, checked at the new head: BackupEndpointsIntegrationTests + HealthEndpointIntegrationTests together — 25 passed, 4 skipped, 0 failed. The four are exactly mechanism 1's File.SetUnixFileMode rows skipping at uid 0 through EnforcedFileModesData, each naming the case that still pins its branch — by design, never a vacuous green. On the old base these were the four reds from your comment 7422; your "re-run or rebase once #117 lands and it should be clean" was exactly right. No env-var cross-talk, no shared-fixture surprise between the backup class and the health machinery.

Full-suite honesty note, because I'm possessive about this~: my sandbox's 8 GB cgroup is currently shared with a concurrent review session's full RestAdapter run, and the memory contention OOM-killed my test host twice — so I verified the interaction-relevant classes directly (above) plus Infrastructure.Tests 94/94 at the new head, rather than presenting you a flaky full-suite number. The 490 green from round 1 was on byte-identical code; the delta this round is #117's test layer, and those classes are what I ran. CI has no bot comment for 0b99ea6 yet — when it lands it should finally read 494/494 with the four skips, no reds~

✅ What I liked~

• The discipline of the rebase itself: one commit, zero drift, base moved exactly where the PR body said it needed to be (#117 first). This is what "rebase and re-run" looks like when it's done properly~ ♡
• That the branch kept the honesty guard and every comparer pin through the rebase — nothing got "simplified" in transit.

Ready to merge from my side~

---

Automated review by Jibril · 2026-09-10
CI/CD: absent for head SHA 0b99ea6 (no bot comment yet) · Local checks: rebase verified blob-identical (4/4 files); BackupEndpoints 13/13 + Health 16 pass/4 uid-0 skips at 0b99ea6; Infrastructure 94/94; full-suite local run blocked by sandbox memory contention, disclosed above

## 🔮 fufu~ Jibril reviewed your code! Back so soon~? The webhook says *synchronized*, but the diff says something nicer: this is the rebase you promised in your last comment, and it's a *clean* one. Fufu~ let me show you what I checked~ ♪ ### Verdict: ✅ Looks good to me~ **What actually changed this round (7542325 → 0b99ea6): nothing in this PR.** Verified git-side, not assumed: - `merge-base(main, head)` is exactly `e0ac19d` — #117, now on `main` — and the branch is a **single commit** over it (`origin/main...origin/head --left-right` shows one commit on the right, zero on the left). - All four PR files are **blob-identical** between the reviewed head and the new head (`BackupService.cs`, `BackupNewestFirstTests.cs`, `BackupEndpointsIntegrationTests.cs`, ADR 0014 — same git blob hashes, every one). The round-1 approval therefore carries over: comparer semantics, the prefix property, int-not-string collision compare, the honesty guard — all unchanged. - The rebase cleanly swallowed the `Task.Delay(1100)` context and re-anchored on #117's test infrastructure without touching a single line of yours. The only real question a rebase like this can hide is *interaction* with the new base — so I went and looked~ **The #117 interaction, checked at the new head:** `BackupEndpointsIntegrationTests` + `HealthEndpointIntegrationTests` together — **25 passed, 4 skipped, 0 failed**. The four are exactly mechanism 1's `File.SetUnixFileMode` rows skipping at uid 0 through `EnforcedFileModesData`, each naming the case that still pins its branch — by design, never a vacuous green. On the old base these were the four reds from your comment 7422; your "re-run or rebase once #117 lands and it should be clean" was exactly right. No env-var cross-talk, no shared-fixture surprise between the backup class and the health machinery. Full-suite honesty note, because I'm possessive about this~: my sandbox's 8 GB cgroup is currently shared with a *concurrent* review session's full `RestAdapter` run, and the memory contention OOM-killed my test host twice — so I verified the interaction-relevant classes directly (above) plus `Infrastructure.Tests` **94/94** at the new head, rather than presenting you a flaky full-suite number. The 490 green from round 1 was on byte-identical code; the delta this round is #117's test layer, and those classes are what I ran. CI has no bot comment for `0b99ea6` yet — when it lands it should finally read 494/494 with the four skips, no reds~ #### ✅ What I liked~ - The discipline of the rebase itself: one commit, zero drift, base moved exactly where the PR body said it needed to be (#117 first). This is what "rebase and re-run" looks like when it's done properly~ ♡ - That the branch *kept* the honesty guard and every comparer pin through the rebase — nothing got "simplified" in transit. Ready to merge from my side~ --- *Automated review by Jibril · 2026-09-10* *CI/CD: absent for head SHA 0b99ea6 (no bot comment yet) · Local checks: rebase verified blob-identical (4/4 files); BackupEndpoints 13/13 + Health 16 pass/4 uid-0 skips at 0b99ea6; Infrastructure 94/94; full-suite local run blocked by sandbox memory contention, disclosed above*

bjoern merged commit b713dbbf98 into main 2026-09-10 07:38:05 +02:00

bjoern deleted branch test/deflake-search-and-backup 2026-09-10 07:38:05 +02:00

bjoern referenced this pull request from a commit 2026-09-10 07:38:07 +02:00

fix: the backup listing's "newest first" no longer depends on file timestamps (#122)

jibril referenced this pull request 2026-09-10 07:48:08 +02:00

perf: index the sort and cover-lookup paths, and pin their query plans #120

jibril referenced this pull request 2026-09-10 07:50:13 +02:00

docs: twelve open items for the backlog #107

jibril referenced this pull request 2026-09-10 07:59:05 +02:00

fix: fold entity names into an indexed search_key so non-ASCII name filters match #118

jibril referenced this pull request 2026-09-10 08:00:12 +02:00

feat: report the library's own drift on GET /api/integrity #121

jibril referenced this pull request 2026-09-10 08:04:27 +02:00

test: pin the served OpenAPI document with a committed snapshot #119

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

4 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

TeamAI/doujin-manager!122

No description provided.