Skip to content

test: cover NotesService.update quiet-duplicate branch and note_detail consumer - #50

Merged
shukebeta merged 1 commit into
masterfrom
test/issue-49-notes-service-quiet-error-tests
Jul 6, 2026
Merged

shukebeta merged 1 commit into
masterfrom
test/issue-49-notes-service-quiet-error-tests

Conversation

@shukebeta

Copy link
Copy Markdown
Collaborator

Closes #49

…l consumer

Adds a FakeNotesDio transport so the real NotesService (not a mock) drives
through NotesApi.update against a fake Dio, exercising all three branches of
update() -- success, real API error, and quiet-duplicate -- plus the same
success/quiet-duplicate outcomes as seen by note_detail.dart's save flow.
Both suites are sensitive to the errorCode != / == AppConfig.quietErrorCode
conditions since they run the real source, not hand-rolled fake logic.

Closes #49
@shukebeta
shukebeta merged commit b4f1760 into master Jul 6, 2026
4 checks passed
@shukebeta
shukebeta deleted the test/issue-49-notes-service-quiet-error-tests branch July 6, 2026 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NotesService.update() quiet-duplicate branch and note_detail.dart consumer have zero test coverage

1 participant