Fix migrating from /mooc -server

Author: Irene

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

@@ -21,7 +21,7 @@ public class Account {
     private boolean sendDiagnostics;
     private boolean sendAnalytics;
     private boolean sendDetailedAnalytics;
-    private static final String DEFAULT_SERVER =  "https://tmc.mooc.fi/staging";
+    private static final String DEFAULT_SERVER =  "https://tmc.mooc.fi";
 
     // for gson
     public Account() {

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

@@ -64,10 +64,14 @@ public void setAccount(CliContext context, Account account) {
 
     private boolean migrateAccount(CliContext context, Account account) throws Exception {
         Io io = context.getIo();
-        io.println("TMC cli has been updated. Please provide the following information.");
+        // the old server doesn't have any organizations
+        if (account.getServerAddress().contains("tmc.mooc.fi/mooc")) {
+            return false;
+        }
         TmcServerAddressNormalizer normalizer = new TmcServerAddressNormalizer();
         normalizer.normalize();
         normalizer.selectOrganizationAndCourse();
+        io.println("TMC cli has been updated. Please provide the following information.");
         if (!context.getSettings().getOrganization().isPresent()) {
             OrganizationCommand organizationCommand = new OrganizationCommand();
             Optional<Organization> organization = organizationCommand.chooseOrganization(context, null);

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

@@ -26,7 +26,6 @@ public class ConfigCommand extends AbstractCommand {
     private static final Set<String> PROGRESS_BAR_COLORS = new HashSet<>(Arrays.asList("black", "red", "green", "blue", "yellow", "blue", "purple", "cyan", "white", "none"));
     private static final String sendAnalyticsKey = "send-analytics";
     private static final String serverAddressKey = "server-address";
-    private static final String updateDateKey = "update-date";
     private static final String testResultRightKey = "testresults-right";
     private static final String testResultLeftKey = "testresults-left";
     private static final String progressBarLeftKey = "progressbar-left";
@@ -84,10 +83,6 @@ public void setter(String value) throws BadValueTypeException {
                 }
                 context.getSettings().setServerAddress(addr);
                 normalizeServerAddress();
-                if (addr.matches("^https?://tmc.mooc.fi/mooc")) {
-                    io.println("Please note that the server http://tmc.mooc.fi/mooc is no longer supported.");
-                }
-                io.println("All courses are now hosted at https://tmc.mooc.fi. We do not advise changing the server address.");
                 SettingsIo.saveCurrentSettingsToAccountList(context.getSettings());
             }
         });
@@ -289,6 +284,16 @@ private void setProperties(String[] arguments) {
                 continue;
             }
             String oldValue = properties.get(parts[0]);
+            if (parts[0].equals(serverAddressKey)) {
+                io.println("All courses are now hosted at https://tmc.mooc.fi. We do not advise changing the server address.");
+                if (parts[0].contains("tmc.mooc.fi/mooc")) {
+                    io.println("The server https://tmc.mooc.fi/mooc is no longer supported by this client.\n" +
+                            "If you'd like to do the migrated courses, you'll have to create an account on the new server.\n" +
+                            "Choose the MOOC organization when logging in.\n\n" +
+                            "For more information, check the course materials on mooc.fi.");
+                    return;
+                }
+            }
             if (io.readConfirmation(" Set " + parts[0] + " to \"" + parts[1] + "\"?", true)) {
                 if (!saveValue(parts[0], parts[1])) {
                     continue;

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

@@ -100,7 +100,7 @@ public Optional<Organization> chooseOrganization(CliContext ctx, CommandLine arg
             oneLine = args.hasOption("o");
         }
         List<Organization> organizations = listOrganizations();
-        if (organizations == null) {
+        if (organizations == null || organizations.isEmpty()) {
             io.errorln("Failed to fetch organizations from server.");
             return Optional.absent();
         }

src/main/java/fi/helsinki/cs/tmc/cli/core/CliContext.java

@@ -224,6 +224,15 @@ public void loadUserInformation(boolean useCache) {
             cachedAccount = list.getAccount();
         }
         hasLogin = cachedAccount != null;
+        if (hasLogin && cachedAccount.getServerAddress().contains("tmc.mooc.fi/mooc")) {
+            io.println("The server https://tmc.mooc.fi/mooc is no longer supported by this client.\n" +
+                    "All the courses have been migrated to our main server https://tmc.mooc.fi.\n" +
+                    "If you'd like to do the migrated courses, you have to create a new account on the new server.\n" +
+                    "Choose the MOOC organization when logging in.\n\n" +
+                    "For more information, check the course materials on mooc.fi.");
+            SettingsIo.delete();
+            return;
+        }
         if (useCache) {
             this.settings.setAccount(this, cachedAccount);
         }