ci(test): add main test for hm-skypilot (#13590)

.github/workflows/test.yml

@@ -48,6 +48,7 @@ jobs:
       hm-pyvista-mount-saint-helens: ${{ steps.filter.outputs.hm-pyvista-mount-saint-helens }}
       hm-rasa: ${{ steps.filter.outputs.hm-rasa }}
       hm-serial: ${{ steps.filter.outputs.hm-serial }}
+      hm-skypilot: ${{ steps.filter.outputs.hm-skypilot }}
       hm-spark-find-retired-people-scala: ${{ steps.filter.outputs.hm-spark-find-retired-people-scala }}
       hm-spark-find-taxi-top-routes: ${{ steps.filter.outputs.hm-spark-find-taxi-top-routes }}
       hm-spark-find-taxi-top-routes-sql: ${{ steps.filter.outputs.hm-spark-find-taxi-top-routes-sql }}
@@ -156,6 +157,9 @@ jobs:
             hm-serial:
               - '.github/workflows/test.yml'
               - 'hm-serial/**'
+            hm-skypilot:
+              - '.github/workflows/test.yml'
+              - 'hm-skypilot/**'
             hm-spark-find-retired-people-scala:
               - '.github/workflows/test.yml'
               - 'hm-spark/applications/find-retired-people-scala/**'
@@ -1356,6 +1360,38 @@ jobs:
         with:
           directory: hm-serial
 
+  skypilot-test:
+    name: SkyPilot | Test
+    needs: detect-changes
+    if: ${{ needs.detect-changes.outputs.hm-skypilot == 'true' }}
+    runs-on: ubuntu-22.04
+    environment: test
+    timeout-minutes: 10
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Install Poetry
+        run: |
+          pipx install poetry
+      - name: Set up Python
+        uses: actions/setup-python@v5
+        with:
+          python-version-file: hm-skypilot/pyproject.toml
+          cache: poetry
+          cache-dependency-path: hm-skypilot/poetry.lock
+      - name: Install dependencies
+        working-directory: hm-skypilot
+        run: |
+          poetry install --no-root
+      - name: Test
+        working-directory: hm-skypilot
+        run: |
+          poetry run poe test-coverage
+      - name: Upload coverage to Codecov
+        uses: codecov/codecov-action@v3
+        with:
+          directory: hm-skypilot
+
   supervision-detect-objects-test:
     name: supervision (detect-objects) | Test
     needs: detect-changes

.mergify.yml

@@ -228,11 +228,14 @@ pull_request_rules:
           - check-success=PyVista (mount-saint-helens) | Test
           - check-skipped=PyVista (mount-saint-helens) | Test
       - or:
-          - check-success=Reinforcement Learning (cart-pole) | Test | Test
-          - check-skipped=Reinforcement Learning (cart-pole) | Test | Test
+          - check-success=Reinforcement Learning (cart-pole) | Test
+          - check-skipped=Reinforcement Learning (cart-pole) | Test
       - or:
           - check-success=Serial | Test
           - check-skipped=Serial | Test
+      - or:
+          - check-success=SkyPilot | Test
+          - check-skipped=SkyPilot | Test
       - or:
           - check-success=supervision (detect-objects) | Test
           - check-skipped=supervision (detect-objects) | Test

hm-skypilot/Makefile

@@ -1,10 +1,15 @@
 poetry-env-use:
-	poetry env use python3.10
+	poetry env use python3.11
 poetry-update-lock-file:
 	poetry lock --no-update
 poetry-install:
 	poetry install --no-root
 
+poetry-run-test:
+	poetry run poe test
+poetry-run-test-coverage:
+	poetry run poe test-coverage
+
 sky-launch:
 	poetry run sky launch skypilot.yaml \
 		--cluster=hm-skypilot-cluster \

hm-skypilot/main_test.py

@@ -0,0 +1,10 @@
+import unittest
+
+
+class TestMain(unittest.TestCase):
+    def test_main(self):
+        self.assertTrue(True)
+
+
+if __name__ == "__main__":
+    unittest.main()