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 Jan 28, 2016
1 parent 8ab4eac commit 959e051
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 959e051

Please sign in to comment.