Skip to content

Commit

Permalink
ci: Get rid of redundant testing
Browse files Browse the repository at this point in the history
It's not necessary to run the runtest tests separately, as they're
already being run by the ctest testing process (as glslang-testsuite).
  • Loading branch information
arcady-lunarg committed Aug 14, 2024
1 parent 3a08cd8 commit 0dc6711
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions .github/workflows/continuous_integration.yml
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,6 @@ jobs:
run: cmake --install build --prefix build/install
- name: Test
run: ctest --output-on-failure --test-dir build
- name: Test (standalone)
run: cd Test && ./runtests
- name: Check known validation failure list
run: grep -l 'Validation failed' Test/baseResults/* | sort -fd | diff -u Test/baseResults/validation_fails.txt -

Expand Down Expand Up @@ -87,10 +85,6 @@ jobs:
env:
UBSAN_OPTIONS: 'halt_on_error=1:print_stacktrace=1'
run: ctest --output-on-failure --test-dir build
- name: Test (standalone)
env:
UBSAN_OPTIONS: halt_on_error=1:print_stacktrace=1
run: cd Test && ./runtests

# Ensure we can compile/run on an older distro
linux_min:
Expand Down Expand Up @@ -120,8 +114,6 @@ jobs:
run: cmake --install build --prefix build/install
- name: Test
run: ctest --output-on-failure --test-dir build
- name: Test (standalone)
run: cd Test && ./runtests

macos:
runs-on: ${{matrix.os}}
Expand All @@ -142,9 +134,6 @@ jobs:
- run: cmake --build build
- run: cmake --install build --prefix build/install
- run: ctest --output-on-failure --test-dir build
- name: Test Script (standalone)
run: ./runtests
working-directory: Test

windows:
runs-on: ${{matrix.os.genus}}
Expand All @@ -168,8 +157,6 @@ jobs:
cmake --build build --config ${{matrix.cmake_build_type}} --target install
- name: Test
run: ctest -C ${{matrix.cmake_build_type}} --output-on-failure --test-dir build
- name: Test (standalone)
run: bash -c 'cd ./Test && ./runtests'

iOS:
runs-on: macos-13
Expand Down

0 comments on commit 0dc6711

Please sign in to comment.