remove needless shebangs at top of file from test scripts. add in Pyt…

…honGit as pip-package dependency... would be better to do this as a testing dependency specifically but nevermind that for now...

environment.yml

@@ -22,3 +22,5 @@ dependencies:
   - conda-forge::pytest-cov
   - conda-forge::python-cdo
   - conda-forge::cdo>=2.0.0
+  - pip:
+    - GitPython

fre/tests/test_fre_app_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre app" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_catalog_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre catalog" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_check_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre check" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_cmor_cli.py

@@ -1,13 +1,10 @@
-#!/usr/bin/env python3
 ''' test "fre cmor" calls '''
 
-import datetime
 from datetime import date
-import pathlib
 from pathlib import Path
 import subprocess
-import netCDF4 as nc
 
+import netCDF4 as nc
 from click.testing import CliRunner
 
 from fre import fre
@@ -46,10 +43,13 @@ def test_cli_fre_cmor_run_opt_dne():
     result = runner.invoke(fre.fre, args=["cmor", "run", "optionDNE"])
     assert result.exit_code == 2
 
+
+# maybe this is not the right place for this test case? # TODO
+# these unit tests should be more about the cli, rather than the workload
 def test_cli_fre_cmor_run_case1(capfd):
     ''' fre cmor run, test-use case '''
 
-    
+
     # where are we? we're running pytest from the base directory of this repo
     rootdir = 'fre/tests/test_files'
 

fre/tests/test_fre_list_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre list" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_make_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre make" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_pp_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre pp" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_run_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre run" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_test_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre test" calls '''
 
 from click.testing import CliRunner

fre/tests/test_fre_yamltools_cli.py

@@ -1,4 +1,3 @@
-#!/usr/bin/env python3
 ''' test "fre yamltools" calls '''
 
 from click.testing import CliRunner

meta.yaml

@@ -43,6 +43,8 @@ requirements:
     - conda-forge::pytest-cov
     - conda-forge::python-cdo
     - conda-forge::cdo>=2.0.0
+    - pip:
+      - GitPython
 
 test:
   source_files: 

setup.py

@@ -17,7 +17,8 @@
         'cylc-flow',
         'cylc-rose',
         'cdo',
-        'metomi-rose'
+        'metomi-rose',
+        'git'
     ],
     entry_points={
         'console_scripts': [