Opened 14 years ago
Closed 13 years ago
#16203 closed Bug (fixed)
'servers' test fails on Windows
| Reported by: | Dmitry Jemerov | Owned by: | nobody |
|---|---|---|---|
| Component: | contrib.admin | Version: | 1.3 |
| Severity: | Release blocker | Keywords: | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
The failure happens because the expected path is normalized with os.path.normcase, but the actually returned path is not.
C:\Python27\python.exe C:/src/django/django/tests/runtests.py --settings=test_sqlite servers
Creating test database for alias 'default'...
Creating test database for alias 'other'...
F
======================================================================
FAIL: test_media_urls (regressiontests.servers.tests.AdminMediaHandlerTests)
----------------------------------------------------------------------
Traceback (most recent call last):
File "C:\src\django\django\tests\regressiontests\servers\tests.py", line 62, in test_media_urls
"Got: %s, Expected: %s, URL was: %s" % (output, desired, url))
AssertionError: Got: C:\src\django\django\django\contrib\admin\media\css\base.css, Expected: c:\src\django\django\django\contrib\admin\media\css\base.css, URL was: /static/admin/css/base.css
----------------------------------------------------------------------
Ran 1 test in 0.001s
FAILED (failures=1)
Destroying test database for alias 'default'...
Destroying test database for alias 'other'...
Attachments (1)
Change History (3)
by , 14 years ago
| Attachment: | 16203.diff added |
|---|
comment:1 by , 14 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 13 years ago
| Component: | Uncategorized → contrib.admin |
|---|---|
| Resolution: | → fixed |
| Severity: | Normal → Release blocker |
| Status: | new → closed |
This was fixed at r17280.
Note:
See TracTickets
for help on using tickets.
Should os.path.normcase should be added in the code, as suggested by the patch, or in the test case?