Skip to content
This repository was archived by the owner on Aug 30, 2021. It is now read-only.

Commit fdf1ad5

Browse files
committed
fixing grunt and gulp e2e tests
1 parent cc80930 commit fdf1ad5

File tree

5 files changed

+43
-21
lines changed

5 files changed

+43
-21
lines changed

gruntfile.js

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,8 @@ module.exports = function (grunt) {
201201
options: {
202202
configFile: 'protractor.conf.js',
203203
keepAlive: true,
204-
noColor: false
204+
noColor: false,
205+
webdriverManagerUpdate: true
205206
},
206207
e2e: {
207208
options: {
@@ -274,9 +275,11 @@ module.exports = function (grunt) {
274275
grunt.registerTask('build', ['env:dev', 'lint', 'ngAnnotate', 'uglify', 'cssmin']);
275276

276277
// Run the project tests
277-
grunt.registerTask('test', ['env:test', 'lint', 'mkdir:upload', 'copy:localConfig', 'server', 'mochaTest', 'karma:unit']);
278+
grunt.registerTask('test', ['env:test', 'lint', 'mkdir:upload', 'copy:localConfig', 'server', 'mochaTest', 'karma:unit', 'protractor']);
278279
grunt.registerTask('test:server', ['env:test', 'lint', 'server', 'mochaTest']);
279280
grunt.registerTask('test:client', ['env:test', 'lint', 'server', 'karma:unit']);
281+
grunt.registerTask('test:e2e', ['env:test', 'lint', 'server', 'protractor']);
282+
280283
// Run project coverage
281284
grunt.registerTask('coverage', ['env:test', 'lint', 'mocha_istanbul:coverage']);
282285

gulpfile.js

Lines changed: 28 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,10 @@ var _ = require('lodash'),
1515
}
1616
}),
1717
path = require('path'),
18-
endOfLine = require('os').EOL;
18+
endOfLine = require('os').EOL,
19+
protractor = require("gulp-protractor").protractor,
20+
webdriver_update = require("gulp-protractor").webdriver_update,
21+
webdriver_standalone = require("gulp-protractor").webdriver_standalone;
1922

2023
// Set NODE_ENV to 'test'
2124
gulp.task('env:test', function () {
@@ -195,17 +198,28 @@ gulp.task('karma', function (done) {
195198
}));
196199
});
197200

198-
// Selenium standalone WebDriver update task
199-
gulp.task('webdriver-update', plugins.protractor.webdriver_update);
201+
// Downloads the selenium webdriver
202+
gulp.task('webdriver_update', webdriver_update);
203+
204+
// Start the standalone selenium server
205+
// NOTE: This is not needed if you reference the
206+
// seleniumServerJar in your protractor.conf.js
207+
gulp.task('webdriver_standalone', webdriver_standalone);
200208

201209
// Protractor test runner task
202-
gulp.task('protractor', function () {
210+
gulp.task('protractor', ['webdriver_update'], function () {
203211
gulp.src([])
204-
.pipe(plugins.protractor.protractor({
212+
.pipe(protractor({
205213
configFile: 'protractor.conf.js'
206214
}))
207-
.on('error', function (e) {
208-
throw e;
215+
.on('end', function() {
216+
console.log('E2E Testing complete');
217+
// exit with success.
218+
process.exit(0);
219+
})
220+
.on('error', function(err) {
221+
console.log('E2E Tests failed');
222+
process.exit(1);
209223
});
210224
});
211225

@@ -221,15 +235,19 @@ gulp.task('build', function (done) {
221235

222236
// Run the project tests
223237
gulp.task('test', function (done) {
224-
runSequence('env:test', 'karma', 'mocha', done);
238+
runSequence('env:test', 'lint', 'mocha', 'karma', 'nodemon', 'protractor', done);
225239
});
226240

227241
gulp.task('test:server', function (done) {
228-
runSequence('env:test', 'mocha', done);
242+
runSequence('env:test', 'lint', 'mocha', done);
229243
});
230244

231245
gulp.task('test:client', function (done) {
232-
runSequence('env:test', 'karma', done);
246+
runSequence('env:test', 'lint', 'karma', done);
247+
});
248+
249+
gulp.task('test:e2e', function (done) {
250+
runSequence('env:test', 'lint', 'nodemon', 'protractor', done);
233251
});
234252

235253
// Run the project in development mode

modules/articles/tests/e2e/articles.e2e.tests.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
describe('Articles E2E Tests:', function () {
44
describe('Test articles page', function () {
55
it('Should report missing credentials', function () {
6-
browser.get('http://localhost:3000/articles');
6+
browser.get('http://localhost:3001/articles');
77
expect(element.all(by.repeater('article in articles')).count()).toEqual(0);
88
});
99
});

modules/users/tests/e2e/users.e2e.tests.js

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,12 @@
33
describe('Users E2E Tests:', function () {
44
describe('Signin Validation', function () {
55
it('Should report missing credentials', function () {
6-
browser.get('http://localhost:3000/authentication/signin');
6+
browser.get('http://localhost:3001/authentication/signin');
77
element(by.css('button[type=submit]')).click();
8-
element(by.binding('error')).getText().then(function (errorText) {
9-
expect(errorText).toBe('Missing credentials');
10-
});
8+
// Username Error
9+
expect(element.all(by.css('.error-text')).get(0).getText()).toBe('Username is required.');
10+
// Password Error
11+
expect(element.all(by.css('.error-text')).get(1).getText()).toBe('Password is required.');
1112
});
1213
});
1314
});

package.json

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@
6565
"validator": "^3.41.2"
6666
},
6767
"devDependencies": {
68+
"coveralls": "^2.11.4",
6869
"grunt-concurrent": "^2.0.0",
6970
"grunt-contrib-copy": "~0.8.0",
7071
"grunt-contrib-csslint": "~0.4.0",
@@ -76,12 +77,12 @@
7677
"grunt-contrib-watch": "~0.6.1",
7778
"grunt-env": "~0.4.4",
7879
"grunt-karma": "~0.11.2",
79-
"grunt-mocha-test": "~0.12.7",
8080
"grunt-mocha-istanbul": "^2.4.0",
81+
"grunt-mocha-test": "~0.12.7",
8182
"grunt-ng-annotate": "^1.0.1",
8283
"grunt-node-inspector": "~0.2.0",
8384
"grunt-nodemon": "~0.4.0",
84-
"grunt-protractor-runner": "^2.0.0",
85+
"grunt-protractor-runner": "^2.1.0",
8586
"gulp": "^3.9.0",
8687
"gulp-angular-templatecache": "^1.7.0",
8788
"gulp-autoprefixer": "^2.3.1",
@@ -111,7 +112,6 @@
111112
"load-grunt-tasks": "^3.2.0",
112113
"run-sequence": "^1.1.1",
113114
"should": "^7.0.1",
114-
"supertest": "^1.0.1",
115-
"coveralls": "^2.11.4"
115+
"supertest": "^1.0.1"
116116
}
117117
}

0 commit comments

Comments
 (0)