Skip to content

Commit

Permalink
test: Turn windows tests on in github CI
Browse files Browse the repository at this point in the history
Now that the Vulkan-Loader's find_package support is available, this repo can enable
testing on windows and have it work.

Github actions runs everything in 'admin mode' which causes the loader to ignore
environment variables, which is how the tests find MockICD. Thus, to work around
this behavior we build the loader with LOADER_USE_UNSAFE_FILE_SEARCH so that it
will use environment variables again.
  • Loading branch information
charles-lunarg committed Sep 19, 2023
1 parent 475f26c commit f65ad5c
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 1 deletion.
6 changes: 5 additions & 1 deletion .github/workflows/tools.yml
Original file line number Diff line number Diff line change
Expand Up @@ -116,14 +116,18 @@ jobs:
arch: ${{ matrix.arch }}

- name: Configure
run: cmake -S. -B build -G "Ninja" -DCMAKE_BUILD_TYPE=${{matrix.config}} -D UPDATE_DEPS=ON -D INSTALL_ICD=ON
run: cmake -S. -B build -G "Ninja" -DCMAKE_BUILD_TYPE=${{matrix.config}} -D UPDATE_DEPS=ON -D INSTALL_ICD=ON -D BUILD_TESTS=ON

- name: Build
run: cmake --build ./build

- name: Install
run: cmake --install build/ --prefix build/install

- name: Test
working-directory: ./build
run: ctest --output-on-failure

- name: Verify generated source files
run: python scripts/generate_source.py --verify external/${{matrix.config}}/Vulkan-Headers/build/install/share/vulkan/registry

Expand Down
3 changes: 3 additions & 0 deletions scripts/known_good.json
Original file line number Diff line number Diff line change
Expand Up @@ -32,6 +32,9 @@
"sub_dir": "Vulkan-Loader",
"build_dir": "Vulkan-Loader/build",
"install_dir": "Vulkan-Loader/build/install",
"cmake_options": [
"-DLOADER_USE_UNSAFE_FILE_SEARCH=ON"
],
"commit": "d40385b1748ae5270fcab18c5aae1a2e5a301465",
"deps": [
{
Expand Down

0 comments on commit f65ad5c

Please sign in to comment.