Skip tests that requires git when it’s not available

- also fix name of the class

Signed-off-by: Stefan Marr <git@stefan-marr.de>

Author: smarr

rebench/environment.py

@@ -41,6 +41,14 @@ def extract_base(branch_or_tag):
     return branch_or_tag.replace('HEAD -> ', '')
 
 
+def git_not_available():
+    return _exec(['git', '--version']) is None
+
+
+def git_repo_not_initialized():
+    return _exec(['git', 'rev-parse']) is None
+
+
 def determine_source_details(configurator):
     global _source  # pylint: disable=global-statement
     if _source:

rebench/tests/environment_test.py

@@ -1,13 +1,15 @@
-from unittest import TestCase
+from unittest import TestCase, skipIf
 
 from ..denoise import DenoiseResult
 from ..environment import determine_source_details, determine_environment,\
-    init_environment, extract_base
+    init_environment, extract_base, git_not_available, git_repo_not_initialized
 from ..ui import TestDummyUI
 
 
-class ReBenchTestCase(TestCase):
+class EnvironmentTest(TestCase):
 
+    @skipIf(git_not_available() or git_repo_not_initialized(),
+        "git seems not to be installed, or on the path, or the .git dir is missing")
     def test_source_details(self):
         details = determine_source_details(None)
         self.assertEqual(len(details['commitId']), 40)

rebench/tests/perf/issue_166_profiling_test.py

@@ -1,5 +1,7 @@
+from unittest import skipIf
 from ..mock_http_server import MockHTTPServer
 from ...configurator import Configurator, load_config
+from ...environment import git_not_available, git_repo_not_initialized
 from ...executor import Executor
 from ...model.profile_data import ProfileData
 from ...persistence import DataStore
@@ -148,6 +150,8 @@ def test_execute_profiling_profile2(self):
         self.assertEqual(7, run_id.completed_invocations)
         self.assertEqual(7, run_id.get_number_of_data_points())
 
+    @skipIf(git_not_available() or git_repo_not_initialized(),
+        "git source info not available, but needed for reporting to ReBenchDB")
     def test_send_to_rebench_db(self):
         server = MockHTTPServer()
         port = server.get_free_port()

rebench/tests/persistency_test.py

@@ -17,12 +17,14 @@
 # LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
 # FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS
 # IN THE SOFTWARE.
+from unittest import skipIf
 from .mock_http_server import MockHTTPServer
 from .rebench_test_case import ReBenchTestCase
 
 from ..persistence import DataStore
 
 from ..configurator import Configurator, load_config
+from ..environment import git_not_available, git_repo_not_initialized
 from ..executor import Executor
 from ..model.benchmark import Benchmark
 from ..model.benchmark_suite import BenchmarkSuite
@@ -115,6 +117,8 @@ def test_data_discarding(self):
 
         self._assert_runs(cnf2, 1, 10, 10)
 
+    @skipIf(git_not_available() or git_repo_not_initialized(),
+        "git source info not available, but needed for reporting to ReBenchDB")
     def test_rebench_db(self):
         option_parser = ReBench().shell_options()
         cmd_config = option_parser.parse_args(['--experiment=Test', 'persistency.conf'])