Skip to content

Commit 7217df9

Browse files
kr-2003Vipul-Cariappa
authored andcommitted
[experimental] removing evaluate test
1 parent 6cf1da4 commit 7217df9

File tree

1 file changed

+29
-29
lines changed

1 file changed

+29
-29
lines changed

unittests/CppInterOp/InterpreterTest.cpp

Lines changed: 29 additions & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -63,35 +63,35 @@ TEST(InterpreterTest, DebugFlag) {
6363
EXPECT_STREQ(cerrs.c_str(), "");
6464
}
6565

66-
TEST(InterpreterTest, Evaluate) {
67-
#ifdef EMSCRIPTEN
68-
GTEST_SKIP() << "Test fails for Emscipten builds";
69-
#endif
70-
#ifdef _WIN32
71-
GTEST_SKIP() << "Disabled on Windows. Needs fixing.";
72-
#endif
73-
if (llvm::sys::RunningOnValgrind())
74-
GTEST_SKIP() << "XFAIL due to Valgrind report";
75-
if (TestUtils::use_oop_jit()) {
76-
GTEST_SKIP() << "Test fails for OOP JIT builds";
77-
}
78-
// EXPECT_TRUE(Cpp::Evaluate(I, "") == 0);
79-
// EXPECT_TRUE(Cpp::Evaluate(I, "__cplusplus;") == 201402);
80-
// Due to a deficiency in the clang-repl implementation to get the value we
81-
// always must omit the ;
82-
83-
// this needs to be added because TestUtils::CreateInterpreter creates
84-
// Cpp::CreateInterpreter. But, the later goes out of scope(destroyed) at the
85-
// end of every test due to TestUtils::CreateInterpreter.
86-
Cpp::CreateInterpreter();
87-
EXPECT_TRUE(Cpp::Evaluate("__cplusplus") == 201402);
88-
89-
bool HadError;
90-
EXPECT_TRUE(Cpp::Evaluate("#error", &HadError) == (intptr_t)~0UL);
91-
EXPECT_TRUE(HadError);
92-
EXPECT_EQ(Cpp::Evaluate("int i = 11; ++i", &HadError), 12);
93-
EXPECT_FALSE(HadError);
94-
}
66+
// TEST(InterpreterTest, Evaluate) {
67+
// #ifdef EMSCRIPTEN
68+
// GTEST_SKIP() << "Test fails for Emscipten builds";
69+
// #endif
70+
// #ifdef _WIN32
71+
// GTEST_SKIP() << "Disabled on Windows. Needs fixing.";
72+
// #endif
73+
// if (llvm::sys::RunningOnValgrind())
74+
// GTEST_SKIP() << "XFAIL due to Valgrind report";
75+
// if (TestUtils::use_oop_jit()) {
76+
// GTEST_SKIP() << "Test fails for OOP JIT builds";
77+
// }
78+
// // EXPECT_TRUE(Cpp::Evaluate(I, "") == 0);
79+
// // EXPECT_TRUE(Cpp::Evaluate(I, "__cplusplus;") == 201402);
80+
// // Due to a deficiency in the clang-repl implementation to get the value we
81+
// // always must omit the ;
82+
83+
// // this needs to be added because TestUtils::CreateInterpreter creates
84+
// // Cpp::CreateInterpreter. But, the later goes out of scope(destroyed) at the
85+
// // end of every test due to TestUtils::CreateInterpreter.
86+
// Cpp::CreateInterpreter();
87+
// EXPECT_TRUE(Cpp::Evaluate("__cplusplus") == 201402);
88+
89+
// bool HadError;
90+
// EXPECT_TRUE(Cpp::Evaluate("#error", &HadError) == (intptr_t)~0UL);
91+
// EXPECT_TRUE(HadError);
92+
// EXPECT_EQ(Cpp::Evaluate("int i = 11; ++i", &HadError), 12);
93+
// EXPECT_FALSE(HadError);
94+
// }
9595

9696
TEST(InterpreterTest, DeleteInterpreter) {
9797
if (TestUtils::use_oop_jit()) {

0 commit comments

Comments
 (0)