Code

Opened 5 years ago

Closed 5 years ago

#11050 closed (fixed)

Oracle fails introspection tests due to capitalization

Reported by: mboersma Owned by: nobody
Component: Database layer (models, ORM) Version: master
Severity: Keywords: Oracle oracle
Cc: ikelly Triage Stage: Unreviewed
Has patch: no Needs documentation:
Needs tests: Patch needs improvement:
Easy pickings: UI/UX:

Description

The Oracle backend fails two introspection tests currently, I think because it expects lower-case identifiers when Oracle returns ALL_CAPS names by default.

C:\projects\django-trunk\tests>runtests.py --settings=testsettings.oracle introspection
...
======================================================================
FAIL: test_get_table_description_names (regressiontests.introspection.tests.IntrospectionTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\projects\django-trunk\tests\regressiontests\introspection\tests.py", line 28, in _inner
    return func(*args, **kwargs)
  File "C:\projects\django-trunk\tests\regressiontests\introspection\tests.py", line 74, in test_get_table_description_names
    [f.column for f in Reporter._meta.fields])
AssertionError: ['ID', 'FIRST_NAME', 'LAST_NAME', 'EMAIL'] != ['id', 'first_name', 'last_name', 'email']
...
======================================================================
FAIL: test_table_names (regressiontests.introspection.tests.IntrospectionTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "C:\projects\django-trunk\tests\regressiontests\introspection\tests.py", line 28, in _inner
    return func(*args, **kwargs)
  File "C:\projects\django-trunk\tests\regressiontests\introspection\tests.py", line 47, in test_table_names
    "'%s' isn't in table_list()." % Reporter._meta.db_table)
AssertionError: 'introspection_reporter' isn't in table_list().

This happens in both the 1.0.x maintenance branch and the pre-1.1 trunk as of revision [10723]. It hasn't caused real-world problems as far as I know, but I've entered it here as a bug so we don't lose track of it.

Attachments (0)

Change History (1)

comment:1 Changed 5 years ago by mboersma

  • Resolution set to fixed
  • Status changed from new to closed

(In [10744]) [1.0.X] Fixed #11050 -- Oracle now passes all but one introspection unit test.

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed
as The resolution will be set. Next status will be 'closed'
The resolution will be deleted. Next status will be 'new'
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.