Opened 12 years ago
Closed 12 years ago
#20907 closed Bug (fixed)
Test failure on Oracle: test_attribute_name_not_python_keyword
| Reported by: | Shai Berger | Owned by: | Shai Berger |
|---|---|---|---|
| Component: | Database layer (models, ORM) | Version: | 1.4 |
| Severity: | Normal | Keywords: | oracle |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
Running updated stable/1.4.x on Oracle:
FAIL: test_attribute_name_not_python_keyword (regressiontests.inspectdb.tests.InspectDBTestCase)
----------------------------------------------------------------------
Traceback (most recent call last):
File "/home/django/django/django/test/testcases.py", line 861, in skip_wrapper
return test_func(*args, **kwargs)
File "/home/django/django/tests/regressiontests/inspectdb/tests.py", line 17, in test_attribute_name_not_python_keyword
out.getvalue())
AssertionError: 'people_pk = models.ForeignKey(InspectdbPeople, primary_key=True)' not found in
u"# This is an auto-generated Django...
...people_pk = models.ForeignKey(InspectdbPeople, unique=True)\n...
Change History (3)
comment:1 by , 12 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 12 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → assigned |
Cherry-picking the whole commit just for the Oracle problem seems wrong, as it includes fixes for all the backends.
I'll see about applying just the Oracle change.
comment:3 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Note:
See TracTickets
for help on using tickets.
This test failure was fixed by a18e43c5bb8cb7c82c84ca1b42d847a62ac9f077 -- I'm not sure we want to backport it to 1.4.x as it's a non-trivial change? It does seem to cherry-pick cleanly, but I haven't tried running the entire test suite.