Skip to content

test: find the SwagCommercial clone via MQ_COMMERCIAL_CLONE - #236

Merged
Sebastian Hölscher (HoelShare) merged 1 commit into
mainfrom
claude/commercial-clone-env
Oct 9, 2026
Merged

Sebastian Hölscher (HoelShare) merged 1 commit into
mainfrom
claude/commercial-clone-env

Conversation

@HoelShare

Copy link
Copy Markdown
Contributor

Summary

ReleaseCapabilityMatrixTest hardcoded CLONE_PATH as an absolute path under one developer's home directory. That leaked a personal path into this public repo and made the guard skip on every other machine.

It now follows the CoreFloorCompatibilityTest pattern:

  • reads MQ_COMMERCIAL_CLONE
  • falls back to $HOME/projects/swagcommercial
  • skips with No SwagCommercial clone at <path>; set MQ_COMMERCIAL_CLONE to run this guard.

docs/development.md now names the default location and the env var, matching the MQ_CORE_CLONE sentence beside it.

Test plan

  • Clone at the default location: 6 tests, 41 assertions pass
  • MQ_COMMERCIAL_CLONE=<clone>: passes
  • MQ_COMMERCIAL_CLONE=/nonexistent: all 6 skip with the message above
  • composer run quality: exit 0
  • composer run test: 1923 tests OK
  • git grep "/Users/": no other hits in tracked files

🤖 Generated with Claude Code

…rdcoded home path

ReleaseCapabilityMatrixTest pointed at one developer's absolute home
directory, which leaked a personal path into the public repository and
made the guard skip for everyone else. It now reads MQ_COMMERCIAL_CLONE
and falls back to $HOME/projects/swagcommercial, the same shape
CoreFloorCompatibilityTest uses for MQ_CORE_CLONE.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@HoelShare
Sebastian Hölscher (HoelShare) merged commit 73f8b4b into main Oct 9, 2026
1 check passed
@HoelShare
Sebastian Hölscher (HoelShare) deleted the claude/commercial-clone-env branch October 9, 2026 11:56
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