Fix assert for external interpreter

Author: aaronj0

lib/Interpreter/CppInterOp.cpp

@@ -2718,7 +2718,7 @@ namespace Cpp {
   TInterp_t GetInterpreter() { return sInterpreter; }
 
   void UseExternalInterpreter(TInterp_t I) {
-    assert(sInterpreter && "sInterpreter already in use!");
+    assert(!sInterpreter && "sInterpreter already in use!");
     sInterpreter = static_cast<compat::Interpreter*>(I);
     OwningSInterpreter = false;
   }

unittests/CppInterOp/InterpreterTest.cpp

@@ -200,7 +200,7 @@ TEST(InterpreterTest, ExternalInterpreterTest) {
 #endif
 
   EXPECT_NE(ExtInterp, nullptr);
-  Cpp::UseExternalInterpreter(ExtInterp);
+  ASSERT_DEATH(Cpp::UseExternalInterpreter(ExtInterp), "sInterpreter already in use!");
   EXPECT_TRUE(Cpp::GetInterpreter()) << "External Interpreter not set";
 
 #ifdef USE_REPL