Skip to content

Commit 459e0ff

Browse files
author
Han Wang
committed
fix(test): include DeepPotPTExpt.h so the ZBL C++ suite actually runs
Without it BUILD_PT_EXPT is undefined in this translation unit and all 8 cases GTEST_SKIP with "PyTorch support is not enabled" -- which reads as a clean run in the summary. Caught on the T4: the suite reported 22 passed covering only the spin tests, with the ZBL suite silently skipped.
1 parent 648b3db commit 459e0ff

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

source/api_cc/tests/test_deeppot_dpa4_zbl_ptexpt.cc

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,11 @@
2727
#include <vector>
2828

2929
#include "DeepPot.h"
30+
// Defines BUILD_PT_EXPT (via its __has_include probe for the inductor
31+
// headers). Without this include the guards below see it undefined and
32+
// every case GTEST_SKIPs with "PyTorch support is not enabled" -- silently,
33+
// which is how this suite first ran as 8 skips that looked like passes.
34+
#include "DeepPotPTExpt.h"
3035
#include "expected_ref.h"
3136
#include "neighbor_list.h"
3237
#include "test_utils.h"

0 commit comments

Comments
 (0)