Merge pull request #16 from jonas/separate-test-program

Move tests to a separate executable

Author: kornilova203

.travis.yml

@@ -5,7 +5,7 @@ compiler: gcc
 script:
  - docker-compose build ubuntu-16.04-llvm-$LLVM_VERSION
  - docker run --rm -ti scala-native-bindgen:ubuntu-16.04-llvm-$LLVM_VERSION /usr/include/ctype.h -name ctype --
- - docker run --rm -ti scala-native-bindgen:ubuntu-16.04-llvm-$LLVM_VERSION
+ - docker run --rm -ti --entrypoint /src/target/scalaBindgenTest scala-native-bindgen:ubuntu-16.04-llvm-$LLVM_VERSION
 
 matrix:
   include:

CMakeLists.txt

@@ -11,8 +11,8 @@ add_definitions(${LLVM_DEFINITIONS})
 
 add_compile_options(-fexceptions -std=c++11)
 
-add_executable(scalaBindgen
-  Main.cpp
+add_library(scalaNativeBindgen
+  STATIC
   ScalaFrontend.h
   TreeVisitor.h
   TreeVisitor.cpp
@@ -23,12 +23,27 @@ add_executable(scalaBindgen
   HeaderManager.cpp
   CycleDetection.h
   Utils.h
-  catch/catch.hpp
-  SimpleTypeTests.cpp
+  )
+
+add_executable(scalaBindgen
+  Main.cpp
   )
 
 target_link_libraries(scalaBindgen
   PRIVATE
+  scalaNativeBindgen
+  clangFrontend
+  clangTooling
+  )
+
+add_executable(scalaBindgenTest
+  SimpleTypeTests.cpp
+  catch/catch.hpp
+  )
+
+target_link_libraries(scalaBindgenTest
+  PRIVATE
+  scalaNativeBindgen
   clangFrontend
   clangTooling
   )

Main.cpp

@@ -1,9 +1,6 @@
 #include "ScalaFrontend.h"
 #include "Utils.h"
 
-#define CATCH_CONFIG_RUNNER
-#include "catch/catch.hpp"
-
 #include <sys/types.h>
 
 static llvm::cl::OptionCategory Category("Binding Generator");
@@ -15,68 +12,59 @@ static llvm::cl::opt<bool> PrintHeadersLocation ("location", llvm::cl::cat(Categ
 
 
 int main(int argc, char *argv[]) {
+    clang::tooling::CommonOptionsParser op(argc, (const char**)argv, Category);
+    clang::tooling::ClangTool Tool(op.getCompilations(), op.getSourcePathList());
 
-    if(argc <= 1 ){
+    auto lib = LibName.getValue();
+    if(lib == ""){
+        llvm::errs() << "Error: Please specify the lib name using -name paramter\n";
+        return -1;
+    }
 
-        int result = Catch::Session().run( argc, argv );
-        return result;
+    auto stdhead = StdHeaders.getValue();
+    if(stdhead != ""){
+        headerMan.LoadConfig(stdhead);
+    }
 
-    } else{
+    declarations = "";
+    enums = "";
+    helpers = "";
+    locations.clear();
 
-        clang::tooling::CommonOptionsParser op(argc, (const char**)argv, Category);
-        clang::tooling::ClangTool Tool(op.getCompilations(), op.getSourcePathList());
+    int result = Tool.run(clang::tooling::newFrontendActionFactory<ScalaFrontendAction>().get());
 
-        auto lib = LibName.getValue();
-        if(lib == ""){
-            llvm::errs() << "Error: Please specify the lib name using -name paramter\n";
-            return -1;
-        }
+    auto printLoc = PrintHeadersLocation.getValue();
 
-        auto stdhead = StdHeaders.getValue();
-        if(stdhead != ""){
-            headerMan.LoadConfig(stdhead);
+    if(printLoc){
+        for(const auto& location: locations){
+            llvm::outs() << location;
         }
 
-        declarations = "";
-        enums = "";
-        helpers = "";
-        locations.clear();
-
-        int result = Tool.run(clang::tooling::newFrontendActionFactory<ScalaFrontendAction>().get());
-
-        auto printLoc = PrintHeadersLocation.getValue();
-
-        if(printLoc){
-            for(const auto& location: locations){
-                llvm::outs() << location;
-            }
-
-        } else {
-            if(declarations != "" || enums != "")
-            llvm::outs() << "import scala.scalanative._\n"
-                         << "import scala.scalanative.native.Nat._\n\n";
-
-            if(declarations != ""){
-                llvm::outs() << "@native.link(\"" << lib << "\")\n"
-                             << "@native.extern\n"
-                             << "object " << lib << " {\n"
-                             << declarations
-                             << "}\n\n"
-                             << "import " + lib + "._\n\n";
-            }
+    } else {
+        if(declarations != "" || enums != "")
+        llvm::outs() << "import scala.scalanative._\n"
+                     << "import scala.scalanative.native.Nat._\n\n";
+
+        if(declarations != ""){
+            llvm::outs() << "@native.link(\"" << lib << "\")\n"
+                         << "@native.extern\n"
+                         << "object " << lib << " {\n"
+                         << declarations
+                         << "}\n\n"
+                         << "import " + lib + "._\n\n";
+        }
 
-            if(enums != ""){
-                llvm::outs() << "object " << lib << "Enums {\n"
-                             << enums
-                             << "}\n\n";
-            }
+        if(enums != ""){
+            llvm::outs() << "object " << lib << "Enums {\n"
+                         << enums
+                         << "}\n\n";
+        }
 
-            if(helpers != ""){
-                llvm::outs() << "object " << lib << "Helpers {\n"
-                             << helpers
-                             << "}\n\n";
-            }
+        if(helpers != ""){
+            llvm::outs() << "object " << lib << "Helpers {\n"
+                         << helpers
+                         << "}\n\n";
         }
-        return result;
     }
+    return result;
 }

SimpleTypeTests.cpp

@@ -1,10 +1,17 @@
 #include "ScalaFrontend.h"
 
 #include "clang/Tooling/Tooling.h"
+
+#define CATCH_CONFIG_RUNNER
 #include "catch/catch.hpp"
 
 #include <iostream>
 
+int main(int argc, char *argv[]) {
+    int result = Catch::Session().run( argc, argv );
+    return result;
+}
+
 std::string Translate(std::string code){
     declarations = "";
     enums = "";