[MNT] CI workflow to run multiple versions and os (#68)

## Change
updated `.github\workflows\tests.yml` to add `concurrency` (${{
github.workflow }}-${{ github.event.pull_request.number || github.ref
}}), multiple `python versions` ("3.10", "3.11", "3.12") and multiple
`os` (ubuntu-latest, windows-latest) for running `pytests`

## How to Test
in the CI envirionment the tests should run with multiple os and python
versions.

Author: rohansen856

.github/workflows/tests.yml

@@ -15,17 +15,21 @@ on:
   workflow_dispatch:
 
 concurrency:
-  group: ${{ github.ref }}
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
   cancel-in-progress: true
 
 jobs:
   test-code:
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        python-version: ["3.10", "3.11", "3.12", "3.13", "3.14"]
+        os: [ubuntu-latest, windows-latest]
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-python@v5
         with:
-          python-version: "3.11"
+          python-version: ${{ matrix.python-version }}
           cache: pip
       - run: python -m pip install ".[dev]"
         name: Install Dependencies

src/aiod/authentication/authentication.py

@@ -1,4 +1,4 @@
-import datetime
+from datetime import datetime, timezone, timedelta
 import time
 import http.client
 from http import HTTPStatus
@@ -39,9 +39,9 @@ def _on_keycloak_config_changed(_: str, __: str, ___: str) -> None:
 config.subscribe("client_id", on_change=_on_keycloak_config_changed)
 
 
-def _datetime_utc_in(*, seconds: int) -> datetime.datetime:
-    span = datetime.timedelta(seconds=seconds)
-    return datetime.datetime.now(datetime.UTC) + span
+def _datetime_utc_in(*, seconds: int) -> datetime:
+    span = timedelta(seconds=seconds)
+    return datetime.now(timezone.utc) + span
 
 
 class Token:
@@ -71,7 +71,7 @@ def __init__(
     @property
     def has_expired(self) -> bool:
         """Return whether the *access token* has expired based on local data."""
-        return datetime.datetime.now(datetime.UTC) >= self._expiration_date
+        return datetime.now(timezone.utc) >= self._expiration_date
 
     @property
     def headers(self) -> dict[str, str]:
@@ -142,7 +142,7 @@ def from_file(cls, file: Path | None = None) -> "Token":
             "client_secret": str(doc.get("client_secret", "")),
         }
         if "expiration_date" in doc:
-            expiration_date = datetime.datetime.fromisoformat(doc["expiration_date"])
+            expiration_date = datetime.fromisoformat(doc["expiration_date"])
             expires_in = expiration_date - _datetime_utc_in(seconds=0)
             if expires_in.total_seconds() > 0:
                 kwargs.update(

src/aiod/bookmarks.py

@@ -1,4 +1,4 @@
-import datetime as dt
+from datetime import datetime, timezone
 from dataclasses import dataclass
 from http import HTTPStatus
 
@@ -18,12 +18,12 @@ class Bookmark:
     ----------
     identifier: str
         The identifier of the asset on AI-on-Demand, e.g., 'data_xyz...'
-    created: datetime.datetime
+    created: datetime
         The datetime when the bookmark was originally created.
     """
 
     identifier: str
-    created: dt.datetime
+    created: datetime
 
 
 def _bookmarks_url() -> str:
@@ -63,8 +63,8 @@ def register(identifier: str) -> Bookmark:
 
     return Bookmark(
         identifier=res.json()["resource_identifier"],
-        created=dt.datetime.fromisoformat(res.json()["created_at"]).replace(
-            tzinfo=dt.UTC
+        created=datetime.fromisoformat(res.json()["created_at"]).replace(
+            tzinfo=timezone.utc
         ),
     )
 
@@ -115,8 +115,8 @@ def get_list() -> list[Bookmark]:
     return [
         Bookmark(
             identifier=bookmark["resource_identifier"],
-            created=dt.datetime.fromisoformat(bookmark["created_at"]).replace(
-                tzinfo=dt.UTC
+            created=datetime.fromisoformat(bookmark["created_at"]).replace(
+                tzinfo=timezone.utc
             ),
         )
         for bookmark in res.json()

tests/test_endpoints.py

@@ -2,6 +2,7 @@
 import json
 from http import HTTPStatus
 
+from aiohttp.pytest_plugin import loop
 import pytest
 import responses
 from responses import matchers
@@ -232,7 +233,9 @@ def test_search(asset_with_search):
 
 
 def test_endpoint_get_asset_async(asset_name):
-    loop = asyncio.get_event_loop()
+    loop = asyncio.new_event_loop()
+    asyncio.set_event_loop(loop)
+
     with aioresponses() as mocked_responses:
         mocked_responses.get(
             f"{server_url()}{asset_name}/1",
@@ -256,7 +259,9 @@ def test_endpoint_get_asset_async(asset_name):
 
 
 def test_endpoint_get_list_async(asset_name):
-    loop = asyncio.get_event_loop()
+    loop = asyncio.new_event_loop()
+    asyncio.set_event_loop(loop)
+
     with aioresponses() as mocked_responses:
         mocked_responses.get(
             f"{server_url()}{asset_name}?offset=0&limit=2",