Add CI step to build with -DFLATBUFFERS_NO_FILE_TESTS. (google#7729)

* Add CI step to build with -DFLATBUFFERS_NO_FILE_TESTS * Fix cmake syntax * Further fix cmake argumetns * Add workaround for unused-parameter. * Remove build matrix Co-authored-by: Derek Bailey <derekbailey@google.com>
sunwen18 · Dec 25, 2022 · d8792cc · d8792cc
1 parent c45e872
commit d8792cc
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -62,6 +62,16 @@ jobs:
       if: matrix.cxx == 'g++-10' && startsWith(github.ref, 'refs/tags/')
       id: hash-gcc
       run: echo "::set-output name=hashes::$(sha256sum Linux.flatc.binary.${{ matrix.cxx }}.zip | base64 -w0)"
+
+  build-linux-no-file-tests:    
+    name: Build Linux with -DFLATBUFFERS_NO_FILE_TESTS
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - name: cmake
+      run: CXX=clang++-12 cmake -G "Unix Makefiles" -DCMAKE_BUILD_TYPE=Release -DFLATBUFFERS_STRICT_MODE=ON -DFLATBUFFERS_CXX_FLAGS="-DFLATBUFFERS_NO_FILE_TESTS" .
+    - name: build
+      run: make -j
 
   build-linux-cpp-std:
     name: Build Linux C++

diff --git a/tests/test.cpp b/tests/test.cpp
@@ -1511,6 +1511,9 @@ int FlatBufferTests(const std::string &tests_data_path) {
   ParseIncorrectMonsterJsonTest(tests_data_path);
   FixedLengthArraySpanTest(tests_data_path);
   DoNotRequireEofTest(tests_data_path);
+#else
+  // Guard against -Wunused-parameter.
+  (void)tests_data_path;
 #endif
 
   UtilConvertCase();