Skip to content

Commit c490464

Browse files
vkurchatkinMyles Borins
authored and
Myles Borins
committed
test: remove Object.observe from tests
Testing this wasn't really useful, besides Object.observe is going to be deprecated. Also this test fails with Chakra (#4765) for obvious reason. PR-URL: #4769 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Brian White <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
1 parent d6894f8 commit c490464

File tree

2 files changed

+0
-14
lines changed

2 files changed

+0
-14
lines changed

test/parallel/test-microtask-queue-integration-domain.js

-7
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,6 @@ require('domain');
1212
var implementations = [
1313
function(fn) {
1414
Promise.resolve().then(fn);
15-
},
16-
function(fn) {
17-
var obj = {};
18-
19-
Object.observe(obj, fn);
20-
21-
obj.a = 1;
2215
}
2316
];
2417

test/parallel/test-microtask-queue-integration.js

-7
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,6 @@ var assert = require('assert');
55
var implementations = [
66
function(fn) {
77
Promise.resolve().then(fn);
8-
},
9-
function(fn) {
10-
var obj = {};
11-
12-
Object.observe(obj, fn);
13-
14-
obj.a = 1;
158
}
169
];
1710

0 commit comments

Comments
 (0)