Skip to content

Commit b87176f

Browse files
committed
fix: tag tests needing fresh sandbox — Windows can't auto-install toolchain in temp dirs
Tests 02, 24, 27, 32 create a fresh MCPP_HOME in a temp dir and expect mcpp to auto-install a toolchain. On Windows, the xlings LLVM install into a fresh sandbox fails (no pre-seeded xpkgs). Tag with `fresh-sandbox` capability (Linux/macOS only for now).
1 parent 23fa1db commit b87176f

5 files changed

Lines changed: 6 additions & 6 deletions

File tree

tests/e2e/02_new_build_run.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#!/usr/bin/env bash
2-
# requires:
2+
# requires: fresh-sandbox
33
# Single-module hello world: mcpp new → build → run
44
set -e
55

tests/e2e/24_git_dependency.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#!/usr/bin/env bash
2-
# requires:
2+
# requires: fresh-sandbox
33
# 24_git_dependency.sh — M4 #5: git-based dep clones to ~/.mcpp/git/<hash>/
44
# and is treated as a path dep.
55
set -e

tests/e2e/27_namespace_dependencies.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#!/usr/bin/env bash
2-
# requires:
2+
# requires: fresh-sandbox
33
# Namespaced dependencies: `[dependencies.<ns>] name = { path = "..." }`
44
# is parsed correctly and the dep is actually picked up by the build.
55
# Also verifies that the legacy `"<ns>.<name>" = "..."` quoted form still

tests/e2e/32_semver_merge.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
#!/usr/bin/env bash
2-
# requires:
2+
# requires: fresh-sandbox
33
# 32_semver_merge.sh — SemVer merge in the transitive walker:
44
# * Two consumers of the same package with overlapping constraints
55
# (one exact, one range) merge to a single satisfying version

tests/e2e/run_all.sh

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ OS="$(uname -s)"
4343

4444
case "$OS" in
4545
Linux)
46-
CAPS+=(elf unix-shell)
46+
CAPS+=(elf unix-shell fresh-sandbox)
4747
command -v g++ &>/dev/null && CAPS+=(gcc)
4848
command -v patchelf &>/dev/null && CAPS+=(patchelf)
4949
# musl-gcc: check both system PATH and xlings-managed locations
@@ -58,7 +58,7 @@ case "$OS" in
5858
fi
5959
;;
6060
Darwin)
61-
CAPS+=(unix-shell)
61+
CAPS+=(unix-shell fresh-sandbox)
6262
# macOS g++ is Apple Clang, not real GCC — don't add gcc capability.
6363
# Tests requiring gcc need actual GNU GCC (modules, gcm.cache, etc.)
6464
;;

0 commit comments

Comments
 (0)