Skip to content

Commit

Permalink
test: remove Object.observe from tests
Browse files Browse the repository at this point in the history
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 <cjihrig@gmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
  • Loading branch information
vkurchatkin authored and rvagg committed Feb 8, 2016
1 parent e65d363 commit f7ed473
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 14 deletions.
7 changes: 0 additions & 7 deletions test/parallel/test-microtask-queue-integration-domain.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,13 +12,6 @@ require('domain');
var implementations = [
function(fn) {
Promise.resolve().then(fn);
},
function(fn) {
var obj = {};

Object.observe(obj, fn);

obj.a = 1;
}
];

Expand Down
7 changes: 0 additions & 7 deletions test/parallel/test-microtask-queue-integration.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,13 +5,6 @@ var assert = require('assert');
var implementations = [
function(fn) {
Promise.resolve().then(fn);
},
function(fn) {
var obj = {};

Object.observe(obj, fn);

obj.a = 1;
}
];

Expand Down

0 comments on commit f7ed473

Please sign in to comment.