diff --git a/common/djangoapps/student/tests/test_userstanding.py b/common/djangoapps/student/tests/test_userstanding.py index 28ad48f3e1..c65bd9264e 100644 --- a/common/djangoapps/student/tests/test_userstanding.py +++ b/common/djangoapps/student/tests/test_userstanding.py @@ -57,6 +57,13 @@ class UserStandingTest(TestCase): # since it's only possible to disable accounts from lms, we're going # to skip tests for cms + @unittest.skipUnless(settings.ROOT_URLCONF == 'lms.urls', 'Test only valid in lms') + def test_can_access_manage_account_page(self): + response = self.admin_client.get(reverse('manage_user_standing'), { + 'user': self.admin, + }) + self.assertEqual(response.status_code, 200) + @unittest.skipUnless(settings.ROOT_URLCONF == 'lms.urls', 'Test only valid in lms') def test_disable_account(self): self.assertEqual( diff --git a/common/djangoapps/student/views.py b/common/djangoapps/student/views.py index 4083954f77..3efb1254e6 100644 --- a/common/djangoapps/student/views.py +++ b/common/djangoapps/student/views.py @@ -1346,7 +1346,7 @@ def manage_user_standing(request): headers = ['username', 'account_changed_by'] rows = [] for user in all_disabled_users: - row = [user.username, user.standing.all()[0].changed_by] + row = [user.username, user.standing.changed_by] rows.append(row) context = {'headers': headers, 'rows': rows}