Woops, I didn't realize you meant the name was a duplicate. It wasn't
running. I renamed, fixed.


On Wed, Feb 26, 2014 at 12:03 PM, Will Daly <notifications@github.com>wrote:

> In apps/openassessment/peer/test/test_api.py:
>
> > @@ -211,6 +233,19 @@ def test_no_submissions_to_evaluate_for_tim(self):
> >          self._create_student_and_submission("Tim", "Tim's answer", MONDAY)
> >          peer_api.get_submission_to_assess(STUDENT_ITEM, 3)
> >
> > +    @patch.object(Assessment.objects, 'filter')
> > +    @raises(peer_api.PeerAssessmentInternalError)
> > +    def test_median_score_db_error(self, mock_filter):
> > +        mock_filter.side_effect = DatabaseError("Bad things happened")
> > +        tim = self._create_student_and_submission("Tim", "Tim's answer")
> > +        peer_api.get_assessment_median_scores(tim["uuid"], 3)
> > +
> > +    @patch.object(Assessment.objects, 'filter')
> > +    @raises(peer_api.PeerAssessmentInternalError)
> > +    def test_median_score_db_error(self, mock_filter):
> > +        mock_filter.side_effect = DatabaseError("Bad things happened")
> > +        tim = self._create_student_and_submission("Tim", "Tim's answer")
> > +        peer_api.get_assessments(tim["uuid"])
>
> I'm surprised that this isn't causing an issue due to the duplicate test
> names. Are you sure that both tests are running?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/edx/edx-tim/pull/48/files#r10088207>
> .
>