Skip to content

Commit

Permalink
ci: enable unicode string matchers
Browse files Browse the repository at this point in the history
  • Loading branch information
DNKpp committed Oct 4, 2024
1 parent 122980a commit fd4267b
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
9 changes: 8 additions & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
${{ matrix.config.prefix }}
${{ matrix.config.compiler_name }}-${{ matrix.config.compiler_version }}
${{ matrix.config.suffix }}
(C++${{ matrix.cxx_standard }}, ${{ matrix.build_mode }}, ${{ matrix.format_backend }})
(C++${{ matrix.cxx_standard }}, ${{ matrix.build_mode }}, ${{ matrix.format_backend }}, ${{ matrix.str_matcher }})
runs-on: ${{ matrix.config.os }}
container: ${{ matrix.config.container }}

Expand All @@ -21,6 +21,7 @@ jobs:
build_mode: [Debug, Release]
cxx_standard: [20, 23]
format_backend: [std, fmt]
str_matcher: [char, unicode]
config:
#clang-18
- {
Expand Down Expand Up @@ -226,6 +227,12 @@ jobs:
run: |
echo "CMAKE_CONFIG_EXTRA=$(echo $CMAKE_CONFIG_EXTRA -DMIMICPP_CONFIG_USE_FMT=YES)" >> $GITHUB_ENV
- name: Enable unicode support for string matchers
if: startsWith(matrix.str_matcher, 'unicode')
shell: bash
run: |
echo "CMAKE_CONFIG_EXTRA=$(echo $CMAKE_CONFIG_EXTRA -MIMICPP_CONFIG_EXPERIMENTAL_UNICODE_STR_MATCHER=YES)" >> $GITHUB_ENV
# ASan has some serious trouble with libc++ exception mechanism
# see: https://github.com/llvm/llvm-project/issues/59432
- name: Disable alloc_dealloc_mismatch detection with libc++
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/coverage.yml
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,8 @@ jobs:
-B build \
-D CMAKE_BUILD_TYPE="Debug" \
-D MIMICPP_ENABLE_ADAPTER_TESTS=YES \
-D MIMICPP_CONFIG_EXPERIMENTAL_CATCH2_MATCHER_INTEGRATION=YES
-D MIMICPP_CONFIG_EXPERIMENTAL_CATCH2_MATCHER_INTEGRATION=YES \
-D MIMICPP_CONFIG_EXPERIMENTAL_UNICODE_STR_MATCHER=YES
- name: Build
run: |
Expand Down

0 comments on commit fd4267b

Please sign in to comment.