Refactor runner: runner/main -> runner/core

CMakeLists.txt

@@ -91,11 +91,11 @@ set(INCLUDE
         include/tcframe/os/OperatingSystem.hpp
         include/tcframe/runner/logger.hpp
         include/tcframe/runner/logger/RunnerLogger.hpp
-        include/tcframe/runner/main.hpp
-        include/tcframe/runner/main/Args.hpp
-        include/tcframe/runner/main/ArgsParser.hpp
-        include/tcframe/runner/main/RunnerMain.hpp
-        include/tcframe/runner/main/SlugParser.hpp
+        include/tcframe/runner/core.hpp
+        include/tcframe/runner/core/Args.hpp
+        include/tcframe/runner/core/ArgsParser.hpp
+        include/tcframe/runner/core/Runner.hpp
+        include/tcframe/runner/core/SlugParser.hpp
         include/tcframe/spec.hpp
         include/tcframe/spec/constraint.hpp
         include/tcframe/spec/constraint/Constraint.hpp
@@ -189,12 +189,12 @@ set(TEST_UNIT
         test/unit/tcframe/os/MockOperatingSystem.hpp
         test/unit/tcframe/runner/logger/MockRunnerLogger.hpp
         test/unit/tcframe/runner/logger/RunnerLoggerTests.cpp
-        test/unit/tcframe/runner/main/ArgsParserTests.cpp
-        test/unit/tcframe/runner/main/BatchRunnerMainTests.cpp
-        test/unit/tcframe/runner/main/BaseRunnerMainTests.hpp
-        test/unit/tcframe/runner/main/InteractiveRunnerMainTests.cpp
-        test/unit/tcframe/runner/main/RunnerMainTests.cpp
-        test/unit/tcframe/runner/main/SlugParserTests.cpp
+        test/unit/tcframe/runner/core/ArgsParserTests.cpp
+        test/unit/tcframe/runner/core/BatchRunnerTests.cpp
+        test/unit/tcframe/runner/core/BaseRunnerTests.hpp
+        test/unit/tcframe/runner/core/InteractiveRunnerTests.cpp
+        test/unit/tcframe/runner/core/RunnerTests.cpp
+        test/unit/tcframe/runner/core/SlugParserTests.cpp
         test/unit/tcframe/spec/constraint/ConstraintSuiteTests.cpp
         test/unit/tcframe/spec/constraint/ConstraintSuiteBuilderTests.cpp
         test/unit/tcframe/spec/core/BaseTestSpecTests.cpp

include/tcframe/runner.hpp

@@ -0,0 +1,4 @@
+#pragma once
+
+#include "tcframe/runner/core.hpp"
+#include "tcframe/runner/logger.hpp"

include/tcframe/runner/core.hpp

@@ -0,0 +1,6 @@
+#pragma once
+
+#include "tcframe/runner/core/Args.hpp"
+#include "tcframe/runner/core/ArgsParser.hpp"
+#include "tcframe/runner/core/Runner.hpp"
+#include "tcframe/runner/core/SlugParser.hpp"

include/tcframe/runner/main/RunnerMain.hpp → include/tcframe/runner/core/Runner.hpp

@@ -28,7 +28,7 @@ struct RunnerDefaults {
 };
 
 template<typename TProblemSpec>
-class RunnerMain {
+class Runner {
 private:
     string specPath_;
 
@@ -45,7 +45,7 @@ class RunnerMain {
     AggregatorRegistry* aggregatorRegistry_;
 
 public:
-    RunnerMain(
+    Runner(
             const string& specPath,
             BaseTestSpec<TProblemSpec>* testSpec,
             LoggerEngine* loggerEngine,

src/tcframe/runner.cpp

@@ -1,11 +1,11 @@
-#include "tcframe/runner/main.hpp"
+#include "tcframe/runner.hpp"
 
 #include __TCFRAME_SPEC_FILE__
 
-using tcframe::RunnerMain;
+using tcframe::Runner;
 
 int main(int argc, char* argv[]) {
-    RunnerMain<ProblemSpec> runner(
+    Runner<ProblemSpec> runner(
             __TCFRAME_SPEC_FILE__,
             new TestSpec(),
             new SimpleLoggerEngine(),

test/unit/tcframe/runner/main/ArgsParserTests.cpp → test/unit/tcframe/runner/core/ArgsParserTests.cpp

@@ -1,6 +1,6 @@
 #include "gmock/gmock.h"
 
-#include "tcframe/runner/main/ArgsParser.hpp"
+#include "tcframe/runner/core/ArgsParser.hpp"
 
 using ::testing::Eq;
 using ::testing::StrEq;

test/unit/tcframe/runner/main/BaseRunnerMainTests.hpp → test/unit/tcframe/runner/core/BaseRunnerTests.hpp

@@ -11,7 +11,7 @@
 #include "../../grader/MockGraderLoggerFactory.hpp"
 #include "../../os/MockOperatingSystem.hpp"
 #include "../logger/MockRunnerLogger.hpp"
-#include "tcframe/runner/main/RunnerMain.hpp"
+#include "tcframe/runner/core/Runner.hpp"
 
 using ::testing::_;
 using ::testing::Eq;
@@ -23,7 +23,7 @@ using ::testing::Truly;
 
 namespace tcframe {
 
-class BaseRunnerMainTests : public Test {
+class BaseRunnerTests : public Test {
 protected:
     class ProblemSpec : public BaseProblemSpec {
     protected:
@@ -63,8 +63,8 @@ class BaseRunnerMainTests : public Test {
     }
 
     template<typename TProblem>
-    RunnerMain<TProblem> createRunner(BaseTestSpec<TProblem>* testSpec) {
-        return RunnerMain<TProblem>(
+    Runner<TProblem> createRunner(BaseTestSpec<TProblem>* testSpec) {
+        return Runner<TProblem>(
                 specPath, testSpec, loggerEngine, &os,&runnerLoggerFactory, &graderLoggerFactory,
                 &generatorFactory, &graderFactory, &evaluatorRegistry, &aggregatorRegistry);
     }

test/unit/tcframe/runner/main/BatchRunnerMainTests.cpp → test/unit/tcframe/runner/core/BatchRunnerTests.cpp

@@ -1,8 +1,8 @@
-#include "BaseRunnerMainTests.hpp"
+#include "BaseRunnerTests.hpp"
 
 namespace tcframe {
 
-class BatchRunnerMainTests : public BaseRunnerMainTests {
+class BatchRunnerTests : public BaseRunnerTests {
 protected:
     class BatchProblemSpec : public ProblemSpec {
     protected:
@@ -22,39 +22,39 @@ class BatchRunnerMainTests : public BaseRunnerMainTests {
     class BatchTestSpec : public BaseTestSpec<BatchProblemSpec> {};
     class BatchWithCustomScorerTestSpec : public BaseTestSpec<BatchWithCustomScorerProblemSpec> {};
 
-    RunnerMain<BatchProblemSpec> runner = createRunner(new BatchTestSpec());
-    RunnerMain<BatchWithCustomScorerProblemSpec> runnerWithCustomScorer = createRunner(new BatchWithCustomScorerTestSpec());
+    Runner<BatchProblemSpec> runner = createRunner(new BatchTestSpec());
+    Runner<BatchWithCustomScorerProblemSpec> runnerWithCustomScorer = createRunner(new BatchWithCustomScorerTestSpec());
 };
 
-TEST_F(BatchRunnerMainTests, Run_Generation_EvaluatorRegistry_NoCustomScorer) {
+TEST_F(BatchRunnerTests, Run_Generation_EvaluatorRegistry_NoCustomScorer) {
     EXPECT_CALL(evaluatorRegistry, get(EvaluationStyle::BATCH, _, Truly(HelperKeyIs("scorer", ""))));
     runner.run(argc, argv);
 }
 
-TEST_F(BatchRunnerMainTests, Run_Generation_EvaluatorRegistry_CustomScorer_Default) {
+TEST_F(BatchRunnerTests, Run_Generation_EvaluatorRegistry_CustomScorer_Default) {
     EXPECT_CALL(evaluatorRegistry,
                 get(EvaluationStyle::BATCH, _,  Truly(HelperKeyIs("scorer", string(RunnerDefaults::SCORER_COMMAND)))));
 
     runnerWithCustomScorer.run(argc, argv);
 }
 
-TEST_F(BatchRunnerMainTests, Run_Generation_EvaluatorRegistry_CustomScorer_Args) {
+TEST_F(BatchRunnerTests, Run_Generation_EvaluatorRegistry_CustomScorer_Args) {
     EXPECT_CALL(evaluatorRegistry, get(EvaluationStyle::BATCH, _, Truly(HelperKeyIs("scorer", "\"java Scorer\""))));
     runnerWithCustomScorer.run(2, new char*[3]{
             (char*) "./runner",
             (char*) "--scorer=\"java Scorer\"",
             nullptr});
 }
 
-TEST_F(BatchRunnerMainTests, Run_Grading_EvaluatorRegistry_NoCustomScorer) {
+TEST_F(BatchRunnerTests, Run_Grading_EvaluatorRegistry_NoCustomScorer) {
     EXPECT_CALL(evaluatorRegistry, get(EvaluationStyle::BATCH, _, Truly(HelperKeyIs("scorer", ""))));
     runner.run(2, new char*[3]{
             (char*) "./runner",
             (char*) "grade",
             nullptr});
 }
 
-TEST_F(BatchRunnerMainTests, Run_Grading_EvaluatorRegistry_CustomScorer_Default) {
+TEST_F(BatchRunnerTests, Run_Grading_EvaluatorRegistry_CustomScorer_Default) {
     EXPECT_CALL(evaluatorRegistry,
                 get(EvaluationStyle::BATCH, _,  Truly(HelperKeyIs("scorer", string(RunnerDefaults::SCORER_COMMAND)))));
 
@@ -64,7 +64,7 @@ TEST_F(BatchRunnerMainTests, Run_Grading_EvaluatorRegistry_CustomScorer_Default)
             nullptr});
 }
 
-TEST_F(BatchRunnerMainTests, Run_Grading_EvaluatorRegistry_CustomScorer_Args) {
+TEST_F(BatchRunnerTests, Run_Grading_EvaluatorRegistry_CustomScorer_Args) {
     EXPECT_CALL(evaluatorRegistry, get(EvaluationStyle::BATCH, _, Truly(HelperKeyIs("scorer", "\"java Scorer\""))));
     runnerWithCustomScorer.run(3, new char*[4]{
             (char*) "./runner",

test/unit/tcframe/runner/main/InteractiveRunnerMainTests.cpp → test/unit/tcframe/runner/core/InteractiveRunnerTests.cpp

@@ -1,8 +1,8 @@
-#include "BaseRunnerMainTests.hpp"
+#include "BaseRunnerTests.hpp"
 
 namespace tcframe {
 
-class InteractiveRunnerMainTests : public BaseRunnerMainTests {
+class InteractiveRunnerTests : public BaseRunnerTests {
 protected:
     class InteractiveProblemSpec : public ProblemSpec {
     protected:
@@ -23,22 +23,22 @@ class InteractiveRunnerMainTests : public BaseRunnerMainTests {
     class InteractiveWithRedundantNoOutputTestSpec :
             public BaseTestSpec<InteractiveWithRedundantNoOutputProblemSpec> {};
 
-    RunnerMain<InteractiveProblemSpec> runner = createRunner(new InteractiveTestSpec());
-    RunnerMain<InteractiveWithRedundantNoOutputProblemSpec> runnerWithRedundantNoOutput =
+    Runner<InteractiveProblemSpec> runner = createRunner(new InteractiveTestSpec());
+    Runner<InteractiveWithRedundantNoOutputProblemSpec> runnerWithRedundantNoOutput =
             createRunner(new InteractiveWithRedundantNoOutputTestSpec());
 };
 
-TEST_F(InteractiveRunnerMainTests, Run_Generation_NoOutput) {
+TEST_F(InteractiveRunnerTests, Run_Generation_NoOutput) {
     EXPECT_CALL(generator, generate(_, Property(&GenerationOptions::needsOutput, false)));
     runner.run(argc, argv);
 }
 
-TEST_F(InteractiveRunnerMainTests, Run_Generation_NoOutput_Redundant) {
+TEST_F(InteractiveRunnerTests, Run_Generation_NoOutput_Redundant) {
     EXPECT_CALL(generator, generate(_, Property(&GenerationOptions::needsOutput, false)));
     runnerWithRedundantNoOutput.run(argc, argv);
 }
 
-TEST_F(InteractiveRunnerMainTests, Run_Generation_EvaluatorRegistry_Communicator_Default) {
+TEST_F(InteractiveRunnerTests, Run_Generation_EvaluatorRegistry_Communicator_Default) {
     EXPECT_CALL(evaluatorRegistry, get(
             EvaluationStyle::INTERACTIVE,
             _,
@@ -47,7 +47,7 @@ TEST_F(InteractiveRunnerMainTests, Run_Generation_EvaluatorRegistry_Communicator
     runner.run(argc, argv);
 }
 
-TEST_F(InteractiveRunnerMainTests, Run_Generation_EvaluatorRegistry_Communicator_Args) {
+TEST_F(InteractiveRunnerTests, Run_Generation_EvaluatorRegistry_Communicator_Args) {
     EXPECT_CALL(evaluatorRegistry, get(
             EvaluationStyle::INTERACTIVE,
             _,
@@ -59,7 +59,7 @@ TEST_F(InteractiveRunnerMainTests, Run_Generation_EvaluatorRegistry_Communicator
             nullptr});
 }
 
-TEST_F(InteractiveRunnerMainTests, Run_Grading_EvaluatorRegistry_Communicator_Default) {
+TEST_F(InteractiveRunnerTests, Run_Grading_EvaluatorRegistry_Communicator_Default) {
     EXPECT_CALL(evaluatorRegistry, get(
             EvaluationStyle::INTERACTIVE,
             _,
@@ -71,7 +71,7 @@ TEST_F(InteractiveRunnerMainTests, Run_Grading_EvaluatorRegistry_Communicator_De
             nullptr});
 }
 
-TEST_F(InteractiveRunnerMainTests, Run_Grading_EvaluatorRegistry_Communicator_Args) {
+TEST_F(InteractiveRunnerTests, Run_Grading_EvaluatorRegistry_Communicator_Args) {
     EXPECT_CALL(evaluatorRegistry, get(
             EvaluationStyle::INTERACTIVE,
             _,