Improve the test coverage of the C API -- Take 1

Author: Gnimuc

lib/Interpreter/CppInterOp.cpp

@@ -783,12 +783,16 @@ namespace Cpp {
     return false;
   }
 
-  TCppFunction_t GetDefaultConstructor(TCppScope_t scope) {
+  TCppFunction_t GetDefaultConstructor(compat::Interpreter& interp, TCppScope_t scope) {
     if (!HasDefaultConstructor(scope))
       return nullptr;
 
     auto *CXXRD = (clang::CXXRecordDecl*)scope;
-    return getSema().LookupDefaultConstructor(CXXRD);
+    return interp.getCI()->getSema().LookupDefaultConstructor(CXXRD);
+  }
+
+  TCppFunction_t GetDefaultConstructor(TCppScope_t scope) {
+    return GetDefaultConstructor(getInterp(), scope);
   }
 
   TCppFunction_t GetDestructor(TCppScope_t scope) {
@@ -3376,7 +3380,7 @@ namespace Cpp {
     if (!HasDefaultConstructor(Class))
       return nullptr;
 
-    auto* const Ctor = GetDefaultConstructor(Class);
+    auto* const Ctor = GetDefaultConstructor(interp, Class);
     if (JitCall JC = MakeFunctionCallable(&interp, Ctor)) {
       if (arena) {
         JC.Invoke(&arena, {}, (void*)~0); // Tell Invoke to use placement new.

unittests/CppInterOp/InterpreterTest.cpp

@@ -117,6 +117,16 @@ TEST(InterpreterTest, CreateInterpreter) {
   auto CXI = clang_createInterpreterFromRawPtr(I);
   auto CLI = clang_Interpreter_getClangInterpreter(CXI);
   EXPECT_TRUE(CLI);
+
+  clang_Interpreter_addSearchPath(CXI, "dummy", false, false);
+  clang_Interpreter_addIncludePath(CXI, "dummy");
+  clang_Interpreter_declare(CXI, "#include <iostream>", false);
+  clang_Interpreter_process(CXI, "int c = 42;");
+  auto CXV = clang_createValue();
+  auto Res = clang_Interpreter_evaluate(CXI, "c", CXV);
+  EXPECT_EQ(Res, CXError_Success);
+  
+  clang_Value_dispose(CXV);
   auto I2 = clang_Interpreter_takeInterpreterAsPtr(CXI);
   EXPECT_EQ(I, I2);
   clang_Interpreter_dispose(CXI);