Skip to content

Commit f931f12

Browse files
Limited double wrapping, added skipped -0 test case
1 parent 65f4263 commit f931f12

File tree

1 file changed

+12
-4
lines changed

1 file changed

+12
-4
lines changed

test/node/double_tests.js

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -77,16 +77,24 @@ describe('Double', function () {
7777

7878
it('0', () => {
7979
let value = 0;
80-
let orig = new Double(value).valueOf();
80+
let orig = new Double(value);
8181
let newVal = twiceSerialized(orig);
82-
expect(orig).to.equal(newVal);
82+
expect(value).to.equal(newVal);
8383
});
8484

8585
it('-0', () => {
8686
let value = -0;
87-
let orig = new Double(value).valueOf();
87+
let orig = new Double(value);
8888
let newVal = twiceSerialized(orig);
89-
expect(orig).to.equal(newVal);
89+
expect(Object.is(newVal, -0)).to.be.true;
90+
});
91+
92+
// TODO (NODE-4335): -0 should be serialized as double
93+
it.skip('-0 serializes as Double', () => {
94+
let value = -0;
95+
let serializedDouble = BSON.serialize({ d: value });
96+
let type = serializedDouble[5];
97+
expect(type).to.not.equal(BSON.BSON_DATA_NUMBER);
9098
});
9199

92100
it('Number.EPSILON', () => {

0 commit comments

Comments
 (0)