test: move tick-processor tests to own directory

Makefile

@@ -241,6 +241,9 @@ test-debugger: all
 test-inspector: all
 	$(PYTHON) tools/test.py inspector
 
+test-tick-processor: all
+	$(PYTHON) tools/test.py tick-processor
+
 test-known-issues: all
 	$(PYTHON) tools/test.py known_issues
 

test/README.md

@@ -122,6 +122,17 @@ Various tests that are run sequentially.
 
 Test configuration utility used by various test suites.
 
+### inspector
+
+Tests for the V8 tick processor integration. The tests are for the logic in
+`lib/internal/v8_prof_processor.js` and `lib/internal/v8_prof_polyfill.js`. The
+tests confirm that the profile processor packages the correct set of scripts
+from V8 and introduces the correct platform specific logic.
+
+| Runs on CI |
+|:----------:|
+| No         |
+
 ### timers
 
 Tests for [timing utilities](https://nodejs.org/api/timers.html) (`setTimeout`

test/parallel/parallel.status

@@ -15,7 +15,6 @@ prefix parallel
 [$arch==arm || $arch==arm64]
 
 [$system==solaris] # Also applies to SmartOS
-test-tick-processor-unknown: PASS,FLAKY
 
 [$system==freebsd]
 

test/parallel/test-tick-processor-unknown.js → test/tick-processor/test-tick-processor-unknown.js

@@ -2,12 +2,12 @@
 const common = require('../common');
 
 // TODO(mhdawson) Currently the test-tick-processor functionality in V8
-// depends on addresses being smaller than a full 64 bits.  Aix supports
+// depends on addresses being smaller than a full 64 bits.  AIX supports
 // the full 64 bits and the result is that it does not process the
 // addresses correctly and runs out of memory
 // Disabling until we get a fix upstreamed into V8
 if (common.isAix) {
-  common.skip('Aix address range too big for scripts.');
+  common.skip('AIX address range too big for scripts.');
   return;
 }
 
@@ -21,7 +21,7 @@ const base = require('./tick-processor-base.js');
 // Unknown checked for to prevent flakiness, if pattern is not found,
 // then a large number of unknown ticks should be present
 base.runTest({
-  pattern: /LazyCompile.*\[eval\]:1|.*%  UNKNOWN/,
+  pattern: /LazyCompile.*\[eval]:1|.*%  UNKNOWN/,
   code: `function f() {
            for (var i = 0; i < 1000000; i++) {
              i++;

test/tick-processor/testcfg.py

@@ -0,0 +1,6 @@
+import sys, os
+sys.path.append(os.path.join(os.path.dirname(__file__), '..'))
+import testpy
+
+def GetConfiguration(context, root):
+  return testpy.SimpleTestConfiguration(context, root, 'tick-processor')

vcbuild.bat

@@ -63,6 +63,7 @@ if /i "%1"=="test-simple"   set test_args=%test_args% sequential parallel -J&got
 if /i "%1"=="test-message"  set test_args=%test_args% message&goto arg-ok
 if /i "%1"=="test-gc"       set test_args=%test_args% gc&set buildnodeweak=1&goto arg-ok
 if /i "%1"=="test-inspector" set test_args=%test_args% inspector&goto arg-ok
+if /i "%1"=="test-tick-processor" set test_args=%test_args% tick-processor&goto arg-ok
 if /i "%1"=="test-internet" set test_args=%test_args% internet&goto arg-ok
 if /i "%1"=="test-pummel"   set test_args=%test_args% pummel&goto arg-ok
 if /i "%1"=="test-all"      set test_args=%test_args% sequential parallel message gc inspector internet pummel&set buildnodeweak=1&set jslint=1&goto arg-ok