Skip to content

Commit eebfd72

Browse files
committed
workflow: test both case whether need volatile
1 parent 491a52e commit eebfd72

File tree

2 files changed

+13
-1
lines changed

2 files changed

+13
-1
lines changed

.github/workflows/ci.yml

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,8 @@ jobs:
1515
strategy:
1616
matrix:
1717
os: [ubuntu-latest, macos-latest]
18+
need_volatile: [ON, OFF]
19+
build_unit_test_separately: [ON, OFF]
1820

1921
runs-on: ${{matrix.os}}
2022

@@ -48,7 +50,13 @@ jobs:
4850
- name: Configure CMake
4951
# Configure CMake in a 'build' subdirectory. `CMAKE_BUILD_TYPE` is only required if you are using a single-configuration generator such as make.
5052
# See https://cmake.org/cmake/help/latest/variable/CMAKE_BUILD_TYPE.html?highlight=cmake_build_type
51-
run: cmake -B ${{github.workspace}}/build -DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}} -DBUILD_TEST=TRUE -DENABLE_MYOSTREAM_WATCH=TRUE
53+
run: >
54+
cmake -B ${{github.workspace}}/build
55+
-DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}}
56+
-DBUILD_TEST=TRUE
57+
-DENABLE_MYOSTREAM_WATCH=TRUE
58+
-DNEED_VOLATILE=${{matrix.need_volatile}}
59+
-DUNIT_TEST_SEPARATE=${{matrix.build_unit_test_separately}}
5260
5361
- name: Build
5462
# Build your program with the given configuration

test/unit_test/register_member.cpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -570,6 +570,10 @@ TEST(register_member, nonvolatile_member) {
570570

571571
const bool SupportV = PEACALM_LUAW_SUPPORT_VOLATILE_OBJECT;
572572

573+
#if SupportV
574+
EXPECT_TRUE(false);
575+
#endif
576+
573577
{
574578
volatile Obj o;
575579
l.set("o", &o);

0 commit comments

Comments
 (0)