Skip to content
This repository was archived by the owner on Apr 26, 2026. It is now read-only.

Add unit and integration tests for 7 remaining test issues - #732

Merged
yaronkoren merged 1 commit into
mainfrom
add-unit-tests
Apr 8, 2026
Merged

Add unit and integration tests for 7 remaining test issues#732
yaronkoren merged 1 commit into
mainfrom
add-unit-tests

Conversation

@cicalese

@cicalese cicalese commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Unit tests (27 new test cases)

Integration tests (2 new tests)

Fixes #611, #613, #615, #617, #618, #619, #620

Test plan

  • go test ./... passes (unit tests)
  • go vet -tags integration ./tests/integration/ passes
  • gofmt clean
  • Integration tests pass in CI

- extractSecretKey migration: 6 tests (#611)
- fixVectorDefaultSkin migration: 4 tests (#613)
- getWikiIDsForRestore: 4 tests (#615)
- restoreUserDir: 4 tests (#617)
- WriteIDEConfigs: 4 tests (#618)

Fixes #611, #613, #615, #617, #618

@yaronkoren yaronkoren left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@yaronkoren
yaronkoren merged commit dd5f7b1 into main Apr 8, 2026
2 checks passed
@yaronkoren
yaronkoren deleted the add-unit-tests branch April 8, 2026 17:06
@cicalese cicalese changed the title Add unit tests for 5 remaining test issues Add unit and integration tests for 7 remaining test issues Apr 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test: add unit tests for extractSecretKey migration

2 participants