Confirmed — make test passes on fix/issue-3:
- Lint: ✅ 0 issues
- Vet: ✅ clean
- Tests: ✅ all pass
ok git.eeqj.de/sneak/secret/internal/cli 5.104s
ok git.eeqj.de/sneak/secret/…
Updated the test to remove the mock. Now uses realVault — a minimal VaultInterface implementation backed by a real afero.MemMapFs with the same directory layout that vault.CreateVault…
Re: using a real vault — getLongTermPrivateKey is unexported, so the test must be in package secret. But vault.Vault is in internal/vault which imports internal/secret, creating a…
Test Results for TestGetLongTermPrivateKeyUsesVaultDerivationIndex
The test creates a vault with DerivationIndex: 5 and verifies that getLongTermPrivateKey derives the key at index 5…
make test output (branch fix/issue-6)
This PR removes redundant code (duplicate longterm.age encryption) but does not add new test files. Existing tests all pass:
golangci-lint run…
Understood — will use make test (which runs lint → vet → go test) and make check (build + test) from the Makefile going forward, not ad-hoc commands. Will apply this to all repos.
Added TestGetLongTermPrivateKeyUsesVaultDerivationIndex — creates a mock vault with DerivationIndex=5, sets SECRET_MNEMONIC env var, and verifies the derived key matches index 5 (not…
Test Results
Tests with ONLY test changes (no fix applied): FAIL
=== RUN TestVaultOperations
=== RUN TestVaultOperations/CreateVault
=== RUN TestVaultOperations/ListVaults
===…
Rebased again onto latest next (now includes PR#17, #18, #20 merges). All tests pass. Conflict resolved.
The TODO.md content from this PR is already on next (was committed directly earlier). After rebasing, there are no remaining changes to merge. Closing as the content is already incorporated.
Rebased onto current main and resolved the conflict. The metalink "See Also" section is now placed before the Links/Authors/License sections that were added in the README restructure (#8).
Also…