fix linting errors

This commit is contained in:
Esco Obong 2015-03-13 18:30:53 -04:00
parent c764c09837
commit 551261ec16
2 changed files with 13 additions and 13 deletions

View File

@ -63,7 +63,7 @@ module.exports = function(Role) {
*/ */
Role.prototype[pluralName] = function(query, callback) { Role.prototype[pluralName] = function(query, callback) {
listByPrincipalType(model, principalType, query, callback); listByPrincipalType(model, principalType, query, callback);
} };
}); });
/** /**

View File

@ -210,14 +210,14 @@ describe('role model', function() {
}); });
describe('listByPrincipalType', function(){ describe('listByPrincipalType', function() {
var sandbox; var sandbox;
beforeEach(function(){ beforeEach(function() {
sandbox = sinon.sandbox.create(); sandbox = sinon.sandbox.create();
}); });
afterEach(function(){ afterEach(function() {
sandbox.restore(); sandbox.restore();
}); });
@ -232,13 +232,13 @@ describe('role model', function() {
mappings = Object.keys(principalTypesToModels); mappings = Object.keys(principalTypesToModels);
mappings.forEach(function(principalType){ mappings.forEach(function(principalType) {
var Model = principalTypesToModels[principalType]; var Model = principalTypesToModels[principalType];
Model.create({name:"test", email:'x@y.com', password: 'foobar'}, function(err, model){ Model.create({name:'test', email:'x@y.com', password: 'foobar'}, function(err, model) {
Role.create({name:'testRole'}, function(err, role){ Role.create({name:'testRole'}, function(err, role) {
role.principals.create({principalType: principalType, principalId: model.id}, function(err, p){ role.principals.create({principalType: principalType, principalId: model.id}, function(err, p) {
var pluralName = Model.pluralModelName.toLowerCase(); var pluralName = Model.pluralModelName.toLowerCase();
role[pluralName](function(err, models){ role[pluralName](function(err, models) {
assert(!err); assert(!err);
assert.equal(models.length, 1); assert.equal(models.length, 1);
if (++runs === mappings.length) { if (++runs === mappings.length) {
@ -246,8 +246,8 @@ describe('role model', function() {
} }
}); });
}); });
}) });
}) });
}); });
}); });
@ -255,7 +255,7 @@ describe('role model', function() {
User.create({name: 'Raymond', email: 'x@y.com', password: 'foobar'}, function(err, user) { User.create({name: 'Raymond', email: 'x@y.com', password: 'foobar'}, function(err, user) {
Role.create({name: 'userRole'}, function(err, role) { Role.create({name: 'userRole'}, function(err, role) {
role.principals.create({principalType: RoleMapping.USER, principalId: user.id}, function(err, p) { role.principals.create({principalType: RoleMapping.USER, principalId: user.id}, function(err, p) {
var query = {fields:['id','name']}; var query = {fields:['id', 'name']};
sandbox.spy(User, 'find'); sandbox.spy(User, 'find');
role.users(query, function(err, users) { role.users(query, function(err, users) {
assert(!err); assert(!err);
@ -268,6 +268,6 @@ describe('role model', function() {
}); });
}); });
}); });
}) });
}); });