Add a workflow step to run the grpc_generator unit tests. (#25)

* Add a workflow step to run the grpc_generator unit tests.

Signed-off-by: Michael Johansen <[email protected]>

* Move grpc_generator tests into a 'unit' subfolder to work with workflows.

Signed-off-by: Michael Johansen <[email protected]>

* Only run grpc_generator tests on python >= 3.11.

Signed-off-by: Michael Johansen <[email protected]>

* Treat the version as a string when comparing.

Signed-off-by: Michael Johansen <[email protected]>

* Try not using the ${{ }} dereference. Also just try a simple equals.

Signed-off-by: Michael Johansen <[email protected]>

* Try switching back to 'greater than'.

Signed-off-by: Michael Johansen <[email protected]>

* Use fromJSON to convert the version to a number for easier comparision.

Signed-off-by: Michael Johansen <[email protected]>

* Just go back to == 3.11. Testing against the oldest supported version is sufficient.

Signed-off-by: Michael Johansen <[email protected]>

* Fix broken syntax.

Signed-off-by: Michael Johansen <[email protected]>

* Address review feedback. Try a new if statement.

Signed-off-by: Michael Johansen <[email protected]>

---------

Signed-off-by: Michael Johansen <[email protected]>
Co-authored-by: Michael Johansen <[email protected]>

Author: mjohanse-emr

.github/actions/run_and_upload_unit_tests/action.yml

@@ -4,25 +4,29 @@ inputs:
   package-name:
     description: 'The name of the package to run tests for.'
     required: true
+  package-basepath:
+    description: 'Relative path to the parent directory of the package.'
+    required: false
+    default: 'packages'
 runs:
   using: "composite"
   steps:
     - name: Cache ${{ inputs.package-name }} virtualenv
       uses: actions/cache@5a3ec84eff668545956fd18022155c47e93e2684 # v4.2.3
       with:
-        path: ./packages/${{ inputs.package-name }}/.venv
-        key: ${{ inputs.package-name }}-${{ runner.os }}-py${{ env.pythonVersion }}-${{ hashFiles(format('packages/{0}/poetry.lock', inputs.package-name)) }}
+        path: ./${{ inputs.package-basepath }}/${{ inputs.package-name }}/.venv
+        key: ${{ inputs.package-name }}-${{ runner.os }}-py${{ env.pythonVersion }}-${{ hashFiles(format('{0}/{1}/poetry.lock', inputs.package-basepath, inputs.package-name)) }}
     - name: Install ${{ inputs.package-name }}
       run: poetry install -v
-      working-directory: ./packages/${{ inputs.package-name }}
+      working-directory: ./${{ inputs.package-basepath }}/${{ inputs.package-name }}
       shell: bash
     - name: Run ${{ inputs.package-name }} unit tests and code coverage
       run: poetry run pytest ./tests/unit -v --cov=${{ inputs.package-name }} --junitxml=test_results/${{ inputs.package-name }}-${{ matrix.os }}-py${{ matrix.python-version }}.xml
-      working-directory: ./packages/${{ inputs.package-name }}
+      working-directory: ./${{ inputs.package-basepath }}/${{ inputs.package-name }}
       shell: bash
     - name: Upload ${{ inputs.package-name }} test results
       uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
       with:
         name: test_results_unit_${{ inputs.package-name }}_${{ matrix.os }}_py${{ matrix.python-version }}
-        path: ./packages/${{ inputs.package-name }}/test_results/*.xml
+        path: ./${{ inputs.package-basepath }}/${{ inputs.package-name }}/test_results/*.xml
       if: always()

.github/workflows/run_unit_tests.yml

@@ -35,3 +35,10 @@ jobs:
         uses: ./.github/actions/run_and_upload_unit_tests
         with:
           package-name: ni.pythonpanel.v1.proto
+      # Run grpc_generator unit tests on its oldest supported version.
+      - name: Run and upload unit tests - grpc_generator
+        if: ${{ !contains(fromJSON('["3.9", "3.10"]'), matrix.python-version) }}
+        uses: ./.github/actions/run_and_upload_unit_tests
+        with:
+          package-name: grpc_generator
+          package-basepath: tools

tools/grpc_generator/tests/unit/__init__.py

@@ -0,0 +1 @@
+"""Unit tests for grpc_generator."""