Remove unused spec reporter

Author: devversion

package.json

@@ -109,10 +109,8 @@
     "karma-coverage": "^1.1.1",
     "karma-firefox-launcher": "^1.0.1",
     "karma-jasmine": "^1.1.0",
-    "karma-parallel": "^0.2.9",
     "karma-sauce-launcher": "^1.2.0",
     "karma-sourcemap-loader": "^0.3.7",
-    "karma-spec-reporter": "^0.0.32",
     "madge": "^2.2.0",
     "magic-string": "^0.22.4",
     "minimatch": "^3.0.4",

test/karma.conf.js

@@ -14,8 +14,6 @@ module.exports = (config) => {
       require('karma-firefox-launcher'),
       require('karma-sourcemap-loader'),
       require('karma-coverage'),
-      require('karma-spec-reporter'),
-      require('karma-parallel'),
     ],
     files: [
       {pattern: 'node_modules/core-js/client/core.js', included: true, watched: false},
@@ -51,7 +49,7 @@ module.exports = (config) => {
       'dist/packages/**/*.js': ['sourcemap']
     },
 
-    reporters: ['dots', 'spec'],
+    reporters: ['dots'],
     autoWatch: false,
 
     coverageReporter: {
@@ -60,15 +58,6 @@ module.exports = (config) => {
       subdir: '.'
     },
 
-    // Configuration for the advanced karma spec reporter. By default we only want to use that
-    // reporter to display the test that failed. To do this, we have to suppress tests that pass.
-    specReporter: {
-      maxLogLines: Infinity, // Log out the entire stack trace on errors and failures.
-      suppressSkipped: true,
-      suppressPassed: true,
-      showSpecTiming: true,
-    },
-
     sauceLabs: {
       testName: 'material2',
       startConnect: false,
@@ -103,8 +92,7 @@ module.exports = (config) => {
 
     client: {
       jasmine: {
-        // TODO(jelbourn): re-enable random test order once we can de-flake existing issues.
-        random: false
+        random: true
       }
     },
   });
@@ -135,6 +123,8 @@ module.exports = (config) => {
       throw new Error(`Platform "${platform}" unknown, but Travis specified. Exiting.`);
     }
 
+    // To guarantee a better stability for tests running on external browsers, we disable
+    // concurrency. Stability is compared to speed more important.
     config.concurrency = 1;
     config.browsers = platformMap[platform][target.toLowerCase()];
   }