Update unit test dependencies

Author: benjaminapetersen

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

app/scripts/services/applicationGenerator.js

@@ -351,15 +351,17 @@ angular.module("openshiftConsole")
             sourceStrategy: {
               from: {
                 kind: "ImageStreamTag",
-                name: input.imageName + ":" + input.imageTag,
-                namespace: input.namespace
+                name: input.imageName + ":" + input.imageTag
               },
               env: env
             }
           },
           triggers: triggers
         }
       };
+      if(input.namespace) {
+        bc.spec.strategy.namespace = input.namespace;
+      }
       if (_.get(input, 'buildConfig.secrets.gitSecret[0].name')) {
         bc.spec.source.sourceSecret = _.head(input.buildConfig.secrets.gitSecret);
       }

app/scripts/services/membership/membership.js

@@ -96,12 +96,16 @@ angular
                           if(!result[subject.kind].subjects[subjectKey]) {
                             result[subject.kind].subjects[subjectKey]  = {
                               name: subject.name,
-                              namespace: subject.namespace,
                               roles: {}
                             };
+                            if(subject.namespace) {
+                              result[subject.kind].subjects[subjectKey].namespace = subject.namespace;
+                            }
                           }
                           if(!_.includes(result[subject.kind].subjects[subjectKey].roles, roleKey)) {
-                            result[subject.kind].subjects[subjectKey].roles[roleKey] = roles[roleKey];
+                            if(roles[roleKey]) {
+                              result[subject.kind].subjects[subjectKey].roles[roleKey] = roles[roleKey];
+                            }
                           }
                         });
                         return result;

dist/scripts/scripts.js

@@ -1479,16 +1479,15 @@ type: "Source",
 sourceStrategy: {
 from: {
 kind: "ImageStreamTag",
-name: e.imageName + ":" + e.imageTag,
-namespace: e.namespace
+name: e.imageName + ":" + e.imageTag
 },
 env: n
 }
 },
 triggers: a
 }
 };
-return _.get(e, "buildConfig.secrets.gitSecret[0].name") && (l.spec.source.sourceSecret = _.head(e.buildConfig.secrets.gitSecret)), e.buildConfig.contextDir && (l.spec.source.contextDir = e.buildConfig.contextDir), l;
+return e.namespace && (l.spec.strategy.namespace = e.namespace), _.get(e, "buildConfig.secrets.gitSecret[0].name") && (l.spec.source.sourceSecret = _.head(e.buildConfig.secrets.gitSecret)), e.buildConfig.contextDir && (l.spec.source.contextDir = e.buildConfig.contextDir), l;
 }, o._generateImageStream = function(e) {
 return {
 apiVersion: "v1",
@@ -2324,9 +2323,8 @@ return _.each(a.subjects, function(a) {
 var o = n(a.namespace, a.name);
 e[a.kind].subjects[o] || (e[a.kind].subjects[o] = {
 name: a.name,
-namespace: a.namespace,
 roles: {}
-}), _.includes(e[a.kind].subjects[o].roles, r) || (e[a.kind].subjects[o].roles[r] = t[r]);
+}, a.namespace && (e[a.kind].subjects[o].namespace = a.namespace)), _.includes(e[a.kind].subjects[o].roles, r) || t[r] && (e[a.kind].subjects[o].roles[r] = t[r]);
 }), e;
 }, {
 User: {

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",

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,16 @@ module.exports = function(config) {
 
     ngHtml2JsPreprocessor: {
       moduleName: 'openshiftConsoleTemplates',
-      cacheIdFromPath: function(filepath) {
-        return filepath.replace('app/', '');
-      },
+      stripPrefix: '/app',
     },
 
+    // order of reporters matters, input/output may break
     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/'
+    },
   });
 };