Skip to content

Commit

Permalink
GHA: add self-hosted step for windows jobs.
Browse files Browse the repository at this point in the history
Add --no-pty to self-hosted vbs/vbscript TEST_OPTS
  • Loading branch information
kanaka committed Aug 22, 2024
1 parent 7800635 commit fd626a7
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 2 deletions.
16 changes: 14 additions & 2 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ jobs:
if: ${{ github.event.inputs.self-hosted == 'yes' }}
run: |
export ${{ matrix.IMPL }}
if [ "${NO_SELF_HOST}" ]; then
if [ -n "${NO_SELF_HOST:-}" ]; then
echo "Skipping self-host for ${IMPL} due to NO_SELF_HOST variable"
else
DO_SELF_HOST=1 ./ci.sh test ${IMPL}
Expand Down Expand Up @@ -130,7 +130,7 @@ jobs:
if: ${{ github.event.inputs.self-hosted == 'yes' }}
run: |
export ${{ matrix.IMPL }}
if [ "${NO_SELF_HOST}" ]; then
if [ -n "${NO_SELF_HOST:-}" ]; then
echo "Skipping self-host for ${IMPL} due to NO_SELF_HOST variable"
else
DO_SELF_HOST=1 ./ci.sh test ${IMPL}
Expand Down Expand Up @@ -187,6 +187,18 @@ jobs:
run: |
export ${{ matrix.IMPL }}
./ci.sh perf ${IMPL}
- name: Self-hosted Tests
if: ${{ github.event.inputs.self-hosted == 'yes' }}
shell: wsl-bash {0}
run: |
export ${{ matrix.IMPL }}
if [ -n "${NO_SELF_HOST:-}" ]; then
echo "Skipping self-host for ${IMPL} due to NO_SELF_HOST variable"
else
DO_SELF_HOST=1 ./ci.sh test ${IMPL}
# Check that self-hosted mode really ran
[ "`grep -a "mal-user>" test-mal-*${IMPL}.debug | wc -l`" -gt 800 ]
fi
- name: Print debug log
if: failure()
run: cat *.debug
Expand Down
2 changes: 2 additions & 0 deletions Makefile.impls
Original file line number Diff line number Diff line change
Expand Up @@ -77,6 +77,8 @@ ifeq ($(MAL_IMPL),vimscript)
mal_TEST_OPTS = --start-timeout 60 --test-timeout 180
else ifeq ($(MAL_IMPL),powershell)
mal_TEST_OPTS = --start-timeout 60 --test-timeout 180
else ifeq ($(MAL_IMPL),vbs)
mal_TEST_OPTS = --start-timeout 60 --test-timeout 180 --no-pty
endif
xslt_TEST_OPTS = --test-timeout 120
vbs_TEST_OPTS = --no-pty
Expand Down

0 comments on commit fd626a7

Please sign in to comment.