m-labs · Feb 2, 2016 · Feb 2, 2016 · Feb 2, 2016
diff --git a/artiq/compiler/testbench/jit.py b/artiq/compiler/testbench/jit.py
@@ -5,7 +5,7 @@
 from ..targets import NativeTarget
 
 def main():
-    libartiq_support = os.getenv('LIBARTIQ_SUPPORT')
+    libartiq_support = os.getenv("LIBARTIQ_SUPPORT")
     if libartiq_support is not None:
         llvm.load_library_permanently(libartiq_support)
 

diff --git a/artiq/test/compiler/domination.py → artiq/test/compiler/test_domination.py b/artiq/test/compiler/domination.py → artiq/test/compiler/test_domination.py
diff --git a/artiq/test/coredevice/analyzer.py → artiq/test/coredevice/test_analyzer.py b/artiq/test/coredevice/analyzer.py → artiq/test/coredevice/test_analyzer.py
diff --git a/artiq/test/coredevice/cache.py → artiq/test/coredevice/test_cache.py b/artiq/test/coredevice/cache.py → artiq/test/coredevice/test_cache.py
diff --git a/artiq/test/coredevice/embedding.py → artiq/test/coredevice/test_embedding.py b/artiq/test/coredevice/embedding.py → artiq/test/coredevice/test_embedding.py
diff --git a/artiq/test/coredevice/portability.py → artiq/test/coredevice/test_portability.py b/artiq/test/coredevice/portability.py → artiq/test/coredevice/test_portability.py
diff --git a/artiq/test/coredevice/rtio.py → artiq/test/coredevice/test_rtio.py b/artiq/test/coredevice/rtio.py → artiq/test/coredevice/test_rtio.py
diff --git a/artiq/test/harness.py b/artiq/test/harness.py
diff --git a/artiq/test/lit/lit.cfg b/artiq/test/lit/lit.cfg
@@ -1,35 +1,35 @@
 # -*- python -*-
 
-import os, subprocess
+import os
+import sys
+import subprocess
 import lit.util
 import lit.formats
 
-root = os.path.join(os.path.dirname(__file__), '..')
+root = os.path.join(os.path.dirname(__file__), "..")
 
-config.name = 'ARTIQ'
+config.name = "ARTIQ"
 config.test_format = lit.formats.ShTest()
-config.suffixes = ['.py']
+config.suffixes = [".py"]
 
-python_executable = 'python3.5'
-
-harness = os.path.join(root, 'harness.py')
-if os.getenv('COVERAGE'):
-    harness = 'coverage run --parallel-mode --source=artiq {}'.format(harness)
-    config.environment['COVERAGE_FILE'] = os.path.join(root, '..', '.coverage')
-    config.substitutions.append( ('%python', harness) )
+if os.getenv("COVERAGE"):
+    config.environment["COVERAGE_FILE"] = os.path.join(root, "..", ".coverage")
+    python = "coverage run --parallel-mode --source=artiq"
 else:
-    harness = '{} {}'.format(python_executable, harness)
-    config.substitutions.append( ('%python', harness) )
+    python = sys.executable
+config.substitutions.append( ("%python", python) )
+
+config.environment["PYTHONPATH"] = os.getenv("PYTHONPATH")
 
-not_ = '{} {}'.format(python_executable, os.path.join(root, 'not.py'))
-config.substitutions.append( ('%not', not_) )
+not_ = "{} {}".format(sys.executable, os.path.join(root, "not.py"))
+config.substitutions.append( ("%not", not_) )
 
-if os.name == 'posix':
-    support_build = os.path.join(root, 'libartiq_support')
-    if subprocess.call(['make', '-sC', support_build]) != 0:
+if os.name == "posix":
+    support_build = os.path.join(root, "libartiq_support")
+    if subprocess.call(["make", "-sC", support_build]) != 0:
         lit_config.fatal("Unable to build JIT support library")
 
-    support_lib = os.path.join(support_build, 'libartiq_support.so')
-    config.environment['LIBARTIQ_SUPPORT'] = support_lib
+    support_lib = os.path.join(support_build, "libartiq_support.so")
+    config.environment["LIBARTIQ_SUPPORT"] = support_lib
 
-    config.available_features.add('exceptions')
+    config.available_features.add("exceptions")
diff --git a/artiq/test/coefficients.py → artiq/test/test_coefficients.py b/artiq/test/coefficients.py → artiq/test/test_coefficients.py
diff --git a/artiq/test/ctlmgr.py → artiq/test/test_ctlmgr.py b/artiq/test/ctlmgr.py → artiq/test/test_ctlmgr.py
@@ -1,4 +1,5 @@
 import os
+import sys
 import unittest
 import logging
 import asyncio
@@ -55,12 +56,13 @@ async def wait_for_ping(self, host, port, retries=5, timeout=2):
         raise asyncio.TimeoutError
 
     def test_start_ping_stop_controller(self):
+        command = sys.executable + " -m "
         entry = {
             "type": "controller",
             "host": "::1",
             "port": 3253,
-            "command": "lda_controller -p {port} --bind {bind} "
-                    "--no-localhost-bind --simulation",
+            "command": sys.executable + " -m artiq.frontend.lda_controller "
+                    "-p {port} --simulation"
         }
         async def test():
             await self.start("lda_sim", entry)

diff --git a/artiq/test/h5types.py → artiq/test/test_h5types.py b/artiq/test/h5types.py → artiq/test/test_h5types.py
diff --git a/artiq/test/language.py → artiq/test/test_language.py b/artiq/test/language.py → artiq/test/test_language.py
diff --git a/artiq/test/lda.py → artiq/test/test_lda.py b/artiq/test/lda.py → artiq/test/test_lda.py
diff --git a/artiq/test/novatech409b.py → artiq/test/test_novatech409b.py b/artiq/test/novatech409b.py → artiq/test/test_novatech409b.py
diff --git a/artiq/test/pc_rpc.py → artiq/test/test_pc_rpc.py b/artiq/test/pc_rpc.py → artiq/test/test_pc_rpc.py
diff --git a/artiq/test/pdq2.py → artiq/test/test_pdq2.py b/artiq/test/pdq2.py → artiq/test/test_pdq2.py
diff --git a/artiq/test/pipe_ipc.py → artiq/test/test_pipe_ipc.py b/artiq/test/pipe_ipc.py → artiq/test/test_pipe_ipc.py
diff --git a/artiq/test/scheduler.py → artiq/test/test_scheduler.py b/artiq/test/scheduler.py → artiq/test/test_scheduler.py
diff --git a/artiq/test/serialization.py → artiq/test/test_serialization.py b/artiq/test/serialization.py → artiq/test/test_serialization.py
diff --git a/artiq/test/sync_struct.py → artiq/test/test_sync_struct.py b/artiq/test/sync_struct.py → artiq/test/test_sync_struct.py
diff --git a/artiq/test/thorlabs_tcube.py → artiq/test/test_thorlabs_tcube.py b/artiq/test/thorlabs_tcube.py → artiq/test/test_thorlabs_tcube.py
diff --git a/artiq/test/wavesynth.py → artiq/test/test_wavesynth.py b/artiq/test/wavesynth.py → artiq/test/test_wavesynth.py
diff --git a/artiq/test/worker.py → artiq/test/test_worker.py b/artiq/test/worker.py → artiq/test/test_worker.py
diff --git a/doc/manual/faq.rst b/doc/manual/faq.rst
@@ -86,3 +86,8 @@ URL: it allows to select the serial device by its USB vendor ID, product
 ID and/or serial number. Those never change, unlike the device file name.
 
 See the :ref:`TDC001 documentation <tdc001-controller-usage-example>` for an example of ``hwgrep://`` usage.
+
+run unit tests?
+---------------
+
+The unit tests assume that the Python environment has been set up in such a way that ``import artiq`` will import the code being tested, and that this is still true for any subprocess created. This is not the way setuptools operates as it adds the path to ARTIQ to ``sys.path`` which is not passed to subprocesses; as a result, running the tests via ``setup.py`` is not supported. The user must first install the package or set ``PYTHONPATH``, and then run the tests with e.g. ``python3.5 -m unittest discover`` in the ``artiq/test`` folder and ``lit .`` in the ``artiq/test/lit`` folder.
diff --git a/setup.py b/setup.py
@@ -57,7 +57,6 @@
     ],
     packages=find_packages(),
     namespace_packages=[],
-    test_suite="artiq.test",
     include_package_data=True,
     ext_modules=[],
     entry_points={