Skip to content

Commit ec5f363

Browse files
at88mphbsipocz
authored andcommitted
Simplify tests.
1 parent c072559 commit ec5f363

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

astroquery/alma/tests/test_alma_auth.py

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@
22
from ..core import AlmaAuth
33
from ...exceptions import LoginError
44

5-
import json
65
import pytest
76
from unittest.mock import Mock
87

@@ -45,11 +44,10 @@ def _requests_mock_err(method, url, **kwargs):
4544

4645
def test_login_bad_error():
4746
def _response_json():
48-
obj = {
47+
return {
4948
'error': 'Badness',
5049
'error_description': 'Something very bad'
5150
}
52-
return json.dumps(obj, indent=4)
5351

5452
def _requests_mock_err(method, url, **kwargs):
5553
response = Mock()
@@ -69,10 +67,9 @@ def _requests_mock_err(method, url, **kwargs):
6967

7068
def test_login_missing_token():
7169
def _response_json():
72-
obj = {
70+
return {
7371
'irrlevant': 'Weird',
7472
}
75-
return json.dumps(obj, indent=4)
7673

7774
def _requests_mock_err(method, url, **kwargs):
7875
response = Mock()
@@ -93,10 +90,9 @@ def _requests_mock_err(method, url, **kwargs):
9390

9491
def test_login_success():
9592
def _response_json():
96-
obj = {
93+
return {
9794
'access_token': 'MYTOKEN'
9895
}
99-
return json.dumps(obj, indent=4)
10096

10197
def _requests_mock_good(method, url, **kwargs):
10298
response = Mock()

0 commit comments

Comments
 (0)