slightly different approach to test venv build for travis

.travis.yml

@@ -3,9 +3,9 @@ language: python
 python: 3.8
 install:
 - gem install asciidoctor coderay
-- pip3 install -r requirements.txt
+- pip install -r requirements.txt
 script:
-- make html update-code test
+- make html update-code code-venv test
 git:
   submodules: false
 before_install:

Makefile

@@ -9,6 +9,9 @@ update-code:
 	cd code && git fetch
 	./checkout-branches-for-ci.py
 
+code-venv:
+	cd code && python3.8 -m venv .venv
+
 count-todos:
 	ls *.asciidoc | xargs grep -c TODO | sed  s/:/\\t/
 

checkout-branches-for-ci.py

@@ -7,9 +7,9 @@
 cwd = Path(__file__).parent / 'code'
 
 for chap in CHAPTERS + STANDALONE:
-    subprocess.run(['git', 'checkout', chap], cwd=cwd)
+    subprocess.run(['git', 'checkout', chap], cwd=cwd, check=True)
     if chap in NO_EXERCISE:
         continue
-    subprocess.run(['git', 'checkout', f'{chap}_exercise'], cwd=cwd)
+    subprocess.run(['git', 'checkout', f'{chap}_exercise'], cwd=cwd, check=True)
 
-subprocess.run(['git', 'checkout', 'master'], cwd=cwd)
+subprocess.run(['git', 'checkout', 'master'], cwd=cwd, check=True)

tests.py

@@ -230,12 +230,9 @@ def tree_for_branch(chapter_name):
 def code_venv():
     code = Path(__file__).parent / 'code'
     venv = code / '.venv'
-    if (venv / 'bin/pytest').exists():
-        return
-    subprocess.run(['python3.8', '-m', 'venv', venv], check=True)
     assert (venv / 'bin/pip').exists()
     subprocess.run([venv / 'bin/pip', 'install', '-r', code / 'requirements.txt'], check=True)
-    subprocess.run([venv / 'bin/pip', 'install', '-e', code / 'src'])
+    subprocess.run([venv / 'bin/pip', 'install', '-e', code / 'src'], check=True)
 
 
 @pytest.mark.usefixtures('code_venv')