Fix organization command if not logged in

Also small tweaks to prints and tests

Author: Irene

.gitignore

@@ -28,4 +28,7 @@ dependency-reduced-pom.xml
 # our tmc-cli log files
 /logs/
 
-tmc-langs.logl
+tmc-langs.log
+/.classpath
+/.project
+/.settings

src/main/java/fi/helsinki/cs/tmc/cli/backend/SettingsIo.java

@@ -77,7 +77,7 @@ public static void saveCurrentSettingsToAccountList(Settings settings) {
             Account account = i.next();
             if (account.getUsername().equals(settings.getAccount().getUsername())) {
                 if (!account.getUsername().isPresent()) {
-                    logger.error("Saveable account doesn't exist");
+                    logger.error("Savable account doesn't exist");
                     return;
                 }
                 deletables.add(account);

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

@@ -62,6 +62,7 @@ public void run(CliContext context, CommandLine args) {
                         ColorUtil.colorString(
                                 "Server " + settings.getServerAddress(), Color.YELLOW));
             }
+            io.println(settings.toString());
 
             printCourseList(settings);
             isFirst = false;

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

@@ -122,7 +122,7 @@ private boolean getSendDiagnosticsAnswer(String username, String server) {
             return this.ctx.getSettings().getSendDiagnostics();
         }
         while (true) {
-            String sendDiagnostics = io.readLine("Do you want to send crash reports and diagnostics for client development? (y/n)");
+            String sendDiagnostics = io.readLine("Do you want to send crash reports and diagnostics for client development? (y/n) ");
             if (sendDiagnostics.trim().toLowerCase().startsWith("y")) {
                 return true;
             } else if (sendDiagnostics.trim().toLowerCase().startsWith("n")) {

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

@@ -32,6 +32,9 @@ public void getOptions(Options options) {
 
     @Override
     public void run(CliContext ctx, CommandLine args) {
+        if (!ctx.loadBackend()) {
+            return;
+        }
         Optional<Organization> organization = chooseOrganization(ctx, args);
         this.ctx.getSettings().setOrganization(organization);
         SettingsIo.saveCurrentSettingsToAccountList(this.ctx.getSettings());
@@ -61,6 +64,7 @@ private void printOrganizations(List<Organization> organizations, boolean pager)
                         others.add(o);
                     }
                 });
+        io.println("Available Organizations:");
         io.println();
         pinned.stream().forEach(o -> printFormattedOrganization(o));
         io.println("----------");

src/test/java/fi/helsinki/cs/tmc/cli/backend/SettingsIoTest.java

@@ -132,5 +132,6 @@ public void saveCurrentSettingsOverridesOldAccountWithSameUsername() {
         SettingsIo.saveCurrentSettingsToAccountList(settings);
         AccountList newList = SettingsIo.loadAccountList();
         assertEquals("new", newList.getAccount("username", STUB_SERVER).getOrganization().get().getName());
+        SettingsIo.delete();
     }
 }