test: rearrange inspector headers into convention

Test guide describes a conventional layout for test headers, review inspector tests and reorganize to follow the convention. PR-URL: #13428 Reviewed-By: Refael Ackermann <[email protected]> Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Gibson Fahnestock <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: James M Snell <[email protected]>
nodejs · Jun 5, 2017 · 339d220 · 339d220
1 parent 6aeb555
commit 339d220
Show file tree

Hide file tree

Showing 8 changed files with 16 additions and 5 deletions.
diff --git a/test/inspector/test-inspector-debug-brk.js b/test/inspector/test-inspector-debug-brk.js
@@ -1,6 +1,8 @@
 'use strict';
 const common = require('../common');
+
 common.skipIfInspectorDisabled();
+
 const assert = require('assert');
 const helper = require('./inspector-helper.js');
 

diff --git a/test/inspector/test-inspector-ip-detection.js b/test/inspector/test-inspector-ip-detection.js
@@ -1,5 +1,6 @@
 'use strict';
 const common = require('../common');
+
 common.skipIfInspectorDisabled();
 
 const assert = require('assert');

diff --git a/test/inspector/test-inspector-port-zero-cluster.js b/test/inspector/test-inspector-port-zero-cluster.js
@@ -1,8 +1,9 @@
 // Flags: --inspect=0
 'use strict';
-
 const common = require('../common');
+
 common.skipIfInspectorDisabled();
+
 const assert = require('assert');
 const cluster = require('cluster');
 

diff --git a/test/inspector/test-inspector-port-zero.js b/test/inspector/test-inspector-port-zero.js
@@ -1,7 +1,8 @@
 'use strict';
-
 const { mustCall, skipIfInspectorDisabled } = require('../common');
+
 skipIfInspectorDisabled();
+
 const assert = require('assert');
 const { URL } = require('url');
 const { spawn } = require('child_process');

diff --git a/test/inspector/test-inspector-stops-no-file.js b/test/inspector/test-inspector-stops-no-file.js
@@ -1,5 +1,6 @@
 'use strict';
 require('../common');
+
 const spawn = require('child_process').spawn;
 
 const child = spawn(process.execPath,

diff --git a/test/inspector/test-inspector.js b/test/inspector/test-inspector.js
@@ -1,6 +1,8 @@
 'use strict';
 const common = require('../common');
+
 common.skipIfInspectorDisabled();
+
 const assert = require('assert');
 const helper = require('./inspector-helper.js');
 

diff --git a/test/parallel/test-cluster-inspector-debug-port.js b/test/parallel/test-cluster-inspector-debug-port.js
@@ -1,7 +1,9 @@
 'use strict';
 // Flags: --inspect={PORT}
 const common = require('../common');
+
 common.skipIfInspectorDisabled();
+
 const assert = require('assert');
 const cluster = require('cluster');
 const debuggerPort = common.PORT;

diff --git a/test/parallel/test-inspector-invalid-args.js b/test/parallel/test-inspector-invalid-args.js
@@ -1,11 +1,12 @@
 'use strict';
+const common = require('../common');
+
+common.skipIfInspectorDisabled();
+
 const assert = require('assert');
 const execFile = require('child_process').execFile;
 const path = require('path');
 
-const common = require('../common');
-common.skipIfInspectorDisabled();
-
 const mainScript = path.join(common.fixturesDir, 'loop.js');
 const expected =
   '`node --debug` and `node --debug-brk` are invalid. ' +