Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
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
Prev Previous commit
[CHANGE] Implements code review changes - AIL-44
  • Loading branch information
rodrigo-pessoa-lrn committed Nov 6, 2024
commit 1af8eca7be5f1cbc2e13d90ce244667ad11b585b
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,8 @@ jobs:
with:
python-version: '3.9'

- name: Install Mypy Dependencies
run: pip install mypy
- name: Install Test Dependencies
run: pip install .[test]

- name: Type Checking with Mypy
run: mypy
Expand Down
3 changes: 2 additions & 1 deletion learnosity_sdk/request/init.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,8 @@ def __init__(
self.security = security.copy()
self.secret = secret
self.request = request
if request is not None and hasattr(request, 'copy'):
# TODO: Fix improper handling when request is a string
if isinstance(request, dict):
self.request = request.copy()
self.action = action

Expand Down
1 change: 1 addition & 0 deletions setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,7 @@
'pytest-subtests',
'responses >=0.8.1',
'types-requests',
'types-Jinja2',
'mypy',
]

Expand Down
4 changes: 1 addition & 3 deletions tests/unit/test_uuid.py
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
from typing import cast
import unittest
import re
from learnosity_sdk.utils import Uuid
Expand All @@ -13,6 +12,5 @@ def test_generate(self) -> None:
prog = re.compile('[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}')
result = prog.match(generated)

assert result != None
result = cast(re.Match[str], result)
assert result is not None
assert result.group() == generated
Loading