bzuccaro/rustunnel
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: add shell: bash to negative smoke checks for cross-platform portability

Browse Source 
The 'Invalid command fails' and 'Invalid port fails' smoke test steps use
bash if/then/fi syntax that fails on Windows runners (default pwsh shell).
Explicitly setting shell: bash makes these steps portable across the
Windows/Linux/macOS CI matrix.
This commit is contained in:
c4ch3c4d3
2026-06-03 17:59:53 -06:00
parent 70411e56f7
commit 03dab7c2c8
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.github/workflows/ci.yml
+2
View File
@@ -104,12 +104,14 @@ jobs:
- name: rustunnel generate --help - name: rustunnel generate --help
run: cargo run --release -- generate --help run: cargo run --release -- generate --help
- name: Invalid command fails - name: Invalid command fails
shell: bash
run: | run: |
if cargo run --release -- invalid-cmd 2>&1; then if cargo run --release -- invalid-cmd 2>&1; then
echo "ERROR: invalid command should have failed" echo "ERROR: invalid command should have failed"
exit 1 exit 1
fi fi
- name: Invalid port fails - name: Invalid port fails
shell: bash
run: | run: |
if cargo run --release -- listen --listen localhost:bad --cert x --key x --ca-cert x --auth-token x 2>&1; then if cargo run --release -- listen --listen localhost:bad --cert x --key x --ca-cert x --auth-token x 2>&1; then
echo "ERROR: invalid port should have failed" echo "ERROR: invalid port should have failed"