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

Special case printing -0

Browse Source 
Use the 1/x trick to determine if a value is -0 and special case the
printing of it.

[closes #496]
This commit is contained in:
Sheel Choksi
2014-02-09 14:14:33 -08:00
parent 83c0ea7f91
commit 7cf899a4ea
3 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
lib/jasmine-core/jasmine.js
View File

@@ -1354,6 +1354,8 @@ getJasmineRequireObj().pp = function(j$) {
this.emitScalar('undefined'); this.emitScalar('undefined');
} else if (value === null) { } else if (value === null) {
this.emitScalar('null'); this.emitScalar('null');
} else if (value === 0 && 1/value === -Infinity) {
this.emitScalar('-0');
} else if (value === j$.getGlobal()) { } else if (value === j$.getGlobal()) {
this.emitScalar('<global>'); this.emitScalar('<global>');
} else if (value.jasmineToString) { } else if (value.jasmineToString) {

1
spec/core/PrettyPrintSpec.js
View File

@@ -11,6 +11,7 @@ describe("j$.pp", function () {
expect(j$.pp(jasmine.undefined)).toEqual("undefined"); expect(j$.pp(jasmine.undefined)).toEqual("undefined");
expect(j$.pp(3)).toEqual("3"); expect(j$.pp(3)).toEqual("3");
expect(j$.pp(-3.14)).toEqual("-3.14"); expect(j$.pp(-3.14)).toEqual("-3.14");
expect(j$.pp(-0)).toEqual("-0");
}); });
it("should stringify arrays properly", function() { it("should stringify arrays properly", function() {

2
src/core/PrettyPrinter.js
View File

@@ -11,6 +11,8 @@ getJasmineRequireObj().pp = function(j$) {
this.emitScalar('undefined'); this.emitScalar('undefined');
} else if (value === null) { } else if (value === null) {
this.emitScalar('null'); this.emitScalar('null');
} else if (value === 0 && 1/value === -Infinity) {
this.emitScalar('-0');
} else if (value === j$.getGlobal()) { } else if (value === j$.getGlobal()) {
this.emitScalar('<global>'); this.emitScalar('<global>');
} else if (value.jasmineToString) { } else if (value.jasmineToString) {