Change CircleCI tests to use cached lockfile (#353)

* Change CircleCI tests to use cached lockfile

* Fix config

* Add circleci lock file

* Do not use python version for cache key

* Add import test for client install matrix

* Update README.md

* Just use the regular lock file

Author: gatli

.circleci/config.yml

@@ -3,7 +3,7 @@
 version: 2.1
 orbs:
   slack: circleci/slack@4.4.2
-  python: circleci/python@2.0.3
+  python: circleci/python@2.1.0
 
 jobs:
   build_test:
@@ -23,18 +23,17 @@ jobs:
             apt-get -y install curl libgeos-dev
             pip install --upgrade pip
             pip install poetry
-      - run:
-          name: Install Python Dependencies
-          command: |
-              poetry install
+      - python/install-packages:
+            pkg-manager: poetry
+            include-python-in-cache-key: false
       - run:
           name: Test Imports (extras need to be guarded!)
           command: | # Make sure that importing works without extras installed
             poetry run python -c 'import nucleus'
-      - run:
-          name: Install Extra Python Dependencies
-          command: | # install dependencies
-            poetry install -E metrics -E launch
+      - python/install-packages:
+            pkg-manager: poetry
+            args: -E metrics -E launch
+            include-python-in-cache-key: false
       - run:
           name: Black Formatting Check # Only validation, without re-formatting
           command: |
@@ -110,24 +109,6 @@ jobs:
                 exit 1
             fi
             poetry publish --username=$PYPI_USERNAME --password=$PYPI_PASSWORD
-  test_poetry_installation:
-    parameters:
-      python_version:
-        type: string
-    docker:
-      - image: cimg/python:<<parameters.python_version>>
-    steps:
-      - checkout
-      - run:
-          command: |
-            pip install --upgrade pip
-          name: Upgrade pip
-      - python/install-packages:
-            pkg-manager: poetry
-      - run:
-          command: |
-            poetry run python -c 'import nucleus'
-          name: Test import nucleus
   test_client_installation:
     parameters:
       python_version:
@@ -145,10 +126,29 @@ jobs:
             poetry build
           name: Build the package
       - run:
+          name: Install - no extras
           command: |
             export FOUND_PKG=$(find ./dist -name "*.tar.gz")
             pip install $FOUND_PKG
-          name: Install with Python Version
+            python -c 'import nucleus'
+      - run:
+          name: Install with extra metrics
+          command: |
+            export FOUND_PKG=$(find ./dist -name "*.tar.gz")
+            pip install "$FOUND_PKG[metrics]"
+            python -c 'import nucleus'
+      - run:
+          name: Install with extra launch
+          command: |
+            export FOUND_PKG=$(find ./dist -name "*.tar.gz")
+            pip install "$FOUND_PKG[launch]"
+            python -c 'import nucleus'
+      - run:
+          name: Install with extra launch and metrics
+          command: |
+            export FOUND_PKG=$(find ./dist -name "*.tar.gz")
+            pip install "$FOUND_PKG[metrics, launch]"
+            python -c 'import nucleus'
 workflows:
   nightly_build_test:
     triggers:
@@ -163,11 +163,6 @@ workflows:
           context: Nucleus
   installation_matrix:
     jobs:
-      - test_poetry_installation:
-          matrix:
-            parameters:
-              python_version: ['3.6', '3.7', '3.8', '3.9'] #, '3.10'] Need to work out a solution to support dev on 3.10
-          context: Nucleus
       - test_client_installation:
           matrix:
             parameters:
@@ -188,3 +183,4 @@ workflows:
               ignore: /.*/ # Runs for none of the branches
             tags:
               only: /^v\d+\.\d+\.\d+$/ # Runs only for tags with the format [v1.2.3]
+