Skip to content

[CI-3263] Update quiz_version_id tests #76

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.

Sign up for GitHub

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

Merged
esezen merged 2 commits into from
Aug 26, 2024
Merged

[CI-3263] Update quiz_version_id tests #76

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4c5350e
Update quiz_version_id tests
esezen Aug 20, 2024
011166b
Update failing test
esezen Aug 20, 2024
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
2 changes: 1 addition & 1 deletion tests/modules/test_browse.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ def test_get_browse_results_with_valid_filter_name_filter_value_and_hidden_facet
assert isinstance(response.get('response'), dict)
assert isinstance(response.get('result_id'), str)
assert response.get('request').get('fmt_options').get('hidden_facets') == hidden_facets
assert response.get('response').get('facets')[0].get('name') == hidden_facets[0]
assert any(facet.get('name') == hidden_facets[0] for facet in response.get('response').get('facets'))

def test_get_browse_results_with_valid_filter_name_filter_value_and_variations_map():
'''Should return a response with a valid filter_name, filter_value, section, and variations_map''' # pylint: disable=line-too-long
Expand Down
19 changes: 14 additions & 5 deletions tests/modules/test_quizzes.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
TEST_API_KEY = environ['TEST_REQUEST_API_KEY']
TEST_API_TOKEN = environ['TEST_API_TOKEN']
QUIZ_ID = 'test-quiz'
QUIZ_VERSION_ID = 'e03210db-0cc6-459c-8f17-bf014c4f554d'
QUIZ_SESSION_ID = 'ca380401-3805-4ded-8f28-638e5a4baa92'
VALID_QUIZ_ANS = [[1], [1, 2], ['seen']]
VALID_OPTIONS = { 'api_key': TEST_API_KEY, 'api_token': TEST_API_TOKEN}
Expand All @@ -31,13 +30,18 @@ def test_get_quiz_next_question_should_respond_with_matching_parameters():
'''Should return a response with a matching quiz_id, quiz_version_id, quiz_session_id'''

quizzes = ConstructorIO(VALID_OPTIONS).quizzes
initial_response = quizzes.get_quiz_next_question(QUIZ_ID, {
'quiz_session_id': QUIZ_SESSION_ID
})
quiz_version_id = initial_response.get('quiz_version_id')

response = quizzes.get_quiz_next_question(QUIZ_ID, {
'quiz_version_id': QUIZ_VERSION_ID,
'quiz_version_id': quiz_version_id,
'quiz_session_id': QUIZ_SESSION_ID
})

assert response.get('quiz_id') == QUIZ_ID
assert response.get('quiz_version_id') == QUIZ_VERSION_ID
assert response.get('quiz_version_id') == quiz_version_id
assert response.get('quiz_session_id') == QUIZ_SESSION_ID

def test_get_quiz_next_question_with_answer_parameter():
Expand Down Expand Up @@ -109,13 +113,18 @@ def test_get_quiz_results_should_respond_with_matching_parameters():
'''Should return a response with a matching quiz_id, quiz_version_id, quiz_session_id'''

quizzes = ConstructorIO(VALID_OPTIONS).quizzes
initial_response = quizzes.get_quiz_next_question(QUIZ_ID, {
'quiz_session_id': QUIZ_SESSION_ID
})
quiz_version_id = initial_response.get('quiz_version_id')

res = quizzes.get_quiz_results(QUIZ_ID, {
'answers': VALID_QUIZ_ANS,
'quiz_version_id': QUIZ_VERSION_ID,
'quiz_version_id': quiz_version_id,
'quiz_session_id': QUIZ_SESSION_ID})

assert res.get('quiz_id') == QUIZ_ID
assert res.get('quiz_version_id') == QUIZ_VERSION_ID
assert res.get('quiz_version_id') == quiz_version_id
assert res.get('quiz_session_id') == QUIZ_SESSION_ID

def test_get_quiz_results_with_no_quiz_id():
Expand Down