Skip to content

Commit 180dc63

Browse files
Awk34Awk34
committed
2 parents 5eb3bb0 + d3fa78c commit 180dc63

27 files changed

+135
-143
lines changed

Diff for: ‎app/index.js

+6-9
Original file line numberDiff line numberDiff line change
@@ -26,15 +26,12 @@ var AngularFullstackGenerator = yeoman.generators.Base.extend({
2626

2727
this.filters = {};
2828

29-
// dynamic assertion statement
30-
this.does = this.is = function(foo) {
31-
foo = this.engine(foo.replace(/\(;>%%<;\)/g, '<%')
32-
.replace(/\(;>%<;\)/g, '%>'), this);
33-
if (this.filters.should) {
34-
return foo + '.should';
35-
} else {
36-
return 'expect(' + foo + ').to';
37-
}
29+
// dynamic assertion statements
30+
this.expect = function() {
31+
return this.filters.expect ? 'expect(' : '';
32+
}.bind(this);
33+
this.to = function() {
34+
return this.filters.expect ? ').to' : '.should';
3835
}.bind(this);
3936
},
4037

Diff for: ‎app/templates/client/app/account(auth)/login/login(html).html

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
<div class="row">
55
<div class="col-sm-12">
66
<h1>Login</h1>
7-
<p>Accounts are reset on server restart from <code>server/config/seed.js</code>. Default account is <code>test@test.com</code> / <code>test</code></p>
8-
<p>Admin account is <code>admin@admin.com</code> / <code>admin</code></p>
7+
<p>Accounts are reset on server restart from <code>server/config/seed.js</code>. Default account is <code>test@example.com</code> / <code>test</code></p>
8+
<p>Admin account is <code>admin@example.com</code> / <code>admin</code></p>
99
</div>
1010
<div class="col-sm-12">
1111
<form class="form" name="form" ng-submit="login(form)" novalidate>

Diff for: ‎app/templates/client/app/account(auth)/login/login(jade).jade

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,12 @@ navbar
77
| Accounts are reset on server restart from
88
code server/config/seed.js
99
| . Default account is
10-
code test@test.com
10+
code test@example.com
1111
| /
1212
code test
1313
p
1414
| Admin account is
15-
code admin@admin.com
15+
code admin@example.com
1616
| /
1717
code admin
1818

Diff for: ‎app/templates/client/app/admin(auth)/admin.controller(coffee).coffee

+1-3
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,7 @@
33
angular.module '<%= scriptAppName %>'
44
.controller 'AdminCtrl', ($scope, $http, Auth, User) ->
55

6-
$http.get '/api/users'
7-
.success (users) ->
8-
$scope.users = users
6+
$scope.users = User.query()
97

108
$scope.delete = (user) ->
119
User.remove id: user._id

Diff for: ‎app/templates/client/app/main/main.controller(coffee).coffee

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ angular.module '<%= scriptAppName %>'
44
.controller 'MainCtrl', ($scope, $http<% if (filters.socketio) { %>, socket<% } %>) ->
55
$scope.awesomeThings = []
66

7-
$http.get('/api/things').success (awesomeThings) ->
8-
$scope.awesomeThings = awesomeThings
7+
$http.get('/api/things').then (response) ->
8+
$scope.awesomeThings = response.data
99
<% if (filters.socketio) { %>socket.syncUpdates 'thing', $scope.awesomeThings<% } %>
1010
<% if (filters.models) { %>
1111
$scope.addThing = ->

Diff for: ‎app/templates/client/app/main/main.controller(js).js

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@ angular.module('<%= scriptAppName %>')
44
.controller('MainCtrl', function($scope, $http<% if (filters.socketio) { %>, socket<% } %>) {
55
$scope.awesomeThings = [];
66

7-
$http.get('/api/things').success(function(awesomeThings) {
8-
$scope.awesomeThings = awesomeThings;<% if (filters.socketio) { %>
7+
$http.get('/api/things').then(function(response) {
8+
$scope.awesomeThings = response.data;<% if (filters.socketio) { %>
99
socket.syncUpdates('thing', $scope.awesomeThings);<% } %>
1010
});
1111
<% if (filters.models) { %>

Diff for: ‎app/templates/client/app/main/main.controller.spec(coffee).coffee

+1-1
Original file line numberDiff line numberDiff line change
@@ -29,4 +29,4 @@ describe 'Controller: MainCtrl', ->
2929
it 'should attach a list of things to the scope', ->
3030
$httpBackend.flush()<% if (filters.jasmine) { %>
3131
expect(scope.awesomeThings.length).toBe 4 <% } if (filters.mocha) { %>
32-
<%= does("scope.awesomeThings.length") %>.equal 4<% } %>
32+
<%= expect() %>scope.awesomeThings.length<%= to() %>.equal 4<% } %>

Diff for: ‎app/templates/client/app/main/main.controller.spec(js).js

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,6 @@ describe('Controller: MainCtrl', function() {
2828
it('should attach a list of things to the scope', function() {
2929
$httpBackend.flush();<% if (filters.jasmine) { %>
3030
expect(scope.awesomeThings.length).toBe(4);<% } if (filters.mocha) { %>
31-
<%= does("scope.awesomeThings.length") %>.equal(4);<% } %>
31+
<%= expect() %>scope.awesomeThings.length<%= to() %>.equal(4);<% } %>
3232
});
3333
});

Diff for: ‎app/templates/client/components/auth(auth)/user.service(js).js

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,5 @@ angular.module('<%= scriptAppName %>')
1818
id:'me'
1919
}
2020
}
21-
});
21+
});
2222
});

Diff for: ‎app/templates/e2e/account(auth)/login/login.spec(jasmine).js

+1-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ describe('Login View', function() {
1414

1515
var testUser = {
1616
name: 'Test User',
17-
email: 'test@test.com',
17+
email: 'test@example.com',
1818
password: 'test'
1919
};
2020

Diff for: ‎app/templates/e2e/account(auth)/login/login.spec(mocha).js

+11-11
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ describe('Login View', function() {
1414

1515
var testUser = {
1616
name: 'Test User',
17-
email: 'test@test.com',
17+
email: 'test@example.com',
1818
password: 'test'
1919
};
2020

@@ -35,12 +35,12 @@ describe('Login View', function() {
3535
});
3636

3737
it('should include login form with correct inputs and submit button', function() {
38-
<%= does("page.form.email.getAttribute('type')") %>.eventually.equal('email');
39-
<%= does("page.form.email.getAttribute('name')") %>.eventually.equal('email');
40-
<%= does("page.form.password.getAttribute('type')") %>.eventually.equal('password');
41-
<%= does("page.form.password.getAttribute('name')") %>.eventually.equal('password');
42-
<%= does("page.form.submit.getAttribute('type')") %>.eventually.equal('submit');
43-
<%= does("page.form.submit.getText()") %>.eventually.equal('Login');
38+
<%= expect() %>page.form.email.getAttribute('type')<%= to() %>.eventually.equal('email');
39+
<%= expect() %>page.form.email.getAttribute('name')<%= to() %>.eventually.equal('email');
40+
<%= expect() %>page.form.password.getAttribute('type')<%= to() %>.eventually.equal('password');
41+
<%= expect() %>page.form.password.getAttribute('name')<%= to() %>.eventually.equal('password');
42+
<%= expect() %>page.form.submit.getAttribute('type')<%= to() %>.eventually.equal('submit');
43+
<%= expect() %>page.form.submit.getText()<%= to() %>.eventually.equal('Login');
4444
});
4545

4646
describe('with local auth', function() {
@@ -50,8 +50,8 @@ describe('Login View', function() {
5050

5151
var navbar = require('../../components/navbar/navbar.po');
5252

53-
<%= does("browser.getCurrentUrl()") %>.eventually.equal(config.baseUrl + '/');
54-
<%= does("navbar.navbarAccountGreeting.getText()") %>.eventually.equal('Hello ' + testUser.name);
53+
<%= expect() %>browser.getCurrentUrl()<%= to() %>.eventually.equal(config.baseUrl + '/');
54+
<%= expect() %>navbar.navbarAccountGreeting.getText()<%= to() %>.eventually.equal('Hello ' + testUser.name);
5555
});
5656

5757
describe('and invalid credentials', function() {
@@ -65,10 +65,10 @@ describe('Login View', function() {
6565
password: 'badPassword'
6666
});
6767

68-
<%= does("browser.getCurrentUrl()") %>.eventually.equal(config.baseUrl + '/login');
68+
<%= expect() %>browser.getCurrentUrl()<%= to() %>.eventually.equal(config.baseUrl + '/login');
6969

7070
var helpBlock = page.form.element(by.css('.form-group.has-error .help-block.ng-binding'));
71-
<%= does("helpBlock.getText()") %>.eventually.equal('This password is not correct.');
71+
<%= expect() %>helpBlock.getText()<%= to() %>.eventually.equal('This password is not correct.');
7272
});
7373

7474
});

Diff for: ‎app/templates/e2e/account(auth)/logout/logout.spec(jasmine).js

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ describe('Logout View', function() {
1212

1313
var testUser = {
1414
name: 'Test User',
15-
email: 'test@test.com',
15+
email: 'test@example.com',
1616
password: 'test'
1717
};
1818

Diff for: ‎app/templates/e2e/account(auth)/logout/logout.spec(mocha).js

+5-5
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ describe('Logout View', function() {
1212

1313
var testUser = {
1414
name: 'Test User',
15-
email: 'test@test.com',
15+
email: 'test@example.com',
1616
password: 'test'
1717
};
1818

@@ -39,15 +39,15 @@ describe('Logout View', function() {
3939
it('should logout a user and redirecting to "/"', function() {
4040
var navbar = require('../../components/navbar/navbar.po');
4141

42-
<%= does("browser.getCurrentUrl()") %>.eventually.equal(config.baseUrl + '/');
43-
<%= does("navbar.navbarAccountGreeting.getText()") %>.eventually.equal('Hello ' + testUser.name);
42+
<%= expect() %>browser.getCurrentUrl()<%= to() %>.eventually.equal(config.baseUrl + '/');
43+
<%= expect() %>navbar.navbarAccountGreeting.getText()<%= to() %>.eventually.equal('Hello ' + testUser.name);
4444

4545
browser.get(config.baseUrl + '/logout');
4646

4747
navbar = require('../../components/navbar/navbar.po');
4848

49-
<%= does("browser.getCurrentUrl()") %>.eventually.equal(config.baseUrl + '/');
50-
<%= does("navbar.navbarAccountGreeting.isDisplayed()") %>.eventually.equal(false);
49+
<%= expect() %>browser.getCurrentUrl()<%= to() %>.eventually.equal(config.baseUrl + '/');
50+
<%= expect() %>navbar.navbarAccountGreeting.isDisplayed()<%= to() %>.eventually.equal(false);
5151
});
5252

5353
});

Diff for: ‎app/templates/e2e/account(auth)/signup/signup.spec(jasmine).js

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ describe('Signup View', function() {
1515

1616
var testUser = {
1717
name: 'Test',
18-
email: 'test@test.com',
18+
email: 'test@example.com',
1919
password: 'test'
2020
};
2121

Diff for: ‎app/templates/e2e/account(auth)/signup/signup.spec(mocha).js

+14-14
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ describe('Signup View', function() {
1515

1616
var testUser = {
1717
name: 'Test',
18-
email: 'test@test.com',
18+
email: 'test@example.com',
1919
password: 'test'
2020
};
2121

@@ -29,14 +29,14 @@ describe('Signup View', function() {
2929
});
3030

3131
it('should include signup form with correct inputs and submit button', function() {
32-
<%= does("page.form.name.getAttribute('type')") %>.eventually.equal('text');
33-
<%= does("page.form.name.getAttribute('name')") %>.eventually.equal('name');
34-
<%= does("page.form.email.getAttribute('type')") %>.eventually.equal('email');
35-
<%= does("page.form.email.getAttribute('name')") %>.eventually.equal('email');
36-
<%= does("page.form.password.getAttribute('type')") %>.eventually.equal('password');
37-
<%= does("page.form.password.getAttribute('name')") %>.eventually.equal('password');
38-
<%= does("page.form.submit.getAttribute('type')") %>.eventually.equal('submit');
39-
<%= does("page.form.submit.getText()") %>.eventually.equal('Sign up');
32+
<%= expect() %>page.form.name.getAttribute('type')<%= to() %>.eventually.equal('text');
33+
<%= expect() %>page.form.name.getAttribute('name')<%= to() %>.eventually.equal('name');
34+
<%= expect() %>page.form.email.getAttribute('type')<%= to() %>.eventually.equal('email');
35+
<%= expect() %>page.form.email.getAttribute('name')<%= to() %>.eventually.equal('email');
36+
<%= expect() %>page.form.password.getAttribute('type')<%= to() %>.eventually.equal('password');
37+
<%= expect() %>page.form.password.getAttribute('name')<%= to() %>.eventually.equal('password');
38+
<%= expect() %>page.form.submit.getAttribute('type')<%= to() %>.eventually.equal('submit');
39+
<%= expect() %>page.form.submit.getText()<%= to() %>.eventually.equal('Sign up');
4040
});
4141

4242
describe('with local auth', function() {
@@ -51,8 +51,8 @@ describe('Signup View', function() {
5151

5252
var navbar = require('../../components/navbar/navbar.po');
5353

54-
<%= does("browser.getCurrentUrl()") %>.eventually.equal(config.baseUrl + '/');
55-
<%= does("navbar.navbarAccountGreeting.getText()") %>.eventually.equal('Hello ' + testUser.name);
54+
<%= expect() %>browser.getCurrentUrl()<%= to() %>.eventually.equal(config.baseUrl + '/');
55+
<%= expect() %>navbar.navbarAccountGreeting.getText()<%= to() %>.eventually.equal('Hello ' + testUser.name);
5656
});
5757

5858
describe('and invalid credentials', function() {
@@ -63,11 +63,11 @@ describe('Signup View', function() {
6363
it('should indicate signup failures', function() {
6464
page.signup(testUser);
6565

66-
<%= does("browser.getCurrentUrl()") %>.eventually.equal(config.baseUrl + '/signup');
67-
<%= does("page.form.email.getAttribute('class')") %>.eventually.contain('ng-invalid-mongoose');
66+
<%= expect() %>browser.getCurrentUrl()<%= to() %>.eventually.equal(config.baseUrl + '/signup');
67+
<%= expect() %>page.form.email.getAttribute('class')<%= to() %>.eventually.contain('ng-invalid-mongoose');
6868

6969
var helpBlock = page.form.element(by.css('.form-group.has-error .help-block.ng-binding'));
70-
<%= does("helpBlock.getText()") %>.eventually.equal('The specified email address is already in use.');
70+
<%= expect() %>helpBlock.getText()<%= to() %>.eventually.equal('The specified email address is already in use.');
7171
});
7272

7373
});

Diff for: ‎app/templates/e2e/main/main.spec(mocha).js

+3-3
Original file line numberDiff line numberDiff line change
@@ -11,8 +11,8 @@ describe('Main View', function() {
1111
});
1212

1313
it('should include jumbotron with correct data', function() {
14-
<%= does("page.h1El.getText()") %>.eventually.equal('\'Allo, \'Allo!');
15-
<%= does("page.imgEl.getAttribute('src')") %>.eventually.match(/yeoman.png$/);
16-
<%= does("page.imgEl.getAttribute('alt')") %>.eventually.equal('I\'m Yeoman');
14+
<%= expect() %>page.h1El.getText()<%= to() %>.eventually.equal('\'Allo, \'Allo!');
15+
<%= expect() %>page.imgEl.getAttribute('src')<%= to() %>.eventually.match(/yeoman.png$/);
16+
<%= expect() %>page.imgEl.getAttribute('alt')<%= to() %>.eventually.equal('I\'m Yeoman');
1717
});
1818
});

Diff for: ‎app/templates/server/api/user(auth)/index.spec.js

+19-19
Original file line numberDiff line numberDiff line change
@@ -41,65 +41,65 @@ var userIndex = proxyquire('./index', {
4141
describe('User API Router:', function() {
4242

4343
it('should return an express router instance', function() {
44-
userIndex.should.equal(routerStub);
44+
<%= expect() %>userIndex<%= to() %>.equal(routerStub);
4545
});
4646

4747
describe('GET /api/users', function() {
4848

4949
it('should verify admin role and route to user.controller.index', function() {
50-
routerStub.get
51-
.withArgs('/', 'authService.hasRole.admin', 'userCtrl.index')
52-
.should.have.been.calledOnce;
50+
<%= expect() %>routerStub.get
51+
.withArgs('/', 'authService.hasRole.admin', 'userCtrl.index')
52+
<%= to() %>.have.been.calledOnce;
5353
});
5454

5555
});
5656

5757
describe('DELETE /api/users/:id', function() {
5858

5959
it('should verify admin role and route to user.controller.destroy', function() {
60-
routerStub.delete
61-
.withArgs('/:id', 'authService.hasRole.admin', 'userCtrl.destroy')
62-
.should.have.been.calledOnce;
60+
<%= expect() %>routerStub.delete
61+
.withArgs('/:id', 'authService.hasRole.admin', 'userCtrl.destroy')
62+
<%= to() %>.have.been.calledOnce;
6363
});
6464

6565
});
6666

6767
describe('GET /api/users/me', function() {
6868

6969
it('should be authenticated and route to user.controller.me', function() {
70-
routerStub.get
71-
.withArgs('/me', 'authService.isAuthenticated', 'userCtrl.me')
72-
.should.have.been.calledOnce;
70+
<%= expect() %>routerStub.get
71+
.withArgs('/me', 'authService.isAuthenticated', 'userCtrl.me')
72+
<%= to() %>.have.been.calledOnce;
7373
});
7474

7575
});
7676

7777
describe('PUT /api/users/:id/password', function() {
7878

7979
it('should be authenticated and route to user.controller.changePassword', function() {
80-
routerStub.put
81-
.withArgs('/:id/password', 'authService.isAuthenticated', 'userCtrl.changePassword')
82-
.should.have.been.calledOnce;
80+
<%= expect() %>routerStub.put
81+
.withArgs('/:id/password', 'authService.isAuthenticated', 'userCtrl.changePassword')
82+
<%= to() %>.have.been.calledOnce;
8383
});
8484

8585
});
8686

8787
describe('GET /api/users/:id', function() {
8888

8989
it('should be authenticated and route to user.controller.show', function() {
90-
routerStub.get
91-
.withArgs('/:id', 'authService.isAuthenticated', 'userCtrl.show')
92-
.should.have.been.calledOnce;
90+
<%= expect() %>routerStub.get
91+
.withArgs('/:id', 'authService.isAuthenticated', 'userCtrl.show')
92+
<%= to() %>.have.been.calledOnce;
9393
});
9494

9595
});
9696

9797
describe('POST /api/users', function() {
9898

9999
it('should route to user.controller.create', function() {
100-
routerStub.post
101-
.withArgs('/', 'userCtrl.create')
102-
.should.have.been.calledOnce;
100+
<%= expect() %>routerStub.post
101+
.withArgs('/', 'userCtrl.create')
102+
<%= to() %>.have.been.calledOnce;
103103
});
104104

105105
});

Diff for: ‎app/templates/server/api/user(auth)/user.integration.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@ describe('User API:', function() {
1515
<% if (filters.mongooseModels) { %>user = new User({<% }
1616
if (filters.sequelizeModels) { %>user = User.build({<% } %>
1717
name: 'Fake User',
18-
email: 'test@test.com',
18+
email: 'test@example.com',
1919
password: 'password'
2020
});
2121

@@ -37,7 +37,7 @@ describe('User API:', function() {
3737
request(app)
3838
.post('/auth/local')
3939
.send({
40-
email: 'test@test.com',
40+
email: 'test@example.com',
4141
password: 'password'
4242
})
4343
.expect(200)
@@ -55,7 +55,7 @@ describe('User API:', function() {
5555
.expect(200)
5656
.expect('Content-Type', /json/)
5757
.end(function(err, res) {
58-
res.body._id.toString().should.equal(user._id.toString());
58+
<%= expect() %>res.body._id.toString()<%= to() %>.equal(user._id.toString());
5959
done();
6060
});
6161
});

0 commit comments

Comments
 (0)