Update unit test dependencies

benjaminapetersen · benjaminapetersen · commit b99d376203da · 2017-09-13T23:43:54.000-04:00
diff --git a/Gruntfile.js b/Gruntfile.js
@@ -15,6 +15,7 @@ module.exports = function (grunt) {
   var contextRoot = grunt.option('contextRoot') || "dev-console";
   var isMac = /^darwin/.test(process.platform) || grunt.option('mac');
 
+
   // Load grunt tasks automatically
   require('load-grunt-tasks')(grunt, {
     pattern: ['grunt-*', '!grunt-template-jasmine-istanbul']
@@ -602,9 +603,8 @@ module.exports = function (grunt) {
 
     // Test settings
     karma: {
-      unit: {
+      options: {
         configFile: 'test/karma.conf.js',
-        singleRun: true,
         // default in karma.conf.js is Firefox, however, Chrome has much better
         // error messages when writing tests.  Call like this:
         // grunt test
@@ -613,36 +613,38 @@ module.exports = function (grunt) {
         browsers: grunt.option('browsers') ?
                     grunt.option('browsers').split(',') :
                     ['Firefox']
+      },
+      unit: {
+        singleRun: true,
       }
     },
 
     protractor: {
       options: {
-        configFile: "test/protractor.conf.js", // Default config file
+        configFile: "test/protractor.conf.js",
         keepAlive: false, // If false, the grunt process stops when the test fails.
-        noColor: false, // If true, protractor will not use colors in its output.
+        noColor: false,
         args: {
-          // Arguments passed to the command
           suite: grunt.option('suite') || 'full',
           baseUrl: grunt.option('baseUrl') || ("https://localhost:9000/" + contextRoot + "/")
         }
       },
       default: {
         options: {
-          configFile: "test/protractor.conf.js", // Target-specific config file
+          configFile: "test/protractor.conf.js",
           args: {
             baseUrl: grunt.option('baseUrl') || ("https://localhost:9000/" + contextRoot + "/"),
             browser: grunt.option('browser') || "firefox"
-          } // Target-specific arguments
+          }
         }
       },
       mac: {
         options: {
-          configFile: "test/protractor-mac.conf.js", // Target-specific config file
+          configFile: "test/protractor-mac.conf.js",
           args: {
             baseUrl: grunt.option('baseUrl') || ("https://localhost:9000/" + contextRoot + "/"),
             browser: grunt.option('browser') || "firefox"
-          } // Target-specific arguments
+          }
         }
       }
     },
@@ -728,7 +730,7 @@ module.exports = function (grunt) {
   grunt.loadNpmTasks('grunt-angular-templates');
 
   // karma must run prior to coverage since karma will generate the coverage results
-  grunt.registerTask('test', [
+  grunt.registerTask('test-unit', [
     'clean:server',
     'concurrent:test',
     'postcss',
@@ -737,6 +739,10 @@ module.exports = function (grunt) {
     // 'coverage' - add back if we want to enforce coverage percentages
   ]);
 
+  // test as an alias to unit.  after updating protractor,
+  // will make test an alias for both unit & e2e
+  grunt.registerTask('test', ['test-unit']);
+
   grunt.registerTask('test-integration',
     grunt.option('baseUrl') ?
     [isMac ? 'protractor:mac' : 'protractor:default'] : // if a baseUrl is defined assume we dont want to run the local grunt server
diff --git a/package.json b/package.json
@@ -12,13 +12,17 @@
     "clean-css": "3.4.12",
     "connect-modrewrite": "0.7.9",
     "geckodriver": "1.3.0",
+
     "grunt": "0.4.5",
+
     "grunt-angular-templates": "1.0.3",
     "grunt-cli": "1.1.0",
     "grunt-concurrent": "2.3.1",
     "grunt-contrib-clean": "1.0.0",
     "grunt-contrib-concat": "1.0.0",
+
     "grunt-contrib-connect": "1.0.2",
+
     "grunt-contrib-copy": "1.0.0",
     "grunt-contrib-cssmin": "1.0.1",
     "grunt-contrib-htmlmin": "1.3.0",
@@ -29,7 +33,9 @@
     "grunt-contrib-watch": "1.0.0",
     "grunt-htmlhint": "0.4.1",
     "grunt-istanbul-coverage": "0.0.5",
-    "grunt-karma": "0.9.0",
+
+    "grunt-karma": "^2.0.0",
+
     "grunt-newer": "0.7.0",
     "grunt-ng-annotate": "0.3.2",
     "grunt-postcss": "^0.8.0",
@@ -40,19 +46,25 @@
     "grunt-wiredep": "3.0.0",
     "html-minifier": "1.1.1",
     "imagemin": "1.0.5",
+
+    "jasmine-core": "^2.8.0",
     "jasmine-beforeall": "0.1.1",
     "jasmine-spec-reporter": "1.1.2",
+
     "jshint-stylish": "0.2.0",
-    "karma": "0.12.23",
-    "karma-chrome-launcher": "^2.0.0",
-    "karma-coverage": "0.2.6",
-    "karma-firefox-launcher": "^1.0.0",
-    "karma-jasmine": "0.1.5",
-    "karma-ng-html2js-preprocessor": "1.0.0",
+
+    "karma": "^1.7.1",
+    "karma-chrome-launcher": "^2.2.0",
+    "karma-coverage": "^1.1.1",
+    "karma-firefox-launcher": "^1.0.1",
+    "karma-jasmine": "^1.1.0",
+    "karma-ng-html2js-preprocessor": "^1.0.0",
     "karma-phantomjs-launcher": "^1.0.4",
+
     "less": "2.6.1",
     "load-grunt-tasks": "0.4.0",
     "lodash": "3.10.1",
+
     "protractor": "1.7.0",
     "protractor-screenshot-reporter": "0.0.5",
     "serve-static": "1.10.2",
diff --git a/test/karma.conf.js b/test/karma.conf.js
@@ -7,8 +7,14 @@ module.exports = function(config) {
   'use strict';
 
   config.set({
+    // maximum boot-up time allowed for a browser to start and connect to Karma
+    // a browser gets 3x changes within this timeout range to connect to Karma
+    // there are other timeouts as well, consult the config file
+    // docs: https://karma-runner.github.io/1.0/config/configuration-file.html
+    captureTimeout: 3000,
     // enable / disable watching file and executing tests whenever any file changes
-    autoWatch: true,
+    // why set to true when we have grunt watch?
+    autoWatch: false,
 
     // base path, that will be used to resolve files and exclude
     basePath: '../',
@@ -107,7 +113,7 @@ module.exports = function(config) {
     exclude: [],
 
     // web server port
-    port: 8443,
+    // port: 8443,
 
     // Start these browsers, currently available:
     // - Chrome
@@ -128,18 +134,12 @@ module.exports = function(config) {
 
     // Continuous Integration mode
     // if true, it capture browsers, run tests and exit
-    singleRun: false,
-
-    colors: true,
+     singleRun: false,
 
     // level of logging
     // possible values: LOG_DISABLE || LOG_ERROR || LOG_WARN || LOG_INFO || LOG_DEBUG
-    logLevel: config.LOG_DEBUG,
+    logLevel: config.LOG_ERROR,
 
-    // Help karma find the views on disk in the app subdirectory
-    proxies: {
-      '/views/': '/app/views/'
-    },
     // URL root prevent conflicts with the site root
     // urlRoot: '_karma_'
 
@@ -153,18 +153,15 @@ module.exports = function(config) {
 
     ngHtml2JsPreprocessor: {
       moduleName: 'openshiftConsoleTemplates',
-      cacheIdFromPath: function(filepath) {
-        return filepath.replace('app/', '');
-      },
+      stripPrefix: '/app',
     },
 
     reporters: ['progress', 'coverage'],
 
     coverageReporter: {
-      reporters:[
-        {type: 'json', dir:'test/coverage/'},
-        {type: 'text-summary', dir:'test/coverage/'}
-      ]
-    }
+      type: 'text',
+      // outputs the results of coverage reporter to this dir
+      dir: 'test-results/coverage/'
+    },
   });
 };
diff --git a/test/spec/services/applicationGeneratorSpec.js b/test/spec/services/applicationGeneratorSpec.js
@@ -255,77 +255,73 @@ describe("ApplicationGenerator", function(){
     });
 
     it("should generate a BuildConfig for the source", function(){
-      expect(resources.buildConfig).toEqual(
+      var generatedBuildConfig = resources.buildConfig;
+
+      expect(generatedBuildConfig.apiVersion).toEqual('v1');
+      expect(generatedBuildConfig.kind).toEqual('BuildConfig');
+      expect(generatedBuildConfig.metadata.name).toEqual('ruby-hello-world');
+      expect(generatedBuildConfig.metadata.labels).toEqual({
+        "foo" : "bar",
+        "abc" : "xyz"
+      });
+      expect(generatedBuildConfig.metadata.annotations).toEqual({
+        "openshift.io/generated-by": "OpenShiftWebConsole"
+      });
+      expect(generatedBuildConfig.spec.output).toEqual({
+        "to": {
+          "name": "ruby-hello-world:latest",
+          "kind": "ImageStreamTag"
+        }
+      });
+      expect(generatedBuildConfig.spec.source).toEqual({
+        "git": {
+          "ref": "master",
+          "uri": "https://github.com/openshift/ruby-hello-world.git"
+        },
+        "type": "Git"
+      });
+      expect(generatedBuildConfig.spec.strategy.type).toEqual('Source');
+
+      expect(generatedBuildConfig.spec.strategy.sourceStrategy.from.kind).toEqual('ImageStreamTag');
+      expect(generatedBuildConfig.spec.strategy.sourceStrategy.from.name).toEqual('origin-ruby-sample:latest');
+      expect(generatedBuildConfig.spec.strategy.sourceStrategy.from).toEqual({
+        "kind": "ImageStreamTag",
+        "name": "origin-ruby-sample:latest",
+        "namespace": undefined
+      });
+      expect(generatedBuildConfig.spec.strategy.sourceStrategy.env).toEqual([
         {
-            "apiVersion": "v1",
-            "kind": "BuildConfig",
-            "metadata": {
-                "name": "ruby-hello-world",
-                "labels": {
-                  "foo" : "bar",
-                  "abc" : "xyz"
-                },
-                "annotations": {
-                  "openshift.io/generated-by": "OpenShiftWebConsole"
-                }
-            },
-            "spec": {
-                "output": {
-                    "to": {
-                        "name": "ruby-hello-world:latest",
-                        "kind": "ImageStreamTag"
-                    }
-                },
-                "source": {
-                    "git": {
-                        "ref": "master",
-                        "uri": "https://github.com/openshift/ruby-hello-world.git"
-                    },
-                    "type": "Git"
-                },
-                "strategy": {
-                    "type": "Source",
-                    "sourceStrategy" : {
-                      "from": {
-                        "kind": "ImageStreamTag",
-                        "name": "origin-ruby-sample:latest"
-                      },
-                      "env": [
-                        {
-                          "name": "BUILD_ENV_1",
-                          "value": "someValue"
-                        },
-                        {
-                          "name": "BUILD_ENV_2",
-                          "value": "anotherValue"
-                        }
-                      ]
-                    }
-                },
-                "triggers": [
-                    {
-                        "generic": {
-                            "secret": "secret101"
-                        },
-                        "type": "Generic"
-                    },
-                    {
-                        "github": {
-                            "secret": "secret101"
-                        },
-                        "type": "GitHub"
-                    },
-                    {
-                      "imageChange" : {},
-                      "type" : "ImageChange"
-                    },
-                    {
-                        "type": "ConfigChange"
-                    }
-                ]
-            }
-          }
-      );
+          "name": "BUILD_ENV_1",
+          "value": "someValue"
+        },
+        {
+          "name": "BUILD_ENV_2",
+          "value": "anotherValue"
+        }
+      ]);
+
+      expect(generatedBuildConfig.spec.triggers).toEqual([
+        {
+          "generic": {
+              "secret": "secret101"
+          },
+          "type": "Generic"
+        },
+        {
+          "github": {
+              "secret": "secret101"
+          },
+          "type": "GitHub"
+        },
+        {
+          "imageChange" : {},
+          "type" : "ImageChange"
+        },
+        {
+          "type": "ConfigChange"
+        }
+      ]);
+
     });
 
     it("should generate an ImageStream for the build output", function(){
diff --git a/test/spec/services/membership/membershipSpec.js b/test/spec/services/membership/membershipSpec.js
