Add more verbose error messages to frame creation from json. closes kriskbx#56

Author: kriskbx

src/gtt-sync.js

@@ -4,6 +4,7 @@ const program = require('commander');
 const Config = require('./include/file-config');
 const Cli = require('./include/cli');
 const Tasks = require('./include/tasks');
+const Owner = require('./models/owner');
 
 program
     .option('-p --proxy <proxy>', 'use a proxy server with the given url')
@@ -16,10 +17,13 @@ Cli.verbose = program.verbose;
 let config = new Config(process.cwd())
         .set('proxy', program.proxy)
         .set('_checkToken', program.check_token),
-    tasks = new Tasks(config);
+    tasks = new Tasks(config),
+    owner = new Owner(config);
 
 tasks.syncInit()
     .then(() => tasks.sync.frames.length === 0 ? process.exit(0) : null)
+    .then(() => owner.authorized())
+    .catch(e => Cli.x(`Invalid access token!`, e))
     .then(() => {
         Cli.bar(`${Cli.fetch}  Fetching or creating issues & merge requests...`, tasks.sync.frames.length);
         return tasks.syncResolve(Cli.advance);

src/models/frame.js

@@ -34,6 +34,7 @@ class frame {
         frame._stop = json.stop;
         frame.notes = json.notes;
         frame.timezone = json.timezone;
+        frame.validate();
 
         return frame;
     }
@@ -56,6 +57,18 @@ class frame {
         return this;
     }
 
+    validate() {
+        moment.suppressDeprecationWarnings = true;
+
+        if(!moment(this._start).isValid())
+            throw `Error: Start date is not in a valid ISO date format!`;
+
+        if(!moment(this._stop).isValid())
+            throw `Error: Stop date is not in a valid ISO date format!`;
+
+        moment.suppressDeprecationWarnings = false;
+    }
+
     _getCurrentDate() {
         if(this.timezone)
             return moment().tz(this.timezone).format();

src/models/frameCollection.js

@@ -12,7 +12,8 @@ class frameCollection extends Base {
                     try {
                         return Frame.fromFile(this.config, Fs.join(this.config.frameDir, file));
                     } catch (e) {
-                        return false;
+                        console.log(e);
+                        throw `Error parsing frame file: ${file}`
                     }
                 })
                 .filter(frame => frame);