fix(ci): correctly report packages spec failures

Author: filipesilva

package.json

@@ -12,7 +12,7 @@
     "build": "node ./scripts/publish/build.js",
     "build:patch": "node ./scripts/patch.js",
     "build:packages": "for PKG in packages/*; do echo Building $PKG...; tsc -p $PKG; done",
-    "test": "npm run test:packages && npm run test:cli",
+    "test": "npm-run-all -c test:packages test:cli",
     "e2e": "npm run test:e2e",
     "e2e:nightly": "node tests/e2e_runner.js --nightly",
     "mobile_test": "mocha tests/e2e/e2e_workflow.spec.js",

packages/ast-tools/src/route-utils.spec.ts

@@ -293,7 +293,7 @@ export default [\n  { path: 'new-route', component: NewRouteComponent }\n];`);
             `\nexport default [\n` +
             `  { path: 'home', component: HomeComponent,\n` +
             `    children: [\n` +
-            `      { path: 'about/:id', component: AboutComponent } ` +
+            `      { path: 'about/:id', component: AboutComponent }` +
             `\n    ]\n  }\n];`);
         });
     });

packages/ast-tools/src/route-utils.ts

@@ -416,11 +416,11 @@ function addChildPath (parentObject: ts.Node, pathOptions: any, route: string) {
   if (childrenNode.length !== 0) {
     // add to beginning of children array
     pos = childrenNode[0].getChildAt(2).getChildAt(1).pos; // open bracket
-    newContent = `\n${spaces}${content}, `;
+    newContent = `\n${spaces}${content},`;
   } else {
     // no children array, add one
     pos = parentObject.getChildAt(2).pos; // close brace
-    newContent = `,\n${spaces.substring(2)}children: [\n${spaces}${content} ` +
+    newContent = `,\n${spaces.substring(2)}children: [\n${spaces}${content}` +
                  `\n${spaces.substring(2)}]\n${spaces.substring(5)}`;
   }
   return {newContent: newContent, pos: pos};

scripts/run-packages-spec.js

@@ -14,6 +14,8 @@ const projectBaseDir = path.join(__dirname, '../packages');
 const jasmine = new Jasmine({ projectBaseDir: projectBaseDir });
 jasmine.loadConfig({});
 jasmine.addReporter(new JasmineSpecReporter());
+// Manually set exit code (needed with custom reporters)
+jasmine.onComplete((success) => process.exitCode = !success);
 
 // Run the tests.
 const allTests =