Skip to content

Commit

Permalink
Exclude tests from ALL target. (#147)
Browse files Browse the repository at this point in the history
* Exclude tests from ALL target.
* Build tests in CI.
* Ignore tests failing in the CI.
  • Loading branch information
Holt59 authored Jun 8, 2024
1 parent 9da371b commit c6c5f27
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 2 deletions.
13 changes: 13 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
@@ -1,15 +1,28 @@
name: Build UIBase

on:
push:
branches: master
pull_request:
types: [opened, synchronize, reopened]

jobs:
build:
runs-on: windows-2022
steps:
- name: Build UI Base
id: build-uibase
uses: ModOrganizer2/build-with-mob-action@master
with:
mo2-third-parties: gtest spdlog boost
mo2-dependencies: cmake_common
- name: Build UI Base Tests
run: cmake --build vsbuild --config RelWithDebInfo -j4 --target uibase-tests
working-directory: ${{ steps.build-uibase.outputs.working-directory }}

- name: Test UI Base
# there are missing stuffs in the CI currently that fails the test, to be
# investigated
continue-on-error: true
run: ctest --test-dir vsbuild -C RelWithDebInfo --output-on-failure
working-directory: ${{ steps.build-uibase.outputs.working-directory }}
2 changes: 2 additions & 0 deletions .github/workflows/linting.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
name: Lint UIBase

on:
push:
pull_request:
types: [opened, synchronize, reopened]

jobs:
lint:
runs-on: ubuntu-latest
Expand Down
4 changes: 2 additions & 2 deletions tests/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
cmake_minimum_required(VERSION 3.16)

add_executable(uibase_tests)
mo2_configure_tests(uibase_tests
add_executable(uibase-tests EXCLUDE_FROM_ALL)
mo2_configure_tests(uibase-tests
WARNINGS OFF DEPENDS uibase)

0 comments on commit c6c5f27

Please sign in to comment.