Skip to content

Commit 9789728

Browse files
authored
[CI-3263] Update quiz_version_id tests (#76)
* Update quiz_version_id tests * Update failing test
1 parent 726c1e0 commit 9789728

2 files changed

Lines changed: 15 additions & 6 deletions

File tree

tests/modules/test_browse.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -326,7 +326,7 @@ def test_get_browse_results_with_valid_filter_name_filter_value_and_hidden_facet
326326
assert isinstance(response.get('response'), dict)
327327
assert isinstance(response.get('result_id'), str)
328328
assert response.get('request').get('fmt_options').get('hidden_facets') == hidden_facets
329-
assert response.get('response').get('facets')[0].get('name') == hidden_facets[0]
329+
assert any(facet.get('name') == hidden_facets[0] for facet in response.get('response').get('facets'))
330330

331331
def test_get_browse_results_with_valid_filter_name_filter_value_and_variations_map():
332332
'''Should return a response with a valid filter_name, filter_value, section, and variations_map''' # pylint: disable=line-too-long

tests/modules/test_quizzes.py

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@
1111
TEST_API_KEY = environ['TEST_REQUEST_API_KEY']
1212
TEST_API_TOKEN = environ['TEST_API_TOKEN']
1313
QUIZ_ID = 'test-quiz'
14-
QUIZ_VERSION_ID = 'e03210db-0cc6-459c-8f17-bf014c4f554d'
1514
QUIZ_SESSION_ID = 'ca380401-3805-4ded-8f28-638e5a4baa92'
1615
VALID_QUIZ_ANS = [[1], [1, 2], ['seen']]
1716
VALID_OPTIONS = { 'api_key': TEST_API_KEY, 'api_token': TEST_API_TOKEN}
@@ -31,13 +30,18 @@ def test_get_quiz_next_question_should_respond_with_matching_parameters():
3130
'''Should return a response with a matching quiz_id, quiz_version_id, quiz_session_id'''
3231

3332
quizzes = ConstructorIO(VALID_OPTIONS).quizzes
33+
initial_response = quizzes.get_quiz_next_question(QUIZ_ID, {
34+
'quiz_session_id': QUIZ_SESSION_ID
35+
})
36+
quiz_version_id = initial_response.get('quiz_version_id')
37+
3438
response = quizzes.get_quiz_next_question(QUIZ_ID, {
35-
'quiz_version_id': QUIZ_VERSION_ID,
39+
'quiz_version_id': quiz_version_id,
3640
'quiz_session_id': QUIZ_SESSION_ID
3741
})
3842

3943
assert response.get('quiz_id') == QUIZ_ID
40-
assert response.get('quiz_version_id') == QUIZ_VERSION_ID
44+
assert response.get('quiz_version_id') == quiz_version_id
4145
assert response.get('quiz_session_id') == QUIZ_SESSION_ID
4246

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

111115
quizzes = ConstructorIO(VALID_OPTIONS).quizzes
116+
initial_response = quizzes.get_quiz_next_question(QUIZ_ID, {
117+
'quiz_session_id': QUIZ_SESSION_ID
118+
})
119+
quiz_version_id = initial_response.get('quiz_version_id')
120+
112121
res = quizzes.get_quiz_results(QUIZ_ID, {
113122
'answers': VALID_QUIZ_ANS,
114-
'quiz_version_id': QUIZ_VERSION_ID,
123+
'quiz_version_id': quiz_version_id,
115124
'quiz_session_id': QUIZ_SESSION_ID})
116125

117126
assert res.get('quiz_id') == QUIZ_ID
118-
assert res.get('quiz_version_id') == QUIZ_VERSION_ID
127+
assert res.get('quiz_version_id') == quiz_version_id
119128
assert res.get('quiz_session_id') == QUIZ_SESSION_ID
120129

121130
def test_get_quiz_results_with_no_quiz_id():

0 commit comments

Comments
 (0)