From 62e31951336f83eecb07dbab17a987f5415f6046 Mon Sep 17 00:00:00 2001 From: AlexandreSinger Date: Wed, 20 Mar 2024 16:07:08 -0400 Subject: [PATCH] [CI][Capnproto] Added Basic Regression Test with CAPNPROTO Disabled There has been some regressions with regard to CAPNPROTO being disabled since all of the CI tests have CAPNPROTO enabled. Added a basic regression test to the CI which will run the vtr_reg_basic tests with CAPNPROTO disabled. This will test both that everything builds and basic VPR tests will run. In the future we should test direct features that disabling CAPNPROTO would affect (such as writing and reading routing files); however, for now this will prevent the regressions previous seen. --- .github/workflows/test.yml | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 90b99ac89b7..cc5fffb4b55 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -197,6 +197,11 @@ jobs: params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVPR_USE_EZGL=off', suite: 'vtr_reg_basic' }, + { + name: 'Basic with CAPNPROTO disabled', + params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_CAPNPROTO=off', + suite: 'vtr_reg_basic' + }, { name: 'Basic with VTR_ENABLE_DEBUG_LOGGING', params: '-DVTR_ASSERT_LEVEL=3 -DWITH_BLIFEXPLORER=on -DVTR_ENABLE_DEBUG_LOGGING=on',