Merge 5bf63f2 into 8988c4f

src/index.ts

@@ -52,6 +52,7 @@ export interface ParseResult {
   changedDataSources: Array<Changed>;
 }
 
+const NO_CHANGES_STRING = '\nNo changes. Infrastructure is up-to-date.\n';
 const CONTENT_START_STRING = '\nTerraform will perform the following actions:\n';
 const CONTENT_END_STRING = '\nPlan:';
 const OLD_NEW_SEPARATOR = ' => ';
@@ -356,6 +357,11 @@ export function parseStdout (logOutput: string): ParseResult {
 
   let lastChange = null;
 
+  if (logOutput.includes(NO_CHANGES_STRING)) {
+    // no changes to parse...
+    return result;
+  }
+
   const startPos = findParseableContentStartPos(logOutput);
   if (startPos === -1) {
     result.errors.push({

test/unit/data/13-no-changes.expected.json

@@ -0,0 +1,5 @@
+{
+  "errors": [],
+  "changedResources": [],
+  "changedDataSources": []
+}

test/unit/data/13-no-changes.stdout.txt

@@ -0,0 +1,12 @@
+Refreshing Terraform state in-memory prior to plan...
+The refreshed state will be used to calculate this plan, but will not be
+persisted to local or remote state storage.
+
+
+------------------------------------------------------------------------
+
+No changes. Infrastructure is up-to-date.
+
+This means that Terraform did not detect any differences between your
+configuration and real physical resources that exist. As a result, no
+actions need to be performed.

test/unit/terraform-plan-parser.test.ts

@@ -74,3 +74,7 @@ test('should handle tainted resources', async (t) => {
 test('should handle modules', async (t) => {
   return runTest('12-modules', t);
 });
+
+test('should handle no changes', async (t) => {
+  return runTest('13-no-changes', t);
+});