New process for running tests and publishing breadbox client

.github/actions/prepare-breadbox/action.yaml

@@ -12,8 +12,9 @@ runs:
     - name: "Install and configure Poetry"
       uses: snok/install-poetry@v1
       with:
-        version: 1.8.2
+        version: 2.2.1
         virtualenvs-create: true
+        virtualenvs-in-project: true
 
     - name: "Set up poetry cache"
       uses: actions/cache@v3

.github/workflows/build_breadbox.yml

@@ -61,22 +61,6 @@ jobs:
       - name: Check PR title
         run: poetry run cz check -m '${{ github.event.pull_request.title }}'
 
-  pyright-breadbox-client:
-    runs-on: ubuntu-latest
-    steps:
-      - name: "Check out"
-        uses: actions/checkout@v3
-        with:
-          token: "${{ secrets.GITHUB_TOKEN }}"
-          fetch-depth: 0
-
-      - name: Prepare breadbox client
-        uses: ./.github/actions/prepare-breadbox-client # checks out and generates breadbox client code
-
-      - name: Check that the breadbox client passes pyright
-        working-directory: ./breadbox-client
-        run: "poetry install && poetry run pyright breadbox_facade"
-
   build-docker:
     runs-on: ubuntu-latest
     steps:
@@ -141,78 +125,3 @@ jobs:
           chmod +x ./.github/push_docker.py
           # GITHUB_REF##*/ is github actions magic which expands to the branch name
           ./.github/push_docker.py ${{ env.DOCKER_REPO }} ${{ env.DOCKER_REPO }}:${{ env.DOCKER_TAG }} ${GITHUB_REF##*/}
-
-  bump-versions:
-    if: ${{ !startsWith(github.event.head_commit.message, 'bump:') && (!startsWith(github.actor, 'github-actions') || !startsWith(github.event.pusher.name, 'github-actions')) && (github.ref_name == 'master') && (github.event_name == 'push') }}
-    needs: push-docker-tag
-    runs-on: ubuntu-latest
-    name: "Bump breadbox and breadbox-client version and create changelog with commitizen"
-    outputs:
-      # Map the step outputs to job outputs
-      current_version: ${{ steps.current-version.outputs.current_version }}
-      bumped_version: ${{ steps.bumped-version.outputs.bumped_version }}
-    defaults:
-      run:
-        working-directory: ./breadbox
-    steps:
-      - name: Check out
-        uses: actions/checkout@v3
-        with:
-          token: "${{ secrets.GITHUB_TOKEN }}"
-          fetch-depth: 0
-
-      - name: Install uv
-        uses: astral-sh/setup-uv@v6
-        with:
-          version: "0.7.20"
-
-      - name: Store current version
-        id: current-version
-        run: echo "current_version=$(uvx --from commitizen cz version -p)" >> $GITHUB_OUTPUT
-
-      - name: Get current version
-        run: echo "${{ steps.current-version.outputs.current_version}}"
-
-      - name: config git for tagging
-        run: |
-          git config user.name github-actions
-          git config user.email [email protected]
-          git pull origin HEAD:master
-
-      - name: run cz bump
-        run: uvx --from commitizen cz --no-raise 21 bump --check-consistency
-
-      - name: git push tags
-        run: |
-          git push origin HEAD:master
-          git push origin HEAD:master --tags
-
-      - name: Store bumped version
-        id: bumped-version
-        run: echo "bumped_version=$(uvx --from commitizen cz version -p)" >> $GITHUB_OUTPUT
-      - name: Get bumped version
-        run: echo "${{ steps.current-version.outputs.bumped_version}}"
-
-  publish-breadbox-client:
-    needs: bump-versions
-    if: ${{ (needs.bump-versions.outputs.current_version != needs.bump-versions.outputs.bumped_version) && (github.ref_name == 'master') }}
-    runs-on: ubuntu-latest
-    steps:
-      - name: "Check out"
-        uses: actions/checkout@v3
-        with:
-          token: "${{ secrets.GITHUB_TOKEN }}"
-          fetch-depth: 0
-        # pulls latest changes from head
-      - run: |
-          git pull origin HEAD:master
-      - name: "Authenticate to Google Cloud"
-        uses: google-github-actions/auth@v2
-        with:
-          credentials_json: "${{ secrets.DEPMAP_ARTIFACTS_SVC_ACCT }}"
-
-      - name: Prepare breadbox client
-        uses: ./.github/actions/prepare-breadbox-client # checks out and generates breadbox client code
-
-      - name: Publish breadbox client
-        uses: ./.github/actions/publish-breadbox-client

.github/workflows/build_breadbox_client.yaml

@@ -0,0 +1,50 @@
+---
+name: Build breadbox client
+
+on:
+  push:
+    paths:
+      - "breadbox/**"
+      - "breadbox-client/**"
+      - ".github/workflows/build_breadbox_client.yaml"
+
+# 2
+#permissions:
+#  contents: write
+
+env:
+  DOCKER_REPO: us.gcr.io/broad-achilles/depmap-breadbox
+  DOCKER_TAG: ga2-build-${{ github.run_number }}
+
+jobs:
+  build-breadbox-client:
+    runs-on: ubuntu-latest
+    steps:
+      - name: "Check out"
+        uses: actions/checkout@v3
+        with:
+          token: "${{ secrets.GITHUB_TOKEN }}"
+          fetch-depth: 0
+
+      - name: Prepare breadbox client
+        uses: ./.github/actions/prepare-breadbox-client # checks out and generates breadbox client code
+
+      - name: "Install breadbox and dependencies"
+        working-directory: ./breadbox
+        run: "poetry install"
+
+      - name: "Install breadbox_client and dependencies"
+        working-directory: ./breadbox-client
+        run: "poetry install"
+
+      - name: Running pyright
+        working-directory: ./breadbox-client
+        run: "poetry run pyright breadbox_facade"
+
+      - name: Running pytest
+        working-directory: ./breadbox-client
+        run: "poetry run pytest"
+
+      - name: Bump version and publish
+        working-directory: ./breadbox-client
+        run: "python bump_version_and_publish.py --dry-run-if-not-branch master"

breadbox-client/breadbox_facade/client.py

@@ -11,6 +11,7 @@
 import pandas as pd
 
 from breadbox_client import Client
+from breadbox_client.api.health_check import ok
 from breadbox_client.api.api import get_task_status as get_task_status_client
 from breadbox_client.api.compute import compute_univariate_associations as compute_univariate_associations_client
 from breadbox_client.api.data_types import add_data_type as add_data_type_client
@@ -644,3 +645,7 @@ def await_task_result(self, task_id: str, timeout=None):
             return task_response["result"]
         else:
             raise ValueError(f"Unexpected task state: {task_state}")
+
+    def is_ok(self):
+        response = ok.sync_detailed(client=self.client)
+        return response