Merge pull request #876 from SoprisApps/patch-fix-cli-ava-tests

Fix react-server-cli AVA test issues

packages/react-server-cli/test/commands.js

@@ -67,8 +67,14 @@ fs.readdirSync(fixturesPath).forEach(testName => {
 			}, frequency);
 		});
 
-		if (stdoutIncludes) t.true(stdout.includes(stdoutIncludes), 'stdout includes expected output');
-		if (stderrIncludes) t.true(stderr.includes(stderrIncludes), 'stderr includes expected output');
+		t.true(
+			stderrIncludes ? stderr.includes(stderrIncludes) : stderr === '',
+			'stderr does not include expected output.  Instead, it says: ' + stderr
+		);
+		t.true(
+			stdoutIncludes ? stdout.includes(stdoutIncludes) : stdout === '',
+			'stdout does not include expected output.  Instead, it says: ' + stdout
+		);
 
 		server.kill();
 

packages/react-server-cli/test/fixtures/commands/start-basic/options.json

@@ -1,4 +1,4 @@
 {
-	"args": ["start"],
+	"args": ["start", "--port", "8780", "--js-port", "8781"],
 	"stdoutIncludes": "Started HTML server over HTTP on"
 }

packages/react-server-cli/test/fixtures/commands/start-missing-routes/options.json

@@ -1,4 +1,4 @@
 {
-	"args": ["start"],
+	"args": ["start", "--port", "8782", "--js-port", "8783"],
 	"stderrIncludes": "Failed to load routes file at"
 }

packages/react-server-cli/test/fixtures/commands/start-reactserverrc/options.json

@@ -1,4 +1,4 @@
 {
-	"args": ["start"],
+	"args": ["start", "--port", "8784", "--js-port", "8785"],
 	"stdoutIncludes": "Started HTML server over HTTP on"
 }

packages/react-server-cli/test/fixtures/commands/start-routes-with-error/options.json

@@ -1,4 +1,4 @@
 {
-	"args": ["start", "--routes-file", "routes.js"],
+	"args": ["start", "--routes-file", "routes.js", "--port", "8786", "--js-port", "8787"],
 	"stderrIncludes": "ReferenceError: undeclaredVariable is not defined"
 }