Browse Source

Add clearer assertion of process being undefined

Kevin Sawicki 9 years ago
parent
commit
9e66df23d0
2 changed files with 3 additions and 3 deletions
  1. 2 2
      spec/chromium-spec.js
  2. 1 1
      spec/fixtures/pages/window-opener-node.html

+ 2 - 2
spec/chromium-spec.js

@@ -160,7 +160,7 @@ describe('chromium feature', function () {
     it('accepts "nodeIntegration" as feature', function (done) {
       var b
       listener = function (event) {
-        assert.equal(event.data, 'undefined')
+        assert.equal(event.data.isProcessGlobalUndefined, true)
         b.close()
         done()
       }
@@ -185,7 +185,7 @@ describe('chromium feature', function () {
     it('disables node integration when it is disabled on the parent window', function (done) {
       var b
       listener = function (event) {
-        assert.equal(event.data, 'undefined')
+        assert.equal(event.data.isProcessGlobalUndefined, true)
         b.close()
         done()
       }

+ 1 - 1
spec/fixtures/pages/window-opener-node.html

@@ -1,7 +1,7 @@
 <html>
 <body>
 <script type="text/javascript" charset="utf-8">
-  window.opener.postMessage(typeof process, '*')
+  window.opener.postMessage({isProcessGlobalUndefined: typeof process === 'undefined'}, '*')
 </script>
 </body>
 </html>