From 2a3b428b18c0fa1e7b1c8a1a2c8ecd8cbfbb91c1 Mon Sep 17 00:00:00 2001 From: Ximi Hoque Date: Fri, 30 Jan 2026 12:26:55 +0530 Subject: [PATCH 1/5] Release --- .github/workflows/release.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index bb7dca4..a0aa143 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -27,7 +27,7 @@ on: type: boolean env: - PYTHON_VERSION: '3.9' + PYTHON_VERSION: '3.12' jobs: # Test before releasing - unified test suite @@ -41,10 +41,10 @@ jobs: - name: Checkout code uses: actions/checkout@v4 - - name: Set up Python 3.9 + - name: Set up Python 3.12 uses: actions/setup-python@v4 with: - python-version: '3.9' + python-version: '3.12' - name: Install dependencies run: | From a8b9a00c8934e02d5fd4e98c36dd1953e89dd77a Mon Sep 17 00:00:00 2001 From: Ximi Hoque Date: Fri, 30 Jan 2026 14:56:36 +0530 Subject: [PATCH 2/5] Updated tests and yml --- .github/workflows/release.yml | 2 +- .../test_dataset_creation_integration.py | 13 ++++++++++--- 2 files changed, 11 insertions(+), 4 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index a0aa143..1ce6ab4 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -69,7 +69,7 @@ jobs: env: API_KEY: ${{ secrets.LABELLERR_API_KEY }} API_SECRET: ${{ secrets.LABELLERR_API_SECRET }} - CLIENT_ID: ${{ secrets.LABELLERR_CLIENT_ID }} + CLIENT_ID: ${{ vars.LABELLERR_CLIENT_ID }} TEST_EMAIL: ${{ secrets.LABELLERR_TEST_EMAIL }} AWS_CONNECTION_IMAGE: ${{ secrets.AWS_CONNECTION_IMAGE }} AWS_CONNECTION_VIDEO: ${{ secrets.AWS_CONNECTION_VIDEO }} diff --git a/tests/integration/test_dataset_creation_integration.py b/tests/integration/test_dataset_creation_integration.py index b3a019b..48a48a1 100644 --- a/tests/integration/test_dataset_creation_integration.py +++ b/tests/integration/test_dataset_creation_integration.py @@ -301,7 +301,7 @@ class TestDatasetValidationIntegration: """Integration tests for dataset ID validation with real API""" def test_invalid_dataset_id_format_rejected(self, integration_client): - """Test that invalid dataset ID formats are rejected before API call""" + """Test that invalid dataset ID formats are rejected by API with 400 response""" invalid_ids = [ "invalid-id", "not-a-uuid", @@ -311,9 +311,16 @@ def test_invalid_dataset_id_format_rejected(self, integration_client): for invalid_id in invalid_ids: with pytest.raises( - InvalidDatasetIDError, match="Invalid dataset ID format" - ): + (InvalidDatasetError, LabellerrError) + ) as exc_info: LabellerrDataset(integration_client, invalid_id) + + # Verify it's a 400 error for invalid format + error_msg = str(exc_info.value).lower() + assert any( + x in error_msg + for x in ["invalid", "format", "400", "bad request", "dataset"] + ), f"Expected invalid dataset format error, got: {exc_info.value}" def test_valid_uuid_format_but_nonexistent_dataset(self, integration_client): """ From 8ffe1d39183edb2ad68b94cef94dc79080693c78 Mon Sep 17 00:00:00 2001 From: Ximi Hoque Date: Fri, 30 Jan 2026 14:58:22 +0530 Subject: [PATCH 3/5] Formatting --- tests/integration/test_dataset_creation_integration.py | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/tests/integration/test_dataset_creation_integration.py b/tests/integration/test_dataset_creation_integration.py index 48a48a1..d810e02 100644 --- a/tests/integration/test_dataset_creation_integration.py +++ b/tests/integration/test_dataset_creation_integration.py @@ -310,11 +310,9 @@ def test_invalid_dataset_id_format_rejected(self, integration_client): ] for invalid_id in invalid_ids: - with pytest.raises( - (InvalidDatasetError, LabellerrError) - ) as exc_info: + with pytest.raises((InvalidDatasetError, LabellerrError)) as exc_info: LabellerrDataset(integration_client, invalid_id) - + # Verify it's a 400 error for invalid format error_msg = str(exc_info.value).lower() assert any( From 40cced9ebf49e2774c7739ee2ae467b3b83fc007 Mon Sep 17 00:00:00 2001 From: Ximi Hoque Date: Fri, 30 Jan 2026 15:01:43 +0530 Subject: [PATCH 4/5] Udpates --- .github/workflows/release.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 1ce6ab4..2eee6c2 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -67,10 +67,10 @@ jobs: - name: Run integration tests env: - API_KEY: ${{ secrets.LABELLERR_API_KEY }} - API_SECRET: ${{ secrets.LABELLERR_API_SECRET }} - CLIENT_ID: ${{ vars.LABELLERR_CLIENT_ID }} - TEST_EMAIL: ${{ secrets.LABELLERR_TEST_EMAIL }} + API_KEY: ${{ secrets.API_KEY }} + API_SECRET: ${{ secrets.API_SECRET }} + CLIENT_ID: ${{ vars.CLIENT_ID }} + TEST_EMAIL: ${{ vars.CLIENT_EMAIL }} AWS_CONNECTION_IMAGE: ${{ secrets.AWS_CONNECTION_IMAGE }} AWS_CONNECTION_VIDEO: ${{ secrets.AWS_CONNECTION_VIDEO }} GCS_CONNECTION_IMAGE: ${{ secrets.GCS_CONNECTION_IMAGE }} From 7fdb6dc853562ba3ac9baca4e65fe3c05f3f6b81 Mon Sep 17 00:00:00 2001 From: Ximi Hoque Date: Fri, 30 Jan 2026 15:24:32 +0530 Subject: [PATCH 5/5] updates --- Makefile | 2 +- tests/integration/{ => mcp}/run_mcp_integration_tests.py | 0 tests/integration/{ => mcp}/run_mcp_tools_tests.py | 0 tests/integration/{ => mcp}/test_mcp_server.py | 0 tests/integration/{ => mcp}/test_mcp_tools.py | 0 5 files changed, 1 insertion(+), 1 deletion(-) rename tests/integration/{ => mcp}/run_mcp_integration_tests.py (100%) rename tests/integration/{ => mcp}/run_mcp_tools_tests.py (100%) rename tests/integration/{ => mcp}/test_mcp_server.py (100%) rename tests/integration/{ => mcp}/test_mcp_tools.py (100%) diff --git a/Makefile b/Makefile index 2076005..cc6f125 100644 --- a/Makefile +++ b/Makefile @@ -36,7 +36,7 @@ test-unit: ## Run only unit tests test-integration: ## Run only integration tests (requires credentials) @mkdir -p tests/integration/test_reports - $(PYTHON) -m pytest tests/integration/ -v -m "integration and not deprecated" + $(PYTHON) -m pytest tests/integration/ -v -m "integration and not deprecated" --ignore=tests/integration/mcp @echo "" @echo "✅ Integration tests completed! Check output above for report locations." diff --git a/tests/integration/run_mcp_integration_tests.py b/tests/integration/mcp/run_mcp_integration_tests.py similarity index 100% rename from tests/integration/run_mcp_integration_tests.py rename to tests/integration/mcp/run_mcp_integration_tests.py diff --git a/tests/integration/run_mcp_tools_tests.py b/tests/integration/mcp/run_mcp_tools_tests.py similarity index 100% rename from tests/integration/run_mcp_tools_tests.py rename to tests/integration/mcp/run_mcp_tools_tests.py diff --git a/tests/integration/test_mcp_server.py b/tests/integration/mcp/test_mcp_server.py similarity index 100% rename from tests/integration/test_mcp_server.py rename to tests/integration/mcp/test_mcp_server.py diff --git a/tests/integration/test_mcp_tools.py b/tests/integration/mcp/test_mcp_tools.py similarity index 100% rename from tests/integration/test_mcp_tools.py rename to tests/integration/mcp/test_mcp_tools.py