Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
e29ada8c
Commit
e29ada8c
authored
Dec 08, 2016
by
Nimisha Asthagiri
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
Grade Report rename "Not Accessible" to "Not Available".
parent
b24c2280
Hide whitespace changes
Inline
Side-by-side
Showing
3 changed files
with
12 additions
and
12 deletions
+12
-12
lms/djangoapps/instructor_task/tasks_helper.py
+2
-2
lms/djangoapps/instructor_task/tests/test_integration.py
+1
-1
lms/djangoapps/instructor_task/tests/test_tasks_helper.py
+9
-9
No files found.
lms/djangoapps/instructor_task/tasks_helper.py
View file @
e29ada8c
...
@@ -790,7 +790,7 @@ def upload_grades_csv(_xmodule_instance_args, _entry_id, course_id, _task_input,
...
@@ -790,7 +790,7 @@ def upload_grades_csv(_xmodule_instance_args, _entry_id, course_id, _task_input,
try
:
try
:
subsection_grade
=
course_grade
.
graded_subsections_by_format
[
assignment_type
][
subsection_location
]
subsection_grade
=
course_grade
.
graded_subsections_by_format
[
assignment_type
][
subsection_location
]
except
KeyError
:
except
KeyError
:
grade_results
.
append
([
u'Not A
ccessi
ble'
])
grade_results
.
append
([
u'Not A
vaila
ble'
])
else
:
else
:
if
subsection_grade
.
graded_total
.
attempted
:
if
subsection_grade
.
graded_total
.
attempted
:
grade_results
.
append
(
grade_results
.
append
(
...
@@ -968,7 +968,7 @@ def upload_problem_grade_report(_xmodule_instance_args, _entry_id, course_id, _t
...
@@ -968,7 +968,7 @@ def upload_problem_grade_report(_xmodule_instance_args, _entry_id, course_id, _t
try
:
try
:
problem_score
=
course_grade
.
locations_to_scores
[
block_location
]
problem_score
=
course_grade
.
locations_to_scores
[
block_location
]
except
KeyError
:
except
KeyError
:
earned_possible_values
.
append
([
u'Not A
ccessible'
,
u'Not Accessi
ble'
])
earned_possible_values
.
append
([
u'Not A
vailable'
,
u'Not Availa
ble'
])
else
:
else
:
if
problem_score
.
attempted
:
if
problem_score
.
attempted
:
earned_possible_values
.
append
([
problem_score
.
earned
,
problem_score
.
possible
])
earned_possible_values
.
append
([
problem_score
.
earned
,
problem_score
.
possible
])
...
...
lms/djangoapps/instructor_task/tests/test_integration.py
View file @
e29ada8c
...
@@ -646,7 +646,7 @@ class TestGradeReportConditionalContent(TestReportMixin, TestConditionalContent,
...
@@ -646,7 +646,7 @@ class TestGradeReportConditionalContent(TestReportMixin, TestConditionalContent,
{
{
self
.
student_b
:
{
self
.
student_b
:
{
u'Grade'
:
'0.0'
,
u'Grade'
:
'0.0'
,
u'Homework'
:
u'Not A
ccessi
ble'
,
u'Homework'
:
u'Not A
vaila
ble'
,
}
}
},
},
],
],
...
...
lms/djangoapps/instructor_task/tests/test_tasks_helper.py
View file @
e29ada8c
...
@@ -696,7 +696,7 @@ class TestProblemReportSplitTestContent(TestReportMixin, TestConditionalContent,
...
@@ -696,7 +696,7 @@ class TestProblemReportSplitTestContent(TestReportMixin, TestConditionalContent,
In order to verify that the behavior of the grade report is correct, we submit answers for problems
In order to verify that the behavior of the grade report is correct, we submit answers for problems
that the student won't have access to. A/B tests won't restrict access to the problems, but it should
that the student won't have access to. A/B tests won't restrict access to the problems, but it should
not show up in that student's course tree when generating the grade report, hence the Not A
ccessi
ble's
not show up in that student's course tree when generating the grade report, hence the Not A
vaila
ble's
in the grade report.
in the grade report.
"""
"""
# student A will get 100%, student B will get 50% because
# student A will get 100%, student B will get 50% because
...
@@ -726,7 +726,7 @@ class TestProblemReportSplitTestContent(TestReportMixin, TestConditionalContent,
...
@@ -726,7 +726,7 @@ class TestProblemReportSplitTestContent(TestReportMixin, TestConditionalContent,
unicode
(
self
.
student_a
.
id
),
unicode
(
self
.
student_a
.
id
),
self
.
student_a
.
email
,
self
.
student_a
.
email
,
self
.
student_a
.
username
,
self
.
student_a
.
username
,
u'1.0'
,
u'2.0'
,
u'2.0'
,
u'Not A
ccessible'
,
u'Not Accessi
ble'
u'1.0'
,
u'2.0'
,
u'2.0'
,
u'Not A
vailable'
,
u'Not Availa
ble'
]
]
)),
)),
dict
(
zip
(
dict
(
zip
(
...
@@ -734,7 +734,7 @@ class TestProblemReportSplitTestContent(TestReportMixin, TestConditionalContent,
...
@@ -734,7 +734,7 @@ class TestProblemReportSplitTestContent(TestReportMixin, TestConditionalContent,
[
[
unicode
(
self
.
student_b
.
id
),
unicode
(
self
.
student_b
.
id
),
self
.
student_b
.
email
,
self
.
student_b
.
email
,
self
.
student_b
.
username
,
u'0.5'
,
u'Not A
ccessible'
,
u'Not Accessi
ble'
,
u'1.0'
,
u'2.0'
self
.
student_b
.
username
,
u'0.5'
,
u'Not A
vailable'
,
u'Not Availa
ble'
,
u'1.0'
,
u'2.0'
]
]
))
))
])
])
...
@@ -869,19 +869,19 @@ class TestProblemReportCohortedContent(TestReportMixin, ContentGroupTestCase, In
...
@@ -869,19 +869,19 @@ class TestProblemReportCohortedContent(TestReportMixin, ContentGroupTestCase, In
user_grades
=
[
user_grades
=
[
{
{
'user'
:
self
.
staff_user
,
'user'
:
self
.
staff_user
,
'grade'
:
[
u'0.0'
,
u'Not A
ccessible'
,
u'Not Accessible'
,
u'Not Accessible'
,
u'Not Accessi
ble'
],
'grade'
:
[
u'0.0'
,
u'Not A
vailable'
,
u'Not Available'
,
u'Not Available'
,
u'Not Availa
ble'
],
},
},
{
{
'user'
:
self
.
alpha_user
,
'user'
:
self
.
alpha_user
,
'grade'
:
[
u'1.0'
,
u'2.0'
,
u'2.0'
,
u'Not A
ccessible'
,
u'Not Accessi
ble'
],
'grade'
:
[
u'1.0'
,
u'2.0'
,
u'2.0'
,
u'Not A
vailable'
,
u'Not Availa
ble'
],
},
},
{
{
'user'
:
self
.
beta_user
,
'user'
:
self
.
beta_user
,
'grade'
:
[
u'0.5'
,
u'Not A
ccessible'
,
u'Not Accessi
ble'
,
u'1.0'
,
u'2.0'
],
'grade'
:
[
u'0.5'
,
u'Not A
vailable'
,
u'Not Availa
ble'
,
u'1.0'
,
u'2.0'
],
},
},
{
{
'user'
:
self
.
non_cohorted_user
,
'user'
:
self
.
non_cohorted_user
,
'grade'
:
[
u'0.0'
,
u'Not A
ccessible'
,
u'Not Accessible'
,
u'Not Accessible'
,
u'Not Accessi
ble'
],
'grade'
:
[
u'0.0'
,
u'Not A
vailable'
,
u'Not Available'
,
u'Not Available'
,
u'Not Availa
ble'
],
},
},
]
]
...
@@ -1581,9 +1581,9 @@ class TestGradeReport(TestReportMixin, InstructorTaskModuleTestCase):
...
@@ -1581,9 +1581,9 @@ class TestGradeReport(TestReportMixin, InstructorTaskModuleTestCase):
u'Username'
:
self
.
student
.
username
,
u'Username'
:
self
.
student
.
username
,
u'Grade'
:
'0.13'
,
u'Grade'
:
'0.13'
,
u'Homework 1: Subsection'
:
'0.5'
,
u'Homework 1: Subsection'
:
'0.5'
,
u'Homework 2: Hidden'
:
u'Not A
ccessi
ble'
,
u'Homework 2: Hidden'
:
u'Not A
vaila
ble'
,
u'Homework 3: Unattempted'
:
u'Not Attempted'
,
u'Homework 3: Unattempted'
:
u'Not Attempted'
,
u'Homework 4: Empty'
:
u'Not A
ccessi
ble'
,
u'Homework 4: Empty'
:
u'Not A
vaila
ble'
,
u'Homework (Avg)'
:
'0.125'
,
u'Homework (Avg)'
:
'0.125'
,
},
},
],
],
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment