def put(self, exploration_id):
"""Handles PUT requests for submitting ratings at the end of an
exploration.
"""
user_rating = self.payload.get('user_rating')
rating_services.assign_rating(
self.user_id, exploration_id, user_rating)
self.render_json({})
def test_rating_assignation(self):
"""Check ratings are correctly assigned to an exploration"""
self.exploration = exp_domain.Exploration.create_default_exploration(
self.EXP_ID, 'A title', 'A category')
exp_services.save_new_exploration(self.EXP_ID, self.exploration)
self.assertEqual(
rating_services.get_overall_ratings(self.EXP_ID),
{'1': 0, '2': 0, '3': 0, '4': 0, '5': 0})
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_1, self.EXP_ID), None)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 2)
rating_services.assign_rating(self.USER_ID_2, self.EXP_ID, 4)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 3)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_1, self.EXP_ID), 3)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_2, self.EXP_ID), 4)
self.assertEqual(
rating_services.get_overall_ratings(self.EXP_ID),
{'1': 0, '2': 0, '3': 1, '4': 1, '5': 0})
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 4)
self.assertEqual(
rating_services.get_overall_ratings(self.EXP_ID),
{'1': 0, '2': 0, '3': 0, '4': 2, '5': 0})
def test_rating_assignation(self):
"""Check ratings are correctly assigned to an exploration"""
self.exploration = exp_domain.Exploration.create_default_exploration(
self.EXP_ID, 'A title', 'A category')
exp_services.save_new_exploration(self.EXP_ID, self.exploration)
self.assertEqual(
rating_services.get_overall_ratings(self.EXP_ID),
{'1': 0, '2': 0, '3': 0, '4': 0, '5': 0})
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_1, self.EXP_ID), None)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 2)
rating_services.assign_rating(self.USER_ID_2, self.EXP_ID, 4)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 3)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_1, self.EXP_ID), 3)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_2, self.EXP_ID), 4)
self.assertEqual(
rating_services.get_overall_ratings(self.EXP_ID),
{'1': 0, '2': 0, '3': 1, '4': 1, '5': 0})
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 4)
self.assertEqual(
rating_services.get_overall_ratings(self.EXP_ID),
{'1': 0, '2': 0, '3': 0, '4': 2, '5': 0})
def test_time_of_ratings_recorded(self):
"""Check that the time a rating is given is recorded correctly."""
TIME_ALLOWED_FOR_COMPUTATION = datetime.timedelta(seconds=10)
self.exploration = exp_domain.Exploration.create_default_exploration(self.EXP_ID, "A title", "A category")
exp_services.save_new_exploration(self.EXP_ID, self.exploration)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 1)
first_rating_time = rating_services.get_when_rated(self.USER_ID_1, self.EXP_ID)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 3)
second_rating_time = rating_services.get_when_rated(self.USER_ID_1, self.EXP_ID)
self.assertLess(datetime.datetime.utcnow(), first_rating_time + TIME_ALLOWED_FOR_COMPUTATION)
self.assertLess(first_rating_time, second_rating_time)
self.assertLess(second_rating_time, datetime.datetime.utcnow())
def test_invalid_ratings_are_forbidden(self):
with self.assertRaisesRegexp(ValueError, "Expected a rating 1-5, received 0"):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 0)
with self.assertRaisesRegexp(ValueError, "Expected a rating 1-5, received 7"):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 7)
with self.assertRaisesRegexp(ValueError, "Expected the rating to be an integer, received 2"):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, "2")
with self.assertRaisesRegexp(ValueError, "Expected the rating to be an integer, received aaa"):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, "aaa")
def test_time_of_ratings_recorded(self):
"""Check that the time a rating is given is recorded correctly."""
TIME_ALLOWED_FOR_COMPUTATION = datetime.timedelta(seconds=10)
self.exploration = exp_domain.Exploration.create_default_exploration(
self.EXP_ID, 'A title', 'A category')
exp_services.save_new_exploration(self.EXP_ID, self.exploration)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 1)
first_rating_time = rating_services.get_when_rated(
self.USER_ID_1, self.EXP_ID)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 3)
second_rating_time = rating_services.get_when_rated(
self.USER_ID_1, self.EXP_ID)
self.assertLess(datetime.datetime.utcnow(),
first_rating_time + TIME_ALLOWED_FOR_COMPUTATION)
self.assertLess(first_rating_time, second_rating_time)
self.assertLess(second_rating_time, datetime.datetime.utcnow())
def forbid_invalid_ratings(self):
with self.assertRaisesRegexp(ValueError,
'Expected a rating 1-5, received: 0'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 0)
with self.assertRaisesRegexp(ValueError,
'Expected a rating 1-5, received: 7'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 7)
with self.assertRaisesRegexp(
ValueError,
'Expected the rating to be an integer, received: 2'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, '2')
with self.assertRaisesRegexp(ValueError,
'Expected a rating 1-5, received: aaa'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 'aaa')
def forbid_invalid_ratings(self):
with self.assertRaisesRegexp(
ValueError, 'Expected a rating 1-5, received: 0'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 0)
with self.assertRaisesRegexp(
ValueError, 'Expected a rating 1-5, received: 7'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 7)
with self.assertRaisesRegexp(
ValueError,
'Expected the rating to be an integer, received: 2'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, '2')
with self.assertRaisesRegexp(
ValueError, 'Expected a rating 1-5, received: aaa'):
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 'aaa')
def test_rating_assignations_do_not_conflict(self):
"""Check that ratings of different explorations are independant."""
EXP_ID_A = 'exp_id_A'
EXP_ID_B = 'exp_id_B'
self.exploration = exp_domain.Exploration.create_default_exploration(
EXP_ID_A, 'A title', 'A category')
exp_services.save_new_exploration(EXP_ID_A, self.exploration)
self.exploration = exp_domain.Exploration.create_default_exploration(
EXP_ID_B, 'A title', 'A category')
exp_services.save_new_exploration(EXP_ID_B, self.exploration)
rating_services.assign_rating(self.USER_ID_1, EXP_ID_A, 1)
rating_services.assign_rating(self.USER_ID_1, EXP_ID_B, 3)
rating_services.assign_rating(self.USER_ID_2, EXP_ID_A, 2)
rating_services.assign_rating(self.USER_ID_2, EXP_ID_B, 5)
self.assertEqual(
rating_services.get_user_specific_rating(self.USER_ID_1, EXP_ID_A),
1)
self.assertEqual(
rating_services.get_user_specific_rating(self.USER_ID_1, EXP_ID_B),
3)
self.assertEqual(
rating_services.get_user_specific_rating(self.USER_ID_2, EXP_ID_A),
2)
self.assertEqual(
rating_services.get_user_specific_rating(self.USER_ID_2, EXP_ID_B),
5)
self.assertEqual(rating_services.get_overall_ratings(EXP_ID_A), {
'1': 1,
'2': 1,
'3': 0,
'4': 0,
'5': 0
})
self.assertEqual(rating_services.get_overall_ratings(EXP_ID_B), {
'1': 0,
'2': 0,
'3': 1,
'4': 0,
'5': 1
})
def test_rating_assignation(self):
"""Check ratings are correctly assigned to an exploration"""
self.exploration = exp_domain.Exploration.create_default_exploration(self.EXP_ID, "A title", "A category")
exp_services.save_new_exploration(self.EXP_ID, self.exploration)
self.assertEqual(rating_services.get_overall_ratings(self.EXP_ID), {"1": 0, "2": 0, "3": 0, "4": 0, "5": 0})
self.assertEqual(rating_services.get_user_specific_rating(self.USER_ID_1, self.EXP_ID), None)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 2)
rating_services.assign_rating(self.USER_ID_2, self.EXP_ID, 4)
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 3)
self.assertEqual(rating_services.get_user_specific_rating(self.USER_ID_1, self.EXP_ID), 3)
self.assertEqual(rating_services.get_user_specific_rating(self.USER_ID_2, self.EXP_ID), 4)
self.assertEqual(rating_services.get_overall_ratings(self.EXP_ID), {"1": 0, "2": 0, "3": 1, "4": 1, "5": 0})
rating_services.assign_rating(self.USER_ID_1, self.EXP_ID, 4)
self.assertEqual(rating_services.get_overall_ratings(self.EXP_ID), {"1": 0, "2": 0, "3": 0, "4": 2, "5": 0})
def test_rating_assignations_do_not_conflict(self):
"""Check that ratings of different explorations are independant."""
EXP_ID_A = "exp_id_A"
EXP_ID_B = "exp_id_B"
self.exploration = exp_domain.Exploration.create_default_exploration(EXP_ID_A, "A title", "A category")
exp_services.save_new_exploration(EXP_ID_A, self.exploration)
self.exploration = exp_domain.Exploration.create_default_exploration(EXP_ID_B, "A title", "A category")
exp_services.save_new_exploration(EXP_ID_B, self.exploration)
rating_services.assign_rating(self.USER_ID_1, EXP_ID_A, 1)
rating_services.assign_rating(self.USER_ID_1, EXP_ID_B, 3)
rating_services.assign_rating(self.USER_ID_2, EXP_ID_A, 2)
rating_services.assign_rating(self.USER_ID_2, EXP_ID_B, 5)
self.assertEqual(rating_services.get_user_specific_rating(self.USER_ID_1, EXP_ID_A), 1)
self.assertEqual(rating_services.get_user_specific_rating(self.USER_ID_1, EXP_ID_B), 3)
self.assertEqual(rating_services.get_user_specific_rating(self.USER_ID_2, EXP_ID_A), 2)
self.assertEqual(rating_services.get_user_specific_rating(self.USER_ID_2, EXP_ID_B), 5)
self.assertEqual(rating_services.get_overall_ratings(EXP_ID_A), {"1": 1, "2": 1, "3": 0, "4": 0, "5": 0})
self.assertEqual(rating_services.get_overall_ratings(EXP_ID_B), {"1": 0, "2": 0, "3": 1, "4": 0, "5": 1})
def test_rating_assignations_do_not_conflict(self):
"""Check that ratings of different explorations are independant."""
EXP_ID_A = 'exp_id_A'
EXP_ID_B = 'exp_id_B'
self.exploration = exp_domain.Exploration.create_default_exploration(
EXP_ID_A, 'A title', 'A category')
exp_services.save_new_exploration(EXP_ID_A, self.exploration)
self.exploration = exp_domain.Exploration.create_default_exploration(
EXP_ID_B, 'A title', 'A category')
exp_services.save_new_exploration(EXP_ID_B, self.exploration)
rating_services.assign_rating(self.USER_ID_1, EXP_ID_A, 1)
rating_services.assign_rating(self.USER_ID_1, EXP_ID_B, 3)
rating_services.assign_rating(self.USER_ID_2, EXP_ID_A, 2)
rating_services.assign_rating(self.USER_ID_2, EXP_ID_B, 5)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_1, EXP_ID_A), 1)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_1, EXP_ID_B), 3)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_2, EXP_ID_A), 2)
self.assertEqual(
rating_services.get_user_specific_rating(
self.USER_ID_2, EXP_ID_B), 5)
self.assertEqual(
rating_services.get_overall_ratings(EXP_ID_A),
{'1': 1, '2': 1, '3': 0, '4': 0, '5': 0})
self.assertEqual(
rating_services.get_overall_ratings(EXP_ID_B),
{'1': 0, '2': 0, '3': 1, '4': 0, '5': 1})
def test_invalid_exploration_ids_are_forbidden(self):
with self.assertRaisesRegexp(
Exception, 'Invalid exploration id invalid_id'):
rating_services.assign_rating(self.USER_ID_1, 'invalid_id', 3)
def forbid_invalid_exploration_ids(self):
with self.assertRaisesRegexp(
Exception, 'Invalid exploration id invalid_id'):
rating_services.assign_rating(self.USER_ID_1, 'invalid_id', '3')
def test_invalid_exploration_ids_are_forbidden(self):
with self.assertRaisesRegexp(Exception, "Invalid exploration id invalid_id"):
rating_services.assign_rating(self.USER_ID_1, "invalid_id", 3)
