toooooper / oppia

Automatically exported from code.google.com/p/oppia
Apache License 2.0
0 stars 0 forks source link

Need integration tests for ratings #714

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?

It's a bit of a pain to test stuff with ratings, especially now that we display 
these only at the end, because it requires playing through the whole 
exploration and logging in as (at least) five different users. This is the sort 
of thing that is likely to be missed in preparation for a release, and it seems 
like it might be a good idea to add an integration test for this (i.e. have 5 
different users rate stuff and check that the rating shows up in the gallery) 
in order to prevent unexpected regressions.

Original issue reported on code.google.com by s...@seanlip.org on 21 Apr 2015 at 4:54

GoogleCodeExporter commented 9 years ago
For the record, I change the MINIMUM_ACCEPTABLE_NUMBER_OF_RATINGS variable in 
components/ratings.js from 5 to 1 temporarily when I want to test, so it isn't 
too bad.

Original comment by jacobdav...@gmail.com on 21 Apr 2015 at 4:15