Skip to content

Commit e67fcfd

Browse files
authored
fix(ci): fix Ruby integration test failures (#158)
1 parent 5b2b2d7 commit e67fcfd

3 files changed

Lines changed: 16 additions & 4 deletions

File tree

.github/workflows/integration-test-migrate-ruby-macos-rbenv.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,8 @@ jobs:
4949
- name: "Migrate rbenv Ruby to dtvem"
5050
run: |
5151
echo "=== Running migrate detection ==="
52-
echo -e "1\n0\nn\n" | ./dist/dtvem migrate ruby || true
52+
# Select "all" sources and "0" for all versions (rbenv may not be first source)
53+
echo -e "all\n0\nn\n" | ./dist/dtvem migrate ruby || true
5354
echo ""
5455
echo "=== Verifying migration ==="
5556
./dist/dtvem list ruby

.github/workflows/integration-test-migrate-ruby-ubuntu-rbenv.yml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,8 @@ jobs:
5757
- name: "Migrate rbenv Ruby to dtvem"
5858
run: |
5959
echo "=== Running migrate detection ==="
60-
echo -e "1\n0\nn\n" | ./dist/dtvem migrate ruby || true
60+
# Select "all" sources and "0" for all versions (rbenv may not be first source)
61+
echo -e "all\n0\nn\n" | ./dist/dtvem migrate ruby || true
6162
echo ""
6263
echo "=== Verifying migration ==="
6364
./dist/dtvem list ruby

.github/workflows/integration-test-ruby.yml

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -116,17 +116,27 @@ jobs:
116116
# Verify test versions are available using --filter and stripping ANSI codes
117117
echo "Verifying test versions are available..."
118118
119-
# Check version1
119+
# Check version1 - handle rate limiting gracefully
120120
FILTERED=$(./dtvem${{ matrix.ext }} list-all ruby --filter "${{ matrix.version1 }}" 2>&1 | sed 's/\x1B\[[0-9;]*[a-zA-Z]//g')
121+
if echo "$FILTERED" | grep -qE "(HTTP 403|HTTP 429|rate limit)"; then
122+
echo "⚠ Warning: GitHub API rate limited, skipping version verification"
123+
echo "Proceeding with install - it will fail if version doesn't exist"
124+
exit 0
125+
fi
121126
if ! echo "$FILTERED" | grep -qF "${{ matrix.version1 }}"; then
122127
echo "ERROR: Version ${{ matrix.version1 }} not found in available versions"
123128
echo "Filtered output: $FILTERED"
124129
exit 1
125130
fi
126131
echo "✓ Version ${{ matrix.version1 }} is available"
127132
128-
# Check version2
133+
# Check version2 - handle rate limiting gracefully
129134
FILTERED=$(./dtvem${{ matrix.ext }} list-all ruby --filter "${{ matrix.version2 }}" 2>&1 | sed 's/\x1B\[[0-9;]*[a-zA-Z]//g')
135+
if echo "$FILTERED" | grep -qE "(HTTP 403|HTTP 429|rate limit)"; then
136+
echo "⚠ Warning: GitHub API rate limited, skipping version verification"
137+
echo "Proceeding with install - it will fail if version doesn't exist"
138+
exit 0
139+
fi
130140
if ! echo "$FILTERED" | grep -qF "${{ matrix.version2 }}"; then
131141
echo "ERROR: Version ${{ matrix.version2 }} not found in available versions"
132142
echo "Filtered output: $FILTERED"

0 commit comments

Comments
 (0)