Skip to content

Commit 9a5c9c2

Browse files
lpcoxCopilot
andcommitted
smoke-copilot: add artifact readability test to catch permission errors
Adds test step 6 that verifies all files under /tmp/gh-aw/mcp-logs/ are readable. This ensures permission issues (like the wazero-cache 0700 directory) cause an explicit smoke test failure instead of being silently swallowed by the framework's continue-on-error on artifact upload. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent e7ecbe6 commit 9a5c9c2

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

.github/workflows/smoke-copilot.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,7 @@ timeout-minutes: 15
8383
3. **Playwright Testing**: Use the playwright tools to navigate to https://github.com and verify the page title contains "GitHub" (do NOT try to install playwright - use the provided MCP tools)
8484
4. **File Writing Testing**: Create a test file `/tmp/gh-aw/agent/smoke-test-copilot-${{ github.run_id }}.txt` with content "Smoke test passed for Copilot at $(date)" (create the directory if it doesn't exist)
8585
5. **Bash Tool Testing**: Execute bash commands to verify file creation was successful (use `cat` to read the file back)
86+
6. **Artifact Readability Testing**: Run `find /tmp/gh-aw/mcp-logs/ -not -readable 2>/dev/null` and verify the output is empty. If any files or directories are not readable, this test MUST FAIL — report the unreadable paths and mark the overall status as FAIL. This catches permission issues (like restrictive wazero-cache directories) that would silently break artifact uploads.
8687

8788
## Output
8889

0 commit comments

Comments
 (0)