Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Increase coverage #652

Open
wants to merge 44 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
44 commits
Select commit Hold shift + click to select a range
55be6d8
fix: fix asserts and assert messages in tests
juancarlospaco Jan 7, 2025
9837eb4
fix: fix asserts and assert messages in tests
juancarlospaco Jan 7, 2025
e5e71c8
Merge branch 'main' of https://github.com/juancarlospaco/auth-py into…
juancarlospaco Jan 22, 2025
44b3ec3
feat: add missing cases for AuthError
juancarlospaco Jan 22, 2025
54dbe56
fix: fix style wip
juancarlospaco Jan 22, 2025
cc17ee9
fix: fix style wip
juancarlospaco Jan 22, 2025
48a01ca
feat: add tests WIP
juancarlospaco Jan 22, 2025
3ce9b0b
fix: fix style wip
juancarlospaco Jan 22, 2025
bbc7322
feat: add tests WIP
juancarlospaco Jan 22, 2025
6c6f758
feat: add tests WIP
juancarlospaco Jan 22, 2025
a9945b2
feat: add tests WIP
juancarlospaco Jan 22, 2025
9839c17
feat: add tests WIP
juancarlospaco Jan 22, 2025
d8b04e6
feat: add tests WIP
juancarlospaco Jan 22, 2025
c227ddf
fix: add missing await
juancarlospaco Jan 22, 2025
a374715
feat: add tests WIP
juancarlospaco Jan 22, 2025
bb12088
feat: add tests WIP
juancarlospaco Jan 22, 2025
a2144c5
feat: add tests WIP
juancarlospaco Jan 22, 2025
cb845ca
feat: add tests WIP
juancarlospaco Jan 22, 2025
8e38ce6
feat: add tests WIP
juancarlospaco Jan 22, 2025
5eff66b
feat: add tests WIP
juancarlospaco Jan 22, 2025
028c603
feat: add tests WIP
juancarlospaco Jan 22, 2025
9038348
feat: add tests WIP
juancarlospaco Jan 22, 2025
1c9179a
feat: add tests WIP
juancarlospaco Jan 22, 2025
cbd67e2
feat: add tests WIP
juancarlospaco Jan 22, 2025
24e6e04
feat: add tests WIP
juancarlospaco Jan 22, 2025
4e87c2e
feat: add tests WIP
juancarlospaco Jan 22, 2025
4d17c9f
feat: add tests WIP
juancarlospaco Jan 22, 2025
b757e7a
feat: add tests WIP
juancarlospaco Jan 23, 2025
aa24af7
feat: add tests WIP
juancarlospaco Jan 23, 2025
e502f97
feat: add tests WIP
juancarlospaco Jan 23, 2025
44ae129
feat: add tests WIP
juancarlospaco Jan 23, 2025
81a563f
fix: add missing await
juancarlospaco Jan 23, 2025
9cb8a4a
feat: add tests WIP
juancarlospaco Jan 23, 2025
a000d00
feat: add tests WIP
juancarlospaco Jan 23, 2025
9ba15e1
feat: add tests WIP
juancarlospaco Jan 23, 2025
7a397e0
feat: add tests WIP
juancarlospaco Jan 23, 2025
a33acf5
feat: add tests WIP
juancarlospaco Jan 23, 2025
618c2e5
feat: add tests WIP
juancarlospaco Jan 23, 2025
16eed14
feat: add tests WIP
juancarlospaco Jan 23, 2025
5fe8c04
feat: add tests WIP
juancarlospaco Jan 23, 2025
2d6464c
feat: add tests WIP
juancarlospaco Jan 23, 2025
0c30092
feat: add tests WIP
juancarlospaco Jan 23, 2025
2cd3a59
feat: add tests WIP
juancarlospaco Jan 23, 2025
124de1e
feat: add tests WIP
juancarlospaco Jan 23, 2025
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
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ jobs:
pip install pydantic==1.10.12
make tests_only
- name: Upload coverage to Coveralls
if: ${{ matrix.python-version }} == "3.12"
uses: coverallsapp/github-action@v2
with:
github-token: ${{ secrets.GITHUB_TOKEN }}
Expand Down
6 changes: 3 additions & 3 deletions supabase_auth/_async/gotrue_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -349,7 +349,7 @@ async def sign_in_with_sso(self, credentials: SignInWithSSOCredentials):
If you have built an organization-specific login page, you can use the
organization's SSO Identity Provider UUID directly instead.
"""
self._remove_session()
await self._remove_session()
provider_id = credentials.get("provider_id")
domain = credentials.get("domain")
options = credentials.get("options", {})
Expand All @@ -361,7 +361,7 @@ async def sign_in_with_sso(self, credentials: SignInWithSSOCredentials):
skip_http_redirect = options.get("skip_http_redirect", True)

if domain:
return self._request(
return await self._request(
"POST",
"sso",
body={
Expand All @@ -375,7 +375,7 @@ async def sign_in_with_sso(self, credentials: SignInWithSSOCredentials):
xform=parse_sso_response,
)
if provider_id:
return self._request(
return await self._request(
"POST",
"sso",
body={
Expand Down
Loading
Loading