Skip to content

Commit 2f86882

Browse files
authored
cpp: Enable acceptance tests on Ubuntu and OSX (#413)
1 parent 919f543 commit 2f86882

File tree

2 files changed

+23
-1
lines changed

2 files changed

+23
-1
lines changed

.github/workflows/test-cpp.yml

Lines changed: 22 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,3 +44,25 @@ jobs:
4444
working-directory: cpp
4545
run: |
4646
cmake -P cmake/cmate --cc=${{ matrix.config.cc }} build --release
47+
test-cpp-acceptance:
48+
strategy:
49+
matrix:
50+
config:
51+
- os: ubuntu-latest
52+
cc: gcc
53+
- os: ubuntu-latest
54+
cc: clang
55+
- os: macos-latest
56+
cc: clang
57+
# Doesn't work on windows. See: https://github.com/cucumber/gherkin/issues/412
58+
# If fixed, please merge test-cpp and test-cpp-acceptance
59+
# - os: windows-latest
60+
# cc: cl
61+
runs-on: ${{ matrix.config.os }}
62+
steps:
63+
- uses: actions/checkout@v4
64+
- uses: ilammy/msvc-dev-cmd@v1
65+
- name: run acceptance tests
66+
working-directory: cpp
67+
run: |
68+
make acceptance

cpp/Makefile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ acceptance: .built $(TOKENS) $(ASTS) $(PICKLES) $(ERRORS) $(SOURCES) ## Build ac
6868

6969
.built: $(SOURCE_FILES)
7070
./cmake/cmate install
71-
./cmake/cmate build
71+
./cmake/cmate build --release
7272
./cmake/cmate stage
7373
touch $@
7474

0 commit comments

Comments
 (0)