Merge branch 'master' into courseinfoupdating-matike

Author: matike

HACKING.md

@@ -31,3 +31,6 @@ public class ExampleCommand extends AbstractCommand {
 
 ## Architecture
 ...
+
+## Debugging
+### Logging

README.md

@@ -46,9 +46,9 @@ Now that you've installed tmc-cli, you can view all available commands by runnin
 Once installation is complete, you can log in using `tmc login`. This saves your TMC login information to a configuration file in ~/.config/tmc-cli/ (or %LOCALAPPDATA% on Windows) - you will only have to log in once.
 ```
 ~ $ tmc login
-username: my-username
-password:
 server address:
+username:
+password:
 Login successful.
 ```
 

pom.xml

@@ -3,7 +3,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>fi.helsinki.cs.tmc.cli</groupId>
     <artifactId>tmc-cli</artifactId>
-    <version>0.5.2</version>
+    <version>0.6.0</version>
     <packaging>jar</packaging>
     <properties>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>

src/main/java/fi/helsinki/cs/tmc/cli/command/LoginCommand.java

@@ -36,10 +36,10 @@ public void getOptions(Options options) {
     @Override
     public void run(CommandLine args, Io io) {
         this.io = io;
+        String serverAddress = getLoginInfo(args, "s", "server address: ");
         String username = getLoginInfo(args, "u", "username: ");
         String password = getLoginInfo(args, "p", "password: ");
-        String serverAddress = getLoginInfo(args, "s", "server address: ");
-
+        
         Settings settings = new Settings(serverAddress, username, password);
         if (loginPossible(settings) && saveLoginSettings(settings)) {
             io.println("Login successful.");

src/main/java/fi/helsinki/cs/tmc/cli/command/RunTestsCommand.java

@@ -107,9 +107,14 @@ public void run(CommandLine args, Io io) {
                 passed += ResultPrinter.passedTests(runResult.testResults);
                 exercise.setAttempted(true);
                 if (runResult.status == PASSED && !exercise.isCompleted()) {
+                    // add exercise to locally tested exercises
                     if (!info.getLocalCompletedExercises().contains(exercise.getName())) {
                         info.getLocalCompletedExercises().add(exercise.getName());
                     }
+                } else {
+                    if (info.getLocalCompletedExercises().contains(exercise.getName())) {
+                        info.getLocalCompletedExercises().remove(exercise.getName());
+                    }
                 }
             }
             CourseInfoIo.save(info, workDir.getConfigFile());

src/main/java/fi/helsinki/cs/tmc/cli/command/SubmitCommand.java

@@ -40,7 +40,8 @@ public class SubmitCommand extends AbstractCommand {
     public void getOptions(Options options) {
         options.addOption("a", "all", false, "Show all test results");
         options.addOption("d", "details", false, "Show detailed error message");
-        options.addOption("c", "completed", false, "Only exercises that have passed all tests");
+        options.addOption("c", "completed", false,
+                "Filter out exercises that haven't been locally tested");
     }
 
     @Override
@@ -68,6 +69,7 @@ public void run(CommandLine args, Io io) {
 
         List<String> exerciseNames;
         if (args.hasOption("c")) {
+            workDir.addPath(workDir.getCourseDirectory());
             exerciseNames = workDir.getExerciseNames(true, true, false);
         } else {
             exerciseNames = workDir.getExerciseNames();

src/main/java/fi/helsinki/cs/tmc/cli/command/core/CommandAnnotationProcessor.java

@@ -30,38 +30,32 @@ public class CommandAnnotationProcessor extends AbstractProcessor {
     private static final Logger logger = LoggerFactory.getLogger(CommandAnnotationProcessor.class);
 
     private static final String CLASS_NAME = "CommandList";
-    private static final String PACKAGE_NAME = "fi.helsinki.cs.tmc.cli.command";
+    private static final String PACKAGE_NAME = "fi.helsinki.cs.tmc.cli.command.core";
     private static final String TAB = "    ";
 
-    private ProcessingEnvironment processingEnv;
-
-    @Override
-    public void init(ProcessingEnvironment processingEnv) {
-        this.processingEnv = processingEnv;
-    }
-
     private void generateSourceFile(Map<String, String> map) throws IOException {
         JavaFileObject jfo = processingEnv.getFiler().createSourceFile(
-                PACKAGE_NAME + ".core." + CLASS_NAME);
+                PACKAGE_NAME + "." + CLASS_NAME);
 
         try (Writer writer = jfo.openWriter()) {
             BufferedWriter bwriter = new BufferedWriter(writer);
-            bwriter.append("package ");
-            bwriter.append(PACKAGE_NAME);
-            bwriter.append(".core;\n\n");
+            bwriter.append("package " + PACKAGE_NAME + ";\n\n");
+
+            // import the command classes
             bwriter.append("//CHECKSTYLE:OFF\n");
-            bwriter.append("import " + PACKAGE_NAME + ".core.CommandFactory;\n\n");
             for (Entry<String, String> entry : map.entrySet()) {
                 bwriter.append("import " + entry.getValue() + ";\n");
             }
             bwriter.append("//CHECKSTYLE:ON\n");
+
             bwriter.append("\npublic class " + CLASS_NAME + " {\n");
             bwriter.append(TAB + "static {\n");
             for (Entry<String, String> entry : map.entrySet()) {
                 String[] parts = entry.getValue().split("\\.");
                 if (parts.length == 0) {
                     continue;
                 }
+                // print out the lines that add the commands to the command factory.
                 String className = parts[parts.length - 1];
                 bwriter.append(TAB + TAB + "CommandFactory.addCommand(\""
                         + entry.getKey() + "\", "
@@ -76,15 +70,15 @@ private void generateSourceFile(Map<String, String> map) throws IOException {
     @Override
     public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment roundEnv) {
         Map<String, String> map = new HashMap<>();
-        Messager messager = processingEnv.getMessager();
 
         for (Element elem : roundEnv.getElementsAnnotatedWith(Command.class)) {
             if (elem.getKind() != ElementKind.CLASS) {
-                continue;
+                logger.warn("Element with command annotation is not class: " + elem.toString());
+                return false;
             }
             Command command = elem.getAnnotation(Command.class);
-            messager.printMessage(Diagnostic.Kind.NOTE, elem.toString());
-            messager.printMessage(Diagnostic.Kind.NOTE, elem.getClass().getCanonicalName());
+            logger.info("element with annotation: " + elem.toString());
+            logger.info("element name with annotation: " + elem.getClass().getCanonicalName());
 
             TypeElement classElement = (TypeElement) elem;
             map.put(command.name(), processingEnv.getElementUtils()
@@ -94,7 +88,7 @@ public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment
         try {
             generateSourceFile(map);
         } catch (IOException ex) {
-            messager.printMessage(Diagnostic.Kind.NOTE, "Failed to create source file." + ex);
+            logger.warn("Failed to create source file." + ex);
         }
         return true;
     }

src/main/java/fi/helsinki/cs/tmc/cli/io/ResultPrinter.java

@@ -5,16 +5,16 @@
 import fi.helsinki.cs.tmc.langs.domain.SpecialLogs;
 import fi.helsinki.cs.tmc.langs.domain.TestResult;
 
+import java.util.Arrays;
 import java.util.List;
 
 public class ResultPrinter {
 
     private static final String COMPILE_ERROR_MESSAGE
             = Color.colorString("Failed to compile project", Color.AnsiColor.ANSI_PURPLE);
-    private static final String FAIL = Color.colorString("Failed: ", Color.AnsiColor.ANSI_RED);
-    private static final String PASS = Color.colorString("Passed: ", Color.AnsiColor.ANSI_GREEN);
-    private static final String TAB = "        ";
-    private static final char LF = '\n';
+    private static final String FAIL_MESSAGE = "Failed: ";
+    private static final String PASS_MESSAGE = "Passed: ";
+    private final String tab;
 
     private final Io io;
 
@@ -27,6 +27,8 @@ public ResultPrinter(Io io, boolean showDetails, boolean showPassed) {
         this.io = io;
         this.showDetails = showDetails;
         this.showPassed = showPassed;
+
+        this.tab = createPaddingString(PASS_MESSAGE.length());
     }
 
     public boolean isShowDetails() {
@@ -155,9 +157,9 @@ public static int passedTests(List<TestResult> testResults) {
     private void printTestResults(List<TestResult> testResults) {
         for (TestResult testResult : testResults) {
             if (!testResult.isSuccessful()) {
-                io.println(createFailMessage(testResult));
+                printFailMessage(testResult);
             } else if (showPassed) {
-                io.println(createPassMessage(testResult));
+                printPassMessage(testResult);
             }
         }
         io.println("Test results: "
@@ -166,37 +168,45 @@ private void printTestResults(List<TestResult> testResults) {
 
     }
 
-    private String createFailMessage(TestResult testResult) {
-        StringBuilder sb = new StringBuilder();
-        sb.append(FAIL).append(testResult.getName()).append(LF);
-        sb.append(TAB).append(testResult.getMessage()).append(LF);
+    private void printFailMessage(TestResult testResult) {
+        io.print(Color.colorString(FAIL_MESSAGE, Color.AnsiColor.ANSI_RED));
+        io.println(testResult.getName());
+        io.println(this.tab + testResult.getMessage());
 
         if (showDetails) {
-            String details = listToString(testResult.getDetailedMessage(), LF);
+            String details = listToString(testResult.getDetailedMessage());
             if (details != null) {
-                sb.append(LF).append("Detailed message:").append(LF).append(details);
+                io.println("\nDetailed message:");
+                io.println(details);
             }
-            String exception = listToString(testResult.getException(), LF);
+
+            String exception = listToString(testResult.getException());
             if (exception != null) {
-                sb.append(LF).append("Exception:").append(LF).append(exception);
+                io.println("\nException:");
+                io.println(exception);
             }
         }
-        return sb.toString();
     }
 
-    private String createPassMessage(TestResult testResult) {
-        return PASS + testResult.getName() + LF;
+    private void printPassMessage(TestResult testResult) {
+        io.print(Color.colorString(PASS_MESSAGE, Color.AnsiColor.ANSI_GREEN));
+        io.println(testResult.getName());
     }
 
-    private String listToString(List<String> strings, char separator) {
+    private String listToString(List<String> strings) {
         if (strings == null || strings.isEmpty()) {
             return null;
         }
         StringBuilder sb = new StringBuilder();
         for (String string : strings) {
-            sb.append(string).append(separator);
+            sb.append(string).append("\n");
         }
         return sb.toString();
     }
 
-}
+    private String createPaddingString(int size) {
+        char[] charArray = new char[size];
+        Arrays.fill(charArray, ' ');
+        return new String(charArray);
+    }
+}

src/test/java/fi/helsinki/cs/tmc/cli/command/RunTestsCommandTest.java

@@ -3,7 +3,6 @@
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
-
 import static org.mockito.Matchers.anyObject;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.mock;
@@ -133,5 +132,4 @@ public void worksInCourseDirectoryIfExerciseIsGiven() {
         app.run(args);
         assertTrue(io.out().contains("Testing: " + EXERCISE1_NAME));
     }
-
 }