Skip to content

Commit e489836

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

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

.github/workflows/ci.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ jobs:
1515
strategy:
1616
matrix:
1717
os: [ubuntu-latest, macos-latest]
18+
need_volatile: [true, false]
1819

1920
runs-on: ${{matrix.os}}
2021

@@ -48,7 +49,12 @@ jobs:
4849
- name: Configure CMake
4950
# Configure CMake in a 'build' subdirectory. `CMAKE_BUILD_TYPE` is only required if you are using a single-configuration generator such as make.
5051
# 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
52+
run: >
53+
cmake -B ${{github.workspace}}/build
54+
-DCMAKE_BUILD_TYPE=${{env.BUILD_TYPE}}
55+
-DBUILD_TEST=TRUE
56+
-DENABLE_MYOSTREAM_WATCH=TRUE
57+
-DPEACALM_LUAW_SUPPORT_VOLATILE_OBJECT=${{matrix.need_volatile}}
5258
5359
- name: Build
5460
# 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)