fix: check missing config in Session ctor (#426)

johnfrench3 · May 30, 2020 · db658a0 · db658a0
1 parent 4b1e1ab
commit db658a0
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/components/git/land.js b/components/git/land.js
@@ -129,9 +129,7 @@ module.exports = {
 
 async function main(state, argv, cli, req, dir) {
   let session = new LandingSession(cli, req, dir);
-  if (session.warnForMissing()) {
-    return;
-  }
+
   if (state !== AMEND && state !== CONTINUE && session.warnForWrongBranch()) {
     return;
   }

diff --git a/lib/session.js b/lib/session.js
@@ -25,6 +25,10 @@ class Session {
 
     const { upstream, owner, repo } = this;
 
+    if (this.warnForMissing()) {
+      throw new Error('Failed to create new session');
+    }
+
     const upstreamHref = runSync('git', [
       'config', '--get',
       `remote.${upstream}.url`]).trim();

diff --git a/lib/sync_session.js b/lib/sync_session.js
@@ -9,9 +9,6 @@ class SyncSession extends Session {
   }
 
   async sync() {
-    if (this.warnForMissing()) {
-      return;
-    }
     if (this.warnForWrongBranch()) {
       return;
     }