Skip to content

Commit 8df1dd0

Browse files
runningcodeclaude
andcommitted
Update test workflow to use real APK file
- Copy real APK from hackernews project to test_artifacts/ - Update workflow to use app-debug.apk instead of creating fake file - This provides more realistic testing for sentry-cli build upload 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent c52be30 commit 8df1dd0

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

.github/workflows/test-pr-head-sha.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -63,12 +63,11 @@ jobs:
6363
echo ""
6464
fi
6565
66-
# Create a dummy APK file for testing
67-
mkdir -p test_artifacts
68-
echo "fake apk content" > test_artifacts/test.apk
66+
# Use the real APK file that should already be in the repo
67+
ls -la test_artifacts/ || echo "No test_artifacts directory found"
6968
7069
echo "=== Running sentry-cli build upload with debug logging ==="
71-
echo "Command: ./target/release/sentry-cli --log-level debug build upload test_artifacts/test.apk"
70+
echo "Command: ./target/release/sentry-cli --log-level debug build upload test_artifacts/app-debug.apk"
7271
echo ""
7372
echo "🔍 WATCH FOR DEBUG LOGS BELOW:"
7473
echo "Expected debug log: 'Using GitHub Actions PR head SHA from event payload: ${{ github.event.pull_request.head.sha }}'"
@@ -77,7 +76,7 @@ jobs:
7776
7877
# Test the CLI's head-sha detection with debug logging enabled
7978
# Using a dummy auth token to get past auth check and reach the head-sha detection logic
80-
./target/release/sentry-cli --log-level debug --auth-token dummy_token_for_testing build upload test_artifacts/test.apk || true
79+
./target/release/sentry-cli --log-level debug --auth-token dummy_token_for_testing build upload test_artifacts/app-debug.apk || true
8180
8281
echo "--- CLI OUTPUT END ---"
8382
@@ -97,7 +96,7 @@ jobs:
9796
unset GITHUB_EVENT_NAME
9897
9998
echo "Running CLI without GitHub Actions context (with debug logging)..."
100-
./target/release/sentry-cli --log-level debug --auth-token dummy_token_for_testing build upload test_artifacts/test.apk || true
99+
./target/release/sentry-cli --log-level debug --auth-token dummy_token_for_testing build upload test_artifacts/app-debug.apk || true
101100
102101
echo ""
103102
echo "Expected: Should fall back to 'git rev-parse HEAD'"

test_artifacts/app-debug.apk

39.4 MB
Binary file not shown.

0 commit comments

Comments
 (0)