Skip to content

test: neutral machine key instead of a real name - #23

Merged
mbeczynski merged 1 commit into
mainfrom
audyt3/bez-imion
Oct 11, 2026
Merged

mbeczynski merged 1 commit into
mainfrom
audyt3/bez-imion

Conversation

@mbeczynski

Copy link
Copy Markdown
Contributor

Personal-data sweep of the public repo (2026-10-11).

  • The machine key `-mac-studio-3` / display name in MachineBudgetTests, DriveFolderTests and BackupVisibilityTests is plain test data: the real key comes at runtime from `MachineIdentity.storedKey` and the folder from `DriveFolder` (`mac-studio` legacy name is unchanged). Replaced with `home-mac-studio-3`.
  • The doc comment in `MachineBudget.swift` now uses a placeholder.
  • Not changed: the copyright line in `Info.plist` (`NSHumanReadableCopyright`) - author attribution, same as LICENSE.

No behaviour change, so no new test; the existing tests cover the same paths with the new value.

Checked locally: `swift build` OK, `swift test --skip BackupImageServiceTests`: 390 tests, 0 failures. swift-format lint is left to CI (not installed here).

Note: this repo has CD on merge (auto-patch release). A test-only change still publishes a release - consider whether that is wanted before merging.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Ww8BFyxe5Efb9pvhshhtig

The `<name>-mac-studio-3` machine key in the budget, folder and
visibility tests is plain test data (the real key is read at runtime
from MachineIdentity.storedKey), so it is replaced with
`home-mac-studio-3`. The doc comment in MachineBudget uses a
placeholder. No behaviour change. The copyright line in Info.plist is
the author attribution, like LICENSE, and stays.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Ww8BFyxe5Efb9pvhshhtig
@mbeczynski
mbeczynski merged commit 8a8f1f4 into main Oct 11, 2026
2 checks passed
@mbeczynski
mbeczynski deleted the audyt3/bez-imion branch October 11, 2026 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant