Skip to content

ci(he-tme): bump staging to 1.1.0#450

Merged
blanca-pablos merged 2 commits intomainfrom
chore/he-tme-version-bump
Mar 5, 2026
Merged

ci(he-tme): bump staging to 1.1.0#450
blanca-pablos merged 2 commits intomainfrom
chore/he-tme-version-bump

Conversation

@ari-nz
Copy link
Collaborator

@ari-nz ari-nz commented Feb 26, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 26, 2026 09:37
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the staging E2E test configuration for the H&E-TME (“he-tme”) application to reflect a newer deployed version, keeping environment-specific test expectations aligned with what staging runs.

Changes:

  • Bump HETA_APPLICATION_VERSION for the staging environment from 1.0.0 to 1.1.0 in shared test constants.

@codecov
Copy link

codecov bot commented Feb 26, 2026

❌ 3 Tests Failed:

Tests completed Failed Passed Skipped
702 3 699 17
View the top 2 failed test(s) by shortest run time
tests.aignostics.application.gui_test::test_gui_download_dataset_via_application_to_run_cancel_to_find_back
Stack Traces | 3.06s run time
user = <nicegui.testing.user.User object at 0x7f3a20ec0b90>
runner = <typer.testing.CliRunner object at 0x7f3a20ec1310>
silent_logging = None
record_property = <function record_property.<locals>.append_property at 0x7f39f40a3270>

    @pytest.mark.e2e
    @pytest.mark.long_running
    @pytest.mark.flaky(retries=1, delay=5)
    @pytest.mark.timeout(timeout=60 * 10)
    @pytest.mark.sequential
    async def test_gui_download_dataset_via_application_to_run_cancel_to_find_back(  # noqa: PLR0915
        user: User, runner: CliRunner, silent_logging: None, record_property
    ) -> None:
        """Test that the user can download a dataset via the application page and cancel the run, then find it back."""
        record_property("tested-item-id", "TC-APPLICATION-GUI-04, SPEC-GUI-SERVICE")
        with tempfile.TemporaryDirectory() as tmpdir:
            tmp_path = Path(tmpdir)
    
            with patch(
                "aignostics.application._gui._page_application_describe.Path.home",
                return_value=tmp_path,
            ):
                # Download example wsi
                result = runner.invoke(
                    cli,
                    [
                        "dataset",
                        "aignostics",
                        "download",
                        SPOT_1_GS_URL,
                        str(tmp_path),
                    ],
                )
>               assert result.exit_code == 0
E               assert 1 == 0
E                +  where 1 = <Result SystemExit(1)>.exit_code

.../aignostics/application/gui_test.py:207: AssertionError
tests.aignostics.application.cli_test::test_cli_run_execute
Stack Traces | 3.92s run time
runner = <typer.testing.CliRunner object at 0x7f8552d7b620>
tmp_path = PosixPath('.../pytest-22/popen-gw7/test_cli_run_execute0')
record_property = <function record_property.<locals>.append_property at 0x7f8552c6c0f0>

    @pytest.mark.e2e
    @pytest.mark.very_long_running
    @pytest.mark.timeout(timeout=HETA_APPLICATION_DEADLINE_SECONDS + 60 * 30)
    def test_cli_run_execute(runner: CliRunner, tmp_path: Path, record_property) -> None:
        """Check run execution runs e2e."""
        record_property("tested-item-id", "TC-APPLICATION-CLI-03")
    
        # Step 1: Download the sample file
        result = runner.invoke(
            cli,
            [
                "dataset",
                "aignostics",
                "download",
                SPOT_1_GS_URL,
                str(tmp_path),
            ],
        )
    
        # Explore what was download
        print_directory_structure(tmp_path, "download")
    
        # Validate what was downloaded
>       assert "Successfully downloaded" in normalize_output(result.stdout)
E       assert 'Successfully downloaded' in "Downloading ━   0% … 0 0 ? Downloading 9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff                     … …                                                        Error downloading data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': Failed to download data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': 403 GET https://storage.googleapis..../aignx-storage-service-dev/o/sample_data_formatted%2F9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff?fields=name&prettyPrint=false: aignostics-platform-ext-a4f7e9@aignx-papi-prod-zkgprhen.iam.gserviceaccount.com does not have storage.objects.get access to the Google Cloud Storage object. Permission 'storage.objects.get' denied on resource (or it may not exist)."
E        +  where "Downloading ━   0% … 0 0 ? Downloading 9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff                     … …                                                        Error downloading data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': Failed to download data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': 403 GET https://storage.googleapis..../aignx-storage-service-dev/o/sample_data_formatted%2F9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff?fields=name&prettyPrint=false: aignostics-platform-ext-a4f7e9@aignx-papi-prod-zkgprhen.iam.gserviceaccount.com does not have storage.objects.get access to the Google Cloud Storage object. Permission 'storage.objects.get' denied on resource (or it may not exist)." = normalize_output("Downloading ━   0% … 0 0 ? Downloading 9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff\n                     … …                                                        \nError downloading data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': Failed to download data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': 403 GET https://storage.googleapis..../aignx-storage-service-dev/o/sample_data_formatted%2F9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff?fields=name&prettyPrint=false: aignostics-platform-ext-a4f7e9@aignx-papi-prod-zkgprhen.iam.gserviceaccount.com does not have storage.objects.get access to the Google Cloud Storage object. Permission 'storage.objects.get' denied on resource (or it may not exist).\n")
E        +    where "Downloading ━   0% … 0 0 ? Downloading 9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff\n                     … …                                                        \nError downloading data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': Failed to download data from 'gs:.../aignx-storage-service-dev/sample_data_formatted/9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff': 403 GET https://storage.googleapis..../aignx-storage-service-dev/o/sample_data_formatted%2F9375e3ed-28d2-4cf3-9fb9-8df9d11a6627.tiff?fields=name&prettyPrint=false: aignostics-platform-ext-a4f7e9@aignx-papi-prod-zkgprhen.iam.gserviceaccount.com does not have storage.objects.get access to the Google Cloud Storage object. Permission 'storage.objects.get' denied on resource (or it may not exist).\n" = <Result SystemExit(1)>.stdout

.../aignostics/application/cli_test.py:879: AssertionError
View the full list of 1 ❄️ flaky test(s)
tests.aignostics.dataset.cli_test::test_cli_aignostics_download_sample

Flake rate in main: 8.82% (Passed 31 times, Failed 3 times)

Stack Traces | 4.15s run time
runner = <typer.testing.CliRunner object at 0x7f6a65b006b0>
tmp_path = PosixPath('.../pytest-18/popen-gw3/test_cli_aignostics_download_s1')
record_property = <function record_property.<locals>.append_property at 0x7f6a61982140>

    @pytest.mark.e2e
    @pytest.mark.flaky(retries=1, delay=5)
    @pytest.mark.timeout(timeout=60 * 2)
    def test_cli_aignostics_download_sample(runner: CliRunner, tmp_path: Path, record_property) -> None:
        """Check download functionality with dry-run option."""
        record_property("tested-item-id", "TC-DATASET-CLI-01")
        result = runner.invoke(
            cli,
            [
                "dataset",
                "aignostics",
                "download",
                SPOT_1_GS_URL,
                str(tmp_path),
            ],
        )
>       assert result.exit_code == 0
E       assert 1 == 0
E        +  where 1 = <Result SystemExit(1)>.exit_code

.../aignostics/dataset/cli_test.py:145: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@blanca-pablos blanca-pablos self-requested a review February 26, 2026 11:43
@ari-nz ari-nz force-pushed the chore/he-tme-version-bump branch from 8aefaf9 to 8c6e6e6 Compare February 26, 2026 12:58
Copilot AI review requested due to automatic review settings March 4, 2026 19:47
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@sonarqubecloud
Copy link

sonarqubecloud bot commented Mar 4, 2026

@blanca-pablos blanca-pablos merged commit fb150e4 into main Mar 5, 2026
30 of 33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants