Skip to content

Commit

Permalink
added mixed case user logins for all lintIssues tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Alshie committed Sep 10, 2015
1 parent d82e07f commit c72cd7a
Showing 1 changed file with 38 additions and 2 deletions.
40 changes: 38 additions & 2 deletions spec/lintIssues.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,15 @@ describe('lintIssues', function () {
labels: ['a label'],
title: 'This issue has no comments',
html_url: 'This is a URL'
},
{
user: {
login: 'jimBOB'
},
comments: 0,
labels: ['a label'],
title: 'This issue has no comments',
html_url: 'This is a URL'
}
];

Expand All @@ -25,7 +34,7 @@ describe('lintIssues', function () {
}
});

report.failures.length.should.eql(1);
report.failures.length.should.eql(2);
});

it('should return no warnings if all issues are commented upon', function () {
Expand All @@ -38,6 +47,15 @@ describe('lintIssues', function () {
labels: [],
title: 'This issue has no comments',
html_url: 'This is a URL'
},
{
user: {
login: 'jimBOB'
},
comments: 1,
labels: [],
title: 'This issue has no comments',
html_url: 'This is a URL'
}
];

Expand Down Expand Up @@ -96,6 +114,15 @@ describe('lintIssues', function () {
labels: ['alabel'],
title: 'This issue has no comments',
html_url: 'This is a URL'
},
{
user: {
login: 'jimBOB'
},
comments: 1,
labels: ['alabel'],
title: 'This issue has no comments',
html_url: 'This is a URL'
}
];

Expand All @@ -118,6 +145,15 @@ describe('lintIssues', function () {
labels: [],
title: 'This issue has no comments or labels',
html_url: 'This is a URL'
},
{
user: {
login: 'jimBOB'
},
comments: 0,
labels: [],
title: 'This issue has no comments or labels',
html_url: 'This is a URL'
}
];

Expand All @@ -141,6 +177,6 @@ describe('lintIssues', function () {
}
});

report.failures.length.should.eql(2);
report.failures.length.should.eql(4);
});
});

0 comments on commit c72cd7a

Please sign in to comment.