test: test configure ninja

- Updated the tooltest target to run unittest module - Renamed test/tools/test-js2c.py to be discoverable by unittest module - Added test class for `configure` shell script - Added a test to ensure `configure` script exits with status code zero when passed the `--ninja` flag Closes #29415
patrickhousley · Oct 18, 2019 · Oct 25, 2019 · 08b5e133ec2eaa09130e5a088b652b3031170d44
commit 08b5e133ec2eaa09130e5a088b652b3031170d44
diff --git a/Makefile b/Makefile
@@ -300,7 +300,7 @@ jstest: build-addons build-js-native-api-tests build-node-api-tests ## Runs addo
 
 .PHONY: tooltest
 tooltest:
-	@$(PYTHON) test/tools/test-js2c.py
+	@$(PYTHON) -m unittest discover -s ./test/tools
 
 .PHONY: coverage-run-js
 coverage-run-js:

diff --git a/test/tools/test_configure.py b/test/tools/test_configure.py
@@ -0,0 +1,29 @@
+import sys
+import os
+import unittest
+import subprocess
+
+
+class ConfigureTests(unittest.TestCase):
+  def setUp(self):
+    self.working_dir = os.path.abspath(
+      os.path.join(
+        os.path.dirname(__file__),
+        '..', '..'
+      )
+    )
+
+  def test_ninja(self):
+    subprocess.check_call(
+      './configure --ninja',
+      cwd=self.working_dir,
+      shell=True,
+      stdout=subprocess.PIPE,
+      stderr=subprocess.PIPE
+    )
+
+
+if (__name__ == '__main__' and
+    sys.platform in ['linux', 'linux2', 'darwin', 'cygwin']):
+
+  unittest.main()
diff --git a/test/tools/test-js2c.py → test/tools/test_js2c.py b/test/tools/test-js2c.py → test/tools/test_js2c.py