test(config): added a common configuration environment.js

Author: elgalu

Diff for: debugging/failureConf.js

@@ -1,19 +1,19 @@
+var env = require('../environment.js');
+
 // Examples of tests to show how debugging works with Protractor. Tests
 // should be run against the testapp.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to the current working directly when
   // protractor is called.
   specs: [
     'failure_spec.js',
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   // ----- Options to be passed to minijasminenode.
   jasmineNodeOpts: {

Diff for: debugging/timeoutConf.js

@@ -1,20 +1,19 @@
+var env = require('../environment.js');
+
 // Examples of tests to show how timeouts works with Protractor. Tests
 // should be run against the testapp.
-
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to the current working directly when
   // protractor is called.
   specs: [
     'timeout_spec.js',
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   // ----- Options to be passed to minijasminenode.
   jasmineNodeOpts: {

Diff for: spec/altRootConf.js

@@ -1,18 +1,18 @@
+var env = require('./environment.js');
+
 // Tests for an Angular app where ng-app is not on the body.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to this config.
   specs: [
     'altRoot/*_spec.js',
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  // Selector for the element housing the angular app.
-  rootElement: 'div#nested-ng-app',
+  baseUrl: env.baseUrl,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  // Selector for the element housing the angular app.
+  rootElement: 'div#nested-ng-app'
 };

Diff for: spec/basic/navigation_spec.js

@@ -1,3 +1,5 @@
+var env = require('../environment.js');
+
 describe('navigation', function() {
   beforeEach(function() {
     browser.get('index.html#/form');
@@ -28,33 +30,31 @@ describe('navigation', function() {
   // bootstrapping with Angular
   /*   
   it('should navigate back and forward properly', function() {
-    var port =  process.env.HTTP_PORT || '8000';
     browser.get('index.html#/repeater');
     expect(browser.getCurrentUrl()).
-      toEqual('http://localhost:'+port+'/index.html#/repeater');
+      toEqual(env.baseUrl+'/index.html#/repeater');
 
     browser.navigate().back();
     expect(browser.getCurrentUrl()).
-      toEqual('http://localhost:'+port+'/index.html#/form');
+      toEqual(env.baseUrl+'/index.html#/form');
     
     browser.navigate().forward();
     expect(browser.getCurrentUrl()).
-      toEqual('http://localhost:'+port+'/index.html#/repeater'); 
+      toEqual(env.baseUrl+'/index.html#/repeater'); 
   });
   */
 
   it('should navigate back and forward properly from link', function() {
-    var port =  process.env.HTTP_PORT || '8000';
     element(by.linkText('repeater')).click();
     expect(browser.getCurrentUrl()).
-      toEqual('http://localhost:'+port+'/index.html#/repeater');
+      toEqual(env.baseUrl+'/index.html#/repeater');
 
     browser.navigate().back();
     expect(browser.getCurrentUrl()).
-      toEqual('http://localhost:'+port+'/index.html#/form');
+      toEqual(env.baseUrl+'/index.html#/form');
 
     browser.navigate().forward();
     expect(browser.getCurrentUrl()).
-      toEqual('http://localhost:'+port+'/index.html#/repeater'); 
+      toEqual(env.baseUrl+'/index.html#/repeater'); 
   });
 });

Diff for: spec/basicConf.js

@@ -1,6 +1,8 @@
+var env = require('./environment.js');
+
 // The main suite of Protractor tests.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to this directory.
   specs: [
@@ -14,11 +16,9 @@ exports.config = {
 
   chromeOnly: false,
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   params: {
     login: {

Diff for: spec/ciConf.js

@@ -1,3 +1,5 @@
+var env = require('./environment.js');
+
 // The main suite of Protractor tests to be run on CI servers.
 exports.config = {
   sauceUser: process.env.SAUCE_USERNAME,
@@ -30,7 +32,7 @@ exports.config = {
     'selenium-version': '2.41.0'
   }],
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   params: {
     login: {

Diff for: spec/cucumberConf.js

@@ -1,6 +1,8 @@
+var env = require('./environment.js');
+
 // A small suite to make sure the cucumber framework works.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   framework: 'cucumber',
 
@@ -9,11 +11,9 @@ exports.config = {
     'cucumber/*.feature'
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   cucumberOpts: {
     require: 'cucumber/stepDefinitions.js',

Diff for: spec/environment.js

@@ -0,0 +1,20 @@
+ // Common configuration files with defaults plus overrides from environment vars
+module.exports = {
+  // The address of a running selenium server.
+  seleniumAddress: 
+    (process.env.SELENIUM_URL || 'http://localhost:4444/wd/hub'),
+
+  // Capabilities to be passed to the webdriver instance.
+  capabilities: {
+    'browserName':
+        (process.env.TEST_BROWSER_NAME    || 'chrome'),
+    'version':
+        (process.env.TEST_BROWSER_VERSION || 'ANY')
+  },
+
+  // A base URL for your application under test.
+  baseUrl:
+    'http://' + (process.env.HTTP_HOST || 'localhost') +
+          ':' + (process.env.HTTP_PORT || '8000')
+
+};

Diff for: spec/junitOutputConf.js

@@ -1,18 +1,20 @@
+var env = require('./environment.js');
+
 // This configuration file shows an example of using an Jasmine reporter to
 // output test results in XML format. Before running, you will need to
 // npm install jasmine-reporters
 
 // The main suite of Protractor tests.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   specs: [
     'basic/*_spec.js'
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
+
+  baseUrl: env.baseUrl,
 
   onPrepare: function() {
     // The require statement must be down here, since jasmine-reporters
@@ -28,7 +30,5 @@ exports.config = {
       user: 'Jane',
       password: '1234'
     }
-  },
-
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  }
 };

Diff for: spec/login/login_spec.js

@@ -1,6 +1,8 @@
+var env = require('../environment.js');
+
 describe('pages with login', function() {
   it('should log in with a non-Angular page', function() {
-    browser.get('http://localhost:8000/index.html');
+    browser.get(env.baseUrl+'/index.html');
 
     var angularElement = element(by.model('username'));
     expect(angularElement.getAttribute('value')).toEqual('Anon');

Diff for: spec/mochaConf.js

@@ -1,6 +1,8 @@
+var env = require('./environment.js');
+
 // A small suite to make sure the mocha frameowork works.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   framework: 'mocha',
 
@@ -9,9 +11,7 @@ exports.config = {
     'mocha/*_spec.js'
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 };

Diff for: spec/multiConf.js

@@ -1,6 +1,8 @@
 // A suite of tests to run on two browsers at once.
+var env = require('./environment.js');
+
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to this directory.
   specs: [
@@ -15,7 +17,7 @@ exports.config = {
     'browserName': 'firefox'
   }],
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   params: {
     login: {

Diff for: spec/multiSplitConf.js

@@ -1,7 +1,9 @@
 // A suite of tests to run on two browsers at once, splitting test files between
 // the two instances of chrome.
+var env = require('./environment.js');
+
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to this directory.
   specs: [
@@ -21,7 +23,7 @@ exports.config = {
     count: 2
   }],
 
-  baseUrl: 'http://localhost:8000',
+  baseUrl: env.baseUrl,
 
   params: {
     login: {

Diff for: spec/onPrepareConf.js

@@ -1,17 +1,18 @@
 // Configuration using a function in onPrepare to set a parameter before
 // testing.
+var env = require('./environment.js');
+
+// The main suite of Protractor tests.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   specs: [
     'onPrepare/*_spec.js'
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   onPrepare: function() {
     browser.params.password = '12345';

Diff for: spec/onPrepareFileConf.js

@@ -1,18 +1,18 @@
+var env = require('./environment.js');
+
 // Configuration using a string in onPrepare to load a file with code to
 // execute once before tests.
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to this directory.
   specs: [
     'onPrepare/*_spec.js'
   ],
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   onPrepare: 'onPrepare/startup.js'
 };

Diff for: spec/smokeConf.js

@@ -1,3 +1,5 @@
+var env = require('./environment.js');
+
 // Smoke tests to be run on CI servers - covers more browsers than
 // ciConf.js, but does not run all tests.
 exports.config = {
@@ -45,7 +47,7 @@ exports.config = {
     'platform': 'Windows 7'
   }],
 
-  baseUrl: 'http://localhost:' + (process.env.HTTP_PORT || '8000'),
+  baseUrl: env.baseUrl,
 
   params: {
     login: {

Diff for: spec/suitesConf.js

@@ -1,5 +1,7 @@
+var env = require('./environment.js');
+
 exports.config = {
-  seleniumAddress: 'http://localhost:4444/wd/hub',
+  seleniumAddress: env.seleniumAddress,
 
   // Spec patterns are relative to this directory.
   suites: {
@@ -8,9 +10,9 @@ exports.config = {
     failingtest: 'suites/always_fail_spec.js'
   },
 
-  capabilities: {
-    'browserName': 'chrome'
-  },
+  chromeOnly: false,
+
+  capabilities: env.capabilities,
 
-  baseUrl: 'http://localhost:8000',
+  baseUrl: env.baseUrl,
 };