test: fix scriptParsed event expectations

As per Node.js docs, vm.Script instance is not bound to any context. However, this test was expecting otherwise and depended on implementation details which are going to change. Refs: https://chromium-review.googlesource.com/c/v8/v8/+/1013581 Backport-PR-URL: #21668 PR-URL: #21079 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Matteo Collina <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Yang Guo <[email protected]>

Author: RReverser

test/sequential/test-inspector-scriptparsed-context.js

@@ -9,27 +9,17 @@ const script = `
  'use strict'
  const assert = require('assert');
  const vm = require('vm');
- const{kParsingContext } = process.binding('contextify');
  global.outer = true;
  global.inner = false;
  const context = vm.createContext({
  outer: false,
  inner: true
  });
+ const script = new vm.Script("outer");
  debugger;
 
- const scriptMain = new vm.Script("outer");
- debugger;
-
- const scriptContext = new vm.Script("inner",{
- [kParsingContext]: context
- });
- debugger;
-
- assert.strictEqual(scriptMain.runInThisContext(), true);
- assert.strictEqual(scriptMain.runInContext(context), false);
- assert.strictEqual(scriptContext.runInThisContext(), false);
- assert.strictEqual(scriptContext.runInContext(context), true);
+ assert.strictEqual(script.runInThisContext(), true);
+ assert.strictEqual(script.runInContext(context), false);
  debugger;
 
  vm.runInContext('inner', context);
@@ -64,35 +54,25 @@ async function runTests(){
 awaitsession.waitForBreakOnLine(0,'[eval]');
 
 awaitsession.send({'method': 'Runtime.enable'});
-consttopContext=awaitgetContext(session);
+awaitgetContext(session);
 awaitsession.send({'method': 'Debugger.resume'});
 constchildContext=awaitgetContext(session);
 awaitsession.waitForBreakOnLine(13,'[eval]');
 
-console.error('[test]','Script associated with current context by default');
-awaitsession.send({'method': 'Debugger.resume'});
-awaitcheckScriptContext(session,topContext);
-awaitsession.waitForBreakOnLine(16,'[eval]');
-
-console.error('[test]','Script associated with selected context');
-awaitsession.send({'method': 'Debugger.resume'});
-awaitcheckScriptContext(session,childContext);
-awaitsession.waitForBreakOnLine(21,'[eval]');
-
 console.error('[test]','Script is unbound');
 awaitsession.send({'method': 'Debugger.resume'});
-awaitsession.waitForBreakOnLine(27,'[eval]');
+awaitsession.waitForBreakOnLine(17,'[eval]');
 
 console.error('[test]','vm.runInContext associates script with context');
 awaitsession.send({'method': 'Debugger.resume'});
 awaitcheckScriptContext(session,childContext);
-awaitsession.waitForBreakOnLine(30,'[eval]');
+awaitsession.waitForBreakOnLine(20,'[eval]');
 
 console.error('[test]','vm.runInNewContext associates script with context');
 awaitsession.send({'method': 'Debugger.resume'});
 constthirdContext=awaitgetContext(session);
 awaitcheckScriptContext(session,thirdContext);
-awaitsession.waitForBreakOnLine(33,'[eval]');
+awaitsession.waitForBreakOnLine(23,'[eval]');
 
 console.error('[test]','vm.runInNewContext can contain debugger statements');
 awaitsession.send({'method': 'Debugger.resume'});