21207: Remove Python slow/short tests support

After the previous commit there are no more users of this, and this
interface will no longer be supported after our migration to pytest. If
we want this functionality in the future we should set up a custom
"slow" mark in pytest and use pytest's built-in mark filtering.

Arvados-DCO-1.1-Signed-off-by: Brett Smith <brett.smith@curii.com>

diff --git a/sdk/cwl/arvados_version.py b/sdk/cwl/arvados_version.py
index 794b6afe4261cba9c6bfc4c5dd3fee9d6bb6c19b..ca20de4c5813d4d0a9bf7dc6a1e74ce7ac68c43c 100644 (file)
--- a/sdk/cwl/arvados_version.py
+++ b/sdk/cwl/arvados_version.py
@@ -72,14 +72,6 @@ if REPO_PATH is None:
         if (SETUP_DIR / mod_name).is_dir()
     )
 
-def short_tests_only(arglist=sys.argv):
-    try:
-        arglist.remove('--short-tests-only')
-    except ValueError:
-        return False
-    else:
-        return True
-
 def git_log_output(path, *args):
     return subprocess.check_output(
         ['git', '-C', str(REPO_PATH),

diff --git a/sdk/python/arvados_version.py b/sdk/python/arvados_version.py
index 794b6afe4261cba9c6bfc4c5dd3fee9d6bb6c19b..ca20de4c5813d4d0a9bf7dc6a1e74ce7ac68c43c 100644 (file)
--- a/sdk/python/arvados_version.py
+++ b/sdk/python/arvados_version.py
@@ -72,14 +72,6 @@ if REPO_PATH is None:
         if (SETUP_DIR / mod_name).is_dir()
     )
 
-def short_tests_only(arglist=sys.argv):
-    try:
-        arglist.remove('--short-tests-only')
-    except ValueError:
-        return False
-    else:
-        return True
-
 def git_log_output(path, *args):
     return subprocess.check_output(
         ['git', '-C', str(REPO_PATH),

diff --git a/sdk/python/setup.py b/sdk/python/setup.py
index 257968d89eb069b6e88868523b92bc81ef0f0cc3..dc1026906087f040dfdf258b1f57894b48b0b8c2 100644 (file)
--- a/sdk/python/setup.py
+++ b/sdk/python/setup.py
@@ -13,7 +13,6 @@ from setuptools.command import build_py
 
 import arvados_version
 version = arvados_version.get_version()
-short_tests_only = arvados_version.short_tests_only()
 README = os.path.join(arvados_version.SETUP_DIR, 'README.rst')
 
 class BuildPython(build_py.build_py):

diff --git a/sdk/python/tests/slow_test.py b/sdk/python/tests/slow_test.py
deleted file mode 100644 (file)
index ae46f4e..0000000
--- a/sdk/python/tests/slow_test.py
+++ /dev/null
@@ -1,11 +0,0 @@
-# Copyright (C) The Arvados Authors. All rights reserved.
-#
-# SPDX-License-Identifier: Apache-2.0
-
-import __main__
-import os
-import unittest
-
-slow_test = lambda _: unittest.skipIf(
-    __main__.short_tests_only,
-    "running --short tests only")

diff --git a/services/dockercleaner/arvados_version.py b/services/dockercleaner/arvados_version.py
index 794b6afe4261cba9c6bfc4c5dd3fee9d6bb6c19b..ca20de4c5813d4d0a9bf7dc6a1e74ce7ac68c43c 100644 (file)
--- a/services/dockercleaner/arvados_version.py
+++ b/services/dockercleaner/arvados_version.py
@@ -72,14 +72,6 @@ if REPO_PATH is None:
         if (SETUP_DIR / mod_name).is_dir()
     )
 
-def short_tests_only(arglist=sys.argv):
-    try:
-        arglist.remove('--short-tests-only')
-    except ValueError:
-        return False
-    else:
-        return True
-
 def git_log_output(path, *args):
     return subprocess.check_output(
         ['git', '-C', str(REPO_PATH),

diff --git a/services/dockercleaner/setup.py b/services/dockercleaner/setup.py
index b20b776ebf491088bcba550186f6affbbf17b359..cb0fc8d88afae0b7c688aeec4f73a548716b39a1 100644 (file)
--- a/services/dockercleaner/setup.py
+++ b/services/dockercleaner/setup.py
@@ -11,7 +11,6 @@ from setuptools import setup, find_packages
 
 import arvados_version
 version = arvados_version.get_version()
-short_tests_only = arvados_version.short_tests_only()
 README = os.path.join(arvados_version.SETUP_DIR, 'README.rst')
 
 setup(name="arvados-docker-cleaner",

diff --git a/services/fuse/arvados_version.py b/services/fuse/arvados_version.py
index 794b6afe4261cba9c6bfc4c5dd3fee9d6bb6c19b..ca20de4c5813d4d0a9bf7dc6a1e74ce7ac68c43c 100644 (file)
--- a/services/fuse/arvados_version.py
+++ b/services/fuse/arvados_version.py
@@ -72,14 +72,6 @@ if REPO_PATH is None:
         if (SETUP_DIR / mod_name).is_dir()
     )
 
-def short_tests_only(arglist=sys.argv):
-    try:
-        arglist.remove('--short-tests-only')
-    except ValueError:
-        return False
-    else:
-        return True
-
 def git_log_output(path, *args):
     return subprocess.check_output(
         ['git', '-C', str(REPO_PATH),

diff --git a/services/fuse/setup.py b/services/fuse/setup.py
index 002afc22c99a13b1d601e0615884707c70a8ad6a..f9b0fcf91dcb614c9224fefe8b4836cbcffd74cd 100644 (file)
--- a/services/fuse/setup.py
+++ b/services/fuse/setup.py
@@ -11,7 +11,6 @@ from setuptools import setup, find_packages
 
 import arvados_version
 version = arvados_version.get_version()
-short_tests_only = arvados_version.short_tests_only()
 README = os.path.join(arvados_version.SETUP_DIR, 'README.rst')
 
 setup(name='arvados_fuse',

diff --git a/services/fuse/tests/slow_test.py b/services/fuse/tests/slow_test.py
deleted file mode 120000 (symlink)
index c7e1f7f..0000000
--- a/services/fuse/tests/slow_test.py
+++ /dev/null
@@ -1 +0,0 @@
-../../../sdk/python/tests/slow_test.py
\ No newline at end of file

diff --git a/tools/crunchstat-summary/arvados_version.py b/tools/crunchstat-summary/arvados_version.py
index 794b6afe4261cba9c6bfc4c5dd3fee9d6bb6c19b..ca20de4c5813d4d0a9bf7dc6a1e74ce7ac68c43c 100644 (file)
--- a/tools/crunchstat-summary/arvados_version.py
+++ b/tools/crunchstat-summary/arvados_version.py
@@ -72,14 +72,6 @@ if REPO_PATH is None:
         if (SETUP_DIR / mod_name).is_dir()
     )
 
-def short_tests_only(arglist=sys.argv):
-    try:
-        arglist.remove('--short-tests-only')
-    except ValueError:
-        return False
-    else:
-        return True
-
 def git_log_output(path, *args):
     return subprocess.check_output(
         ['git', '-C', str(REPO_PATH),

diff --git a/tools/crunchstat-summary/setup.py b/tools/crunchstat-summary/setup.py
index 95c534c90b8b8734864049c185a197ea1ed79a96..d5adc92367a9f61aa84bb0044c3950c3a5c4b295 100755 (executable)
--- a/tools/crunchstat-summary/setup.py
+++ b/tools/crunchstat-summary/setup.py
@@ -11,7 +11,6 @@ from setuptools import setup, find_packages
 
 import arvados_version
 version = arvados_version.get_version()
-short_tests_only = arvados_version.short_tests_only()
 README = os.path.join(arvados_version.SETUP_DIR, 'README.rst')
 
 setup(name='crunchstat_summary',

diff --git a/tools/user-activity/arvados_version.py b/tools/user-activity/arvados_version.py
index 794b6afe4261cba9c6bfc4c5dd3fee9d6bb6c19b..ca20de4c5813d4d0a9bf7dc6a1e74ce7ac68c43c 100644 (file)
--- a/tools/user-activity/arvados_version.py
+++ b/tools/user-activity/arvados_version.py
@@ -72,14 +72,6 @@ if REPO_PATH is None:
         if (SETUP_DIR / mod_name).is_dir()
     )
 
-def short_tests_only(arglist=sys.argv):
-    try:
-        arglist.remove('--short-tests-only')
-    except ValueError:
-        return False
-    else:
-        return True
-
 def git_log_output(path, *args):
     return subprocess.check_output(
         ['git', '-C', str(REPO_PATH),