Lazy load the course info object in context.

Author: Aleksi Salmela

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

@@ -22,9 +22,10 @@ public class CliContext {
     private Application application;
     private TmcCore tmcCore;
     private Settings settings;
+
+    /* cached values */
     private boolean hasLogin;
     private CourseInfo courseInfo;
-
     private HashMap<String, String> properties;
     private final boolean inTest;
 
@@ -60,6 +61,11 @@ public CliContext(Io io, WorkDir workDir, TmcCore core) {
         this.courseInfo = null;
     }
 
+    /*TODO create reset method for removing all cached data that is called
+     * when working directory is changed or by users demand also use it in
+     * constructor.
+     */
+
     protected void setApp(Application app) {
         this.application = app;
     }
@@ -109,6 +115,24 @@ public Boolean saveProperties() {
         return SettingsIo.saveProperties(properties);
     }
 
+    public CourseInfo getCourseInfo() {
+        if (courseInfo != null) {
+            return courseInfo;
+        }
+        if (workDir.getConfigFile() == null) {
+            return null;
+        }
+        courseInfo = CourseInfoIo.load(workDir.getConfigFile());
+        if (courseInfo == null) {
+            io.println("Course configuration file "
+                    + workDir.getConfigFile().toString()
+                    + "is invalid.");
+            //TODO add a way to rewrite the course config file.
+            return null;
+        }
+        return this.courseInfo;
+    }
+
     // Method is used to help testing
     public void setTmcCore(TmcCore tmcCore) {
         this.tmcCore = tmcCore;
@@ -168,23 +192,18 @@ private void createTmcCore(Settings settings) {
     }
 
     private boolean createTmcCore() {
-        Settings settings;
+        Settings cachedSettings = null;
 
         if (workDir.getConfigFile() != null) {
             // If we're in a course directory, we load settings matching the course
             // Otherwise we just load the last used settings
-            courseInfo = CourseInfoIo.load(workDir.getConfigFile());
-            if (courseInfo == null) {
-                io.println("Course configuration file "
-                        + workDir.getConfigFile().toString()
-                        + "is invalid.");
-                //TODO add a way to rewrite the course config file.
-                return false;
+            courseInfo = getCourseInfo();
+            if (courseInfo != null) {
+                cachedSettings = SettingsIo.load(courseInfo.getUsername(),
+                        courseInfo.getServerAddress());
             }
-            settings = SettingsIo.load(courseInfo.getUsername(),
-                    courseInfo.getServerAddress());
         } else {
-            settings = SettingsIo.load();
+            cachedSettings = SettingsIo.load();
         }
 
         hasLogin = true;
@@ -193,7 +212,7 @@ private boolean createTmcCore() {
             settings = new Settings();
         }
 
-        createTmcCore(settings);
+        createTmcCore(cachedSettings);
         return true;
     }
 }

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

@@ -44,7 +44,13 @@ public void run(CommandLine args, Io io) {
         this.ctx = getContext();
         workDir = ctx.getWorkDir();
 
-        if (!args.hasOption("i")) {
+        boolean fetchFromInternet = args.hasOption("i");
+
+        if (! ctx.loadBackend()) {
+            return;
+        }
+
+        if (!fetchFromInternet) {
             printLocalCourseOrExercise(args);
             return;
         }
@@ -55,9 +61,6 @@ public void run(CommandLine args, Io io) {
             return;
         }
 
-        if (! ctx.loadBackend()) {
-            return;
-        }
         course = TmcUtil.findCourse(ctx, stringArgs[0]);
         if (course == null) {
             io.println("The course " + stringArgs[0] + " doesn't exist on this server.");
@@ -67,8 +70,8 @@ public void run(CommandLine args, Io io) {
     }
 
     private void printLocalCourseOrExercise(CommandLine args) {
-        if (workDir.getConfigFile() != null) {
-            info = CourseInfoIo.load(workDir.getConfigFile());
+        info = ctx.getCourseInfo();
+        if (info != null) {
             course = info.getCourse();
             printCourseOrExercise(args);
         } else {

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

@@ -5,7 +5,6 @@
 import fi.helsinki.cs.tmc.cli.io.Io;
 import fi.helsinki.cs.tmc.cli.tmcstuff.TmcUtil;
 
-import fi.helsinki.cs.tmc.core.TmcCore;
 import fi.helsinki.cs.tmc.core.domain.Course;
 
 import org.apache.commons.cli.CommandLine;

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

@@ -35,7 +35,7 @@ public void getOptions(Options options) {
     public void run(CommandLine args, Io io) {
         this.ctx = getContext();
         this.io = io;
-        
+
         String courseName = getCourseName(args);
         if (courseName == null) {
             return;
@@ -77,10 +77,7 @@ private String getCourseNameFromCurrentDirectory() {
     private CourseInfo getCourseInfoFromCurrentDirectory() {
         WorkDir workDir = ctx.getWorkDir();
         workDir.addPath();
-        if (workDir.getConfigFile() != null) {
-            return CourseInfoIo.load(workDir.getConfigFile());
-        }
-        return null;
+        return ctx.getCourseInfo();
     }
 
     private List<Exercise> getExercisesFromServer(String courseName) {

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

@@ -88,8 +88,8 @@ public void run(CommandLine args, Io io) {
         }
 
         String exerciseName = exercisenames.get(0);
-        CourseInfo courseinfo = CourseInfoIo.load(ctx.getWorkDir().getConfigFile());
-        Exercise exercise = courseinfo.getExercise(exerciseName);
+        CourseInfo info = ctx.getCourseInfo();
+        Exercise exercise = info.getExercise(exerciseName);
         URI uri = TmcUtil.sendPaste(ctx, exercise, message);
         if (uri == null) {
             io.println("Unable to send the paste");

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

@@ -67,7 +67,7 @@ public void run(CommandLine args, Io io) {
             io.println("You have to be in a course directory to run tests");
             return;
         }
-        CourseInfo info = CourseInfoIo.load(workDir.getConfigFile());
+        CourseInfo info = ctx.getCourseInfo();
 
         ResultPrinter resultPrinter
                 = new ResultPrinter(io, this.showDetails, this.showPassed);

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

@@ -88,7 +88,7 @@ public void run(CommandLine args, Io io) {
             return;
         }
 
-        CourseInfo info = CourseInfoIo.load(workDir.getConfigFile());
+        CourseInfo info = ctx.getCourseInfo();
         Course currentCourse = info.getCourse();
         if (currentCourse == null) {
             return;

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

@@ -53,7 +53,7 @@ public void run(CommandLine args, Io io) {
             return;
         }
 
-        CourseInfo info = CourseInfoIo.load(workDir.getConfigFile());
+        CourseInfo info = ctx.getCourseInfo();
         updateExercises(info, workDir.getConfigFile());
     }
 

src/main/java/fi/helsinki/cs/tmc/cli/tmcstuff/SettingsHolder.java

@@ -1,7 +1,6 @@
 package fi.helsinki.cs.tmc.cli.tmcstuff;
 
 import java.util.ArrayList;
-import java.util.HashMap;
 
 /**
  * This is a class for storing all different login settings as a single array.

src/main/java/fi/helsinki/cs/tmc/cli/tmcstuff/WorkDir.java

@@ -81,6 +81,7 @@ public List<String> getExerciseNames(
                 return new ArrayList<>();
             }
         }
+        /*TODO somehow use the ctx.getCourseInfo */
         CourseInfo courseinfo = CourseInfoIo.load(getConfigFile());
         if (courseinfo == null) {
             return new ArrayList<>();