Added scope check for LK id

Author: RobertReaves

specs/workflows/dev.spec.js

@@ -1,21 +1,32 @@
-const resetWorkflow = require("../../src/workflows/dev");
-const run = require("../../src/workflows/steps");
-
 describe("dev workflow", () => {
-	it("should have all of the required steps", () => {
-		expect(resetWorkflow).toEqual([
-			run.fetchUpstream,
-			run.gitCreateBranchOrigin,
-			run.checkoutBaseBranch,
-			run.rebaseUpstreamBaseBranch,
-			run.checkoutWorkingBranch,
-			run.gitCreateBranchUpstream,
-			run.githubUpstream,
-			run.githubOrigin,
-			run.updatePullRequestTitle,
-			run.addLKId,
-			run.updatePullRequestBody,
-			run.createGithubPullRequestAganistBranch
-		]);
+	let resetWorkflow;
+	describe("has LK scope", () => {
+		beforeEach(() => {
+			jest.resetModules();
+			jest.doMock("../../src/utils", () => ({
+				hasLkScope: jest.fn(() => true)
+			}));
+
+			resetWorkflow = require("../../src/workflows/dev");
+		});
+
+		it("should have all of the required steps", () => {
+			expect(resetWorkflow.length).toEqual(12);
+		});
+	});
+
+	describe("doesn't have LK scope", () => {
+		beforeEach(() => {
+			jest.resetModules();
+			jest.doMock("../../src/utils", () => ({
+				hasLkScope: jest.fn(() => false)
+			}));
+
+			resetWorkflow = require("../../src/workflows/dev");
+		});
+
+		it("should have all of the required steps", () => {
+			expect(resetWorkflow.length).toEqual(11);
+		});
 	});
 });

src/workflows/dev.js

@@ -1,4 +1,5 @@
 const run = require("./steps/index");
+const { hasLkScope } = require("../utils");
 
 module.exports = [
 	run.fetchUpstream,
@@ -10,7 +11,7 @@ module.exports = [
 	run.githubUpstream,
 	run.githubOrigin,
 	run.updatePullRequestTitle,
-	run.addLKId,
+	hasLkScope() ? run.addLKId : null,
 	run.updatePullRequestBody,
 	run.createGithubPullRequestAganistBranch
-];
+].filter(x => x);