Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,9 @@

ENHANCEMENTS:

BUG FIXES:
* Configure the default pytest-asyncio fixture loop scope to `function` to remove the deprecation warning. ([#5055](https://github.com/microsoft/AzureTRE/pull/5055))

## (0.29.0) (August 14, 2026)
**BREAKING CHANGES**
* Remove Windows 10 and dsvm image support from Guacamole. ([#4890](https://github.com/microsoft/AzureTRE/issues/4890))
Expand Down
2 changes: 1 addition & 1 deletion api_app/_version.py
Original file line number Diff line number Diff line change
@@ -1 +1 @@
__version__ = "0.26.5"
__version__ = "0.26.6"
2 changes: 2 additions & 0 deletions api_app/pytest.ini
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
[pytest]
filterwarnings =
error
asyncio_mode = strict
asyncio_default_fixture_loop_scope = function
2 changes: 1 addition & 1 deletion api_app/tests_ma/test_api/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -144,7 +144,7 @@ def inner():
return inner


@pytest_asyncio.fixture(scope='module')
@pytest.fixture(scope='module')
def app() -> FastAPI:
from main import get_application

Expand Down
7 changes: 3 additions & 4 deletions api_app/tests_ma/test_api/test_routes/test_airlock.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
import time
import pytest
import pytest_asyncio
from mock import patch
from fastapi import status
from azure.core.exceptions import HttpResponseError
Expand Down Expand Up @@ -129,7 +128,7 @@ def inner():


class TestAirlockRoutesThatRequireOwnerOrResearcherRights():
@pytest_asyncio.fixture(autouse=True, scope='class')
@pytest.fixture(autouse=True, scope='class')
def log_in_with_researcher_user(self, app, researcher_user):
app.dependency_overrides[require_workspace_owner_or_researcher] = researcher_user
app.dependency_overrides[require_workspace_owner_or_researcher_or_airlock_manager] = researcher_user
Expand Down Expand Up @@ -305,7 +304,7 @@ async def test_get_airlock_container_link_returned_as_expected(self, get_airlock


class TestAirlockRoutesThatRequireAirlockManagerRights():
@pytest_asyncio.fixture(autouse=True, scope='class')
@pytest.fixture(autouse=True, scope='class')
def log_in_with_airlock_manager_user(self, app, airlock_manager_user):
app.dependency_overrides[require_airlock_manager] = airlock_manager_user
app.dependency_overrides[require_workspace_owner_or_researcher_or_airlock_manager] = airlock_manager_user
Expand Down Expand Up @@ -465,7 +464,7 @@ async def test_post_revoke_airlock_request_missing_reason_returns_422(self, _, a

class TestAirlockRoutesPermissions():

@pytest_asyncio.fixture()
@pytest.fixture()
def log_in_with_user(self, app):
def inner(user):
app.dependency_overrides[require_workspace_owner_or_researcher] = user
Expand Down
4 changes: 4 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,7 @@
skip = "*.json,*.js,*.ts,*.tsx,*.map,package-lock.json,node_modules,.venv,.git,.yaspeller*,**/*devcontainer*,.gitignore,.gitattributes,LICENSE,Makefile,**/.dockerignore,**/Dockerfile,**/*.tf*,**/*.terraform*,**/requirements.txt,**/*.py,**/*.yml,**/*.yaml,**/*.sh,**/.env.tmpl,**/requirements-dev.txt"
# Words to ignore
ignore-words-list = "tre,aci,requestor,api,AspNet,az,AzureTRE,Codespaces,Conda,dataset,datasets,dev,Dev,DevOps,DotNet,exfiltration,Endpoints,GitHub,lifecycle,maintainer,maintainers,PowerShell,PyPi,QuickStart,quickstart,repo,repos,screenshots,terraform,Terraform,tfvars,tmpl,TREs,Xamarin,devcontainer,DevContainer,endpoints,pytest,tre,VNet,resourceType,isCurrent,msfttreacr,azurecr,io,azuretre,cnab,aci,westeurope,idempotent,Versioning,mixins,yml,microsoft,pre-emptively "

[tool.pytest.ini_options]
asyncio_mode = "strict"
asyncio_default_fixture_loop_scope = "function"
Comment thread
JC-wk marked this conversation as resolved.
Loading