Skip to content

Conversation

@ashwin-ant
Copy link
Collaborator

Modified the setup-claude-code-settings tests to use temporary directories instead of modifying the actual ~/.claude/settings.json file. This prevents test runs from affecting the user's real Claude Code settings.

Changes:

  • Added optional homeDir parameter to setupClaudeCodeSettings function
  • Updated all tests to use a temporary directory in tmpdir()
  • Tests now create isolated environments that are cleaned up after each test

🤖 Generated with Claude Code

Modified the setup-claude-code-settings tests to use temporary directories
instead of modifying the actual ~/.claude/settings.json file. This prevents
test runs from affecting the user's real Claude Code settings.

Changes:
- Added optional homeDir parameter to setupClaudeCodeSettings function
- Updated all tests to use a temporary directory in tmpdir()
- Tests now create isolated environments that are cleaned up after each test

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ashwin-ant ashwin-ant requested a review from a team July 16, 2025 23:49
@ashwin-ant ashwin-ant merged commit 7bbf7c2 into main Jul 17, 2025
13 checks passed
@ashwin-ant ashwin-ant deleted the ashwin/testfix branch July 17, 2025 00:23
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.

3 participants