Merge pull request #14278 from edx/jlajoie/TNL-6186

Change CourseTeamMembership emit to use proper id
This commit is contained in:
Jeff LaJoie
2017-01-06 13:33:46 -05:00
committed by GitHub
3 changed files with 3 additions and 2 deletions

View File

@@ -278,3 +278,4 @@ Casey Litton <caseylitton@gmail.com>
Jhony Avella <jhony.avella@edunext.co>
Tanmay Mohapatra <tanmaykm@gmail.com>
Brian Mesick <bmesick@edx.org>
Jeff LaJoie <jlajoie@edx.org>

View File

@@ -263,5 +263,5 @@ class CourseTeamMembership(models.Model):
membership.team.save()
membership.save()
emit_team_event('edx.team.activity_updated', membership.team.course_id, {
'team_id': membership.team_id,
'team_id': membership.team.team_id,
})

View File

@@ -172,7 +172,7 @@ class TeamSignalsTest(EventTestMixin, SharedModuleStoreTestCase):
self.assertGreater(now, team_membership.last_activity_at)
self.assert_event_emitted(
'edx.team.activity_updated',
team_id=team.id,
team_id=team.team_id,
)
else:
self.assertEqual(team.last_activity_at, team_last_activity)