Improves static testing and improves workflow speed

Author: zgosalvez

.github/README.md

@@ -17,5 +17,7 @@ All workflows use the [Ensure SHA Pinned Actions](https://github.com/marketplace
 
 Also known as CI, Continuous Integration runs Flutter static and dynamic tests, then the coverage report is stored as an artifact for reference. Modify the workflow to further process the code coverage file using [code quality](https://github.com/marketplace?type=actions) or [code review](https://github.com/marketplace?category=code-review&type=actions) actions.
 
+_Note:_ The [Get the Flutter Version Environment](https://github.com/marketplace/actions/get-the-flutter-version-environment) action requires that the [`pubspec.yaml`](pubspec.yaml) file contains an `environment:flutter:` key.
+
 ## License
 The scripts and documentation in this project are released under the [MIT License](LICENSE)

.github/workflows/ci.yml

@@ -33,48 +33,49 @@ jobs:
       uses: zgosalvez/github-actions-get-flutter-version-env@8c616b4685d25cceda9bafa61f6c72670f6aa01e # v1.1.1
       with:
         pubspec-file-path: code/pubspec.yaml
-    - name: Cache Flutter packages
-      id: cache
+    - name: Cache Flutter
+      id: flutter-cache
       uses: actions/cache@0781355a23dac32fd3bac414512f4b903437991a # v2.1.3
       with:
         path: flutter
-        key: pub-${{ env.cache-name }}-${{ steps.get-flutter-version.outputs.version }}-${{ hashFiles('code/pubspec.lock') }}
+        key: ${{ env.cache-name }}-${{ steps.get-flutter-version.outputs.version }}-${{ hashFiles('code/pubspec.lock') }}
         restore-keys: |
-          pub-${{ env.cache-name }}-${{ steps.get-flutter-version.outputs.version }}-
-          pub-${{ env.cache-name }}-
-          pub-
+          ${{ env.cache-name }}-${{ steps.get-flutter-version.outputs.version }}-
+          ${{ env.cache-name }}-
       env:
-        cache-name: cache-flutter-packages
+        cache-name: flutter-cache
     - name: Clone the Flutter repository
-      if: steps.cache.outputs.cache-hit != 'true'
+      if: steps.flutter-cache.outputs.cache-hit != 'true'
       uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f # v2.3.4
       with:
         repository: flutter/flutter
         ref: ${{ steps.get-flutter-version.outputs.version }}
         path: flutter
     - name: Add the flutter tool to the path
-      run: echo "$GITHUB_WORKSPACE/flutter/bin" >> $GITHUB_PATH
+      run: |
+        echo "$GITHUB_WORKSPACE/flutter/bin" >> $GITHUB_PATH
+        echo "$GITHUB_WORKSPACE/flutter/bin/cache/dart-sdk/bin" >> $GITHUB_PATH
     - name: Populate the Flutter tool's cache of binary artifacts
-      if: steps.cache.outputs.cache-hit != 'true'
+      if: steps.flutter-cache.outputs.cache-hit != 'true'
       run: |
         flutter config --no-analytics
         flutter precache
     - name: Get Flutter packages
-      if: steps.cache.outputs.cache-hit != 'true'
       run: flutter pub get
       working-directory: code
 
     # static testing
     - name: Analyze Flutter
       if: matrix.category == 'static'
-      uses: zgosalvez/github-actions-analyze-flutter@42999ac72bf187609b2b61ec93bf0b03e4ebca6e # v1.1.0
+      uses: zgosalvez/github-actions-analyze-flutter@f1a0a3abe4211723932c4d5571702c8f71e11f20 # v1.3.0
       with:
+        fail-on-warnings: true
         working-directory: code
 
     # unit testing
     - name: Run Flutter ${{ matrix.category }} tests
       if: matrix.category != 'static'
-      run: flutter test --coverage --coverage-path=./coverage/lcov.${{ matrix.category }}.info test/${{ matrix.category }}s
+      run: flutter test --no-pub --coverage --coverage-path=./coverage/lcov.${{ matrix.category }}.info test/${{ matrix.category }}s
       working-directory: code
     - name: Upload code coverage to GitHub
       if: matrix.category != 'static'