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
f663a7fe
Commit
f663a7fe
authored
Nov 30, 2014
by
Sarina Canelake
Browse files
Options
Browse Files
Download
Email Patches
Plain Diff
s/pylint: disable=W0612/pylint: disable=unused-variable/
parent
7aab4151
Hide whitespace changes
Inline
Side-by-side
Showing
4 changed files
with
10 additions
and
10 deletions
+10
-10
common/lib/xmodule/xmodule/tests/test_progress.py
+5
-5
lms/djangoapps/instructor/views/api.py
+1
-1
lms/djangoapps/shoppingcart/models.py
+2
-2
lms/djangoapps/shoppingcart/tests/test_views.py
+2
-2
No files found.
common/lib/xmodule/xmodule/tests/test_progress.py
View file @
f663a7fe
...
...
@@ -23,12 +23,12 @@ class ProgressTest(unittest.TestCase):
def
test_create_object
(
self
):
# These should work:
prg1
=
Progress
(
0
,
2
)
# pylint: disable=
W0612
prg2
=
Progress
(
1
,
2
)
# pylint: disable=
W0612
prg3
=
Progress
(
2
,
2
)
# pylint: disable=
W0612
prg1
=
Progress
(
0
,
2
)
# pylint: disable=
unused-variable
prg2
=
Progress
(
1
,
2
)
# pylint: disable=
unused-variable
prg3
=
Progress
(
2
,
2
)
# pylint: disable=
unused-variable
prg4
=
Progress
(
2.5
,
5.0
)
# pylint: disable=
W0612
prg5
=
Progress
(
3.7
,
12.3333
)
# pylint: disable=
W0612
prg4
=
Progress
(
2.5
,
5.0
)
# pylint: disable=
unused-variable
prg5
=
Progress
(
3.7
,
12.3333
)
# pylint: disable=
unused-variable
# These shouldn't
self
.
assertRaises
(
ValueError
,
Progress
,
0
,
0
)
...
...
lms/djangoapps/instructor/views/api.py
View file @
f663a7fe
...
...
@@ -811,7 +811,7 @@ def get_sale_order_records(request, course_id): # pylint: disable=unused-argume
db_columns
=
[
x
[
0
]
for
x
in
query_features
]
csv_columns
=
[
x
[
1
]
for
x
in
query_features
]
sale_data
=
instructor_analytics
.
basic
.
sale_order_record_features
(
course_id
,
db_columns
)
header
,
datarows
=
instructor_analytics
.
csvs
.
format_dictlist
(
sale_data
,
db_columns
)
# pylint: disable=
W0612
header
,
datarows
=
instructor_analytics
.
csvs
.
format_dictlist
(
sale_data
,
db_columns
)
# pylint: disable=
unused-variable
return
instructor_analytics
.
csvs
.
create_csv_response
(
"e-commerce_sale_order_records.csv"
,
csv_columns
,
datarows
)
...
...
lms/djangoapps/shoppingcart/models.py
View file @
f663a7fe
...
...
@@ -1026,7 +1026,7 @@ class CourseRegCodeItem(OrderItem):
super
(
CourseRegCodeItem
,
cls
)
.
add_to_order
(
order
,
course_id
,
cost
,
currency
=
currency
)
item
,
created
=
cls
.
objects
.
get_or_create
(
order
=
order
,
user
=
order
.
user
,
course_id
=
course_id
)
# pylint: disable=
W0612
item
,
created
=
cls
.
objects
.
get_or_create
(
order
=
order
,
user
=
order
.
user
,
course_id
=
course_id
)
# pylint: disable=
unused-variable
item
.
status
=
order
.
status
item
.
mode
=
course_mode
.
slug
item
.
unit_cost
=
cost
...
...
@@ -1057,7 +1057,7 @@ class CourseRegCodeItem(OrderItem):
# file, but there's also a shared dependency on a random string generator which
# is in another PR (for another feature)
from
instructor.views.api
import
save_registration_code
for
i
in
range
(
total_registration_codes
):
# pylint: disable=
W0612
for
i
in
range
(
total_registration_codes
):
# pylint: disable=
unused-variable
save_registration_code
(
self
.
user
,
self
.
course_id
,
invoice
=
None
,
order
=
self
.
order
)
log
.
info
(
"Enrolled {0} in paid course {1}, paid ${2}"
...
...
lms/djangoapps/shoppingcart/tests/test_views.py
View file @
f663a7fe
...
...
@@ -1095,7 +1095,7 @@ class RegistrationCodeRedemptionCourseEnrollment(ModuleStoreTestCase):
cache
.
clear
()
url
=
reverse
(
'register_code_redemption'
,
args
=
[
'asdasd'
])
self
.
login_user
()
for
i
in
xrange
(
30
):
# pylint: disable=
W0612
for
i
in
xrange
(
30
):
# pylint: disable=
unused-variable
response
=
self
.
client
.
post
(
url
,
**
{
'HTTP_HOST'
:
'localhost'
})
self
.
assertEquals
(
response
.
status_code
,
404
)
...
...
@@ -1119,7 +1119,7 @@ class RegistrationCodeRedemptionCourseEnrollment(ModuleStoreTestCase):
cache
.
clear
()
url
=
reverse
(
'register_code_redemption'
,
args
=
[
'asdasd'
])
self
.
login_user
()
for
i
in
xrange
(
30
):
# pylint: disable=
W0612
for
i
in
xrange
(
30
):
# pylint: disable=
unused-variable
response
=
self
.
client
.
get
(
url
,
**
{
'HTTP_HOST'
:
'localhost'
})
self
.
assertEquals
(
response
.
status_code
,
404
)
...
...
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