warren/jasmine
1
0
Fork 0
Code Issues Pull Requests Actions 9 Packages Projects Releases Wiki Activity

Fixed SPEC HAS NO EXPECTATIONS warning in HTML reporter

Browse Source 
[Finishes #153891435]
This commit is contained in:
Steve Gravrock
2017-12-23 11:14:37 -08:00
parent b439e2fb37
commit b943f6b736
3 changed files with 6 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
lib/jasmine-core/jasmine-html.js
View File

@@ -54,10 +54,11 @@ jasmineRequire.HtmlReporter = function(j$) {
};
ResultsStateBuilder.prototype.specStarted = function(result) {
this.currentParent.addChild(result, 'spec');
};
ResultsStateBuilder.prototype.specDone = function(result) {
this.currentParent.addChild(result, 'spec');
if (result.status !== 'disabled') {
this.specsExecuted++;
}

4
spec/html/HtmlReporterSpec.js
View File

@@ -209,7 +209,7 @@ describe("New HtmlReporter", function() {
});
describe("when Jasmine is done", function() {
it("adds EMPTY to the link title of specs that have no expectations", function() {
it("adds a warning to the link title of specs that have no expectations", function() {
if (!window.console) {
window.console = { error: function(){} };
}
@@ -228,7 +228,7 @@ describe("New HtmlReporter", function() {
reporter.initialize();
reporter.jasmineStarted({});
reporter.suiteStarted({id: 1});
reporter.specStarted({id: 1, status: 'passed', passedExpectations: [], failedExpectations: []});
reporter.specStarted({id: 1, passedExpectations: [], failedExpectations: []});
reporter.specDone({
id: 1,
status: 'passed',

3
src/html/HtmlReporter.js
View File

@@ -25,10 +25,11 @@ jasmineRequire.HtmlReporter = function(j$) {
};
ResultsStateBuilder.prototype.specStarted = function(result) {
this.currentParent.addChild(result, 'spec');
};
ResultsStateBuilder.prototype.specDone = function(result) {
this.currentParent.addChild(result, 'spec');
if (result.status !== 'disabled') {
this.specsExecuted++;
}