Check exit codes in integration tests (#2148)

tests_integration/__tests__/debug-check.js

@@ -10,6 +10,7 @@ test("doesn't crash when --debug-check is passed", () => {
 
   expect(result.stdout).toEqual("issue1890.js\n");
   expect(result.stderr).toEqual("");
+  expect(result.status).toEqual(0);
 });
 
 test("checks stdin with --debug-check", () => {
@@ -19,4 +20,5 @@ test("checks stdin with --debug-check", () => {
 
   expect(result.stdout).toEqual("(stdin)\n");
   expect(result.stderr).toEqual("");
+  expect(result.status).toEqual(0);
 });

tests_integration/__tests__/parser-api.js

@@ -36,4 +36,5 @@ test("allows passing a string to resolve a parser", () => {
     "./custom-rename-parser"
   ]);
   expect(output.stdout).toMatchSnapshot();
+  expect(output.status).toEqual(0);
 });

tests_integration/__tests__/with-node-modules.js

@@ -6,6 +6,7 @@ test("ignores node_modules by default", () => {
   const result = runPrettier("cli/with-node-modules", ["**/*.js", "-l"]);
 
   expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(1);
 });
 
 test("doesn't ignore node_modules with --with-node-modules flag", () => {
@@ -16,6 +17,7 @@ test("doesn't ignore node_modules with --with-node-modules flag", () => {
   ]);
 
   expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(1);
 });
 
 test("ignores node_modules by default for file list", () => {
@@ -27,6 +29,7 @@ test("ignores node_modules by default for file list", () => {
   ]);
 
   expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(1);
 });
 
 test("doesn't ignore node_modules with --with-node-modules flag for file list", () => {
@@ -39,4 +42,5 @@ test("doesn't ignore node_modules with --with-node-modules flag for file list",
   ]);
 
   expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(1);
 });

tests_integration/__tests__/with-parser-inference.js

@@ -6,6 +6,7 @@ test("infers postcss parser", () => {
   const result = runPrettier("cli/with-parser-inference", ["*"]);
 
   expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(0);
 });
 
 test("infers postcss parser with --list-different", () => {
@@ -15,4 +16,5 @@ test("infers postcss parser with --list-different", () => {
   ]);
 
   expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(0);
 });

tests_integration/__tests__/with-shebang.js

@@ -6,4 +6,5 @@ test("preserves shebang", () => {
   const result = runPrettier("cli/with-shebang", ["issue1890.js"]);
 
   expect(result.stdout).toMatchSnapshot();
+  expect(result.status).toEqual(0);
 });