OldFormForXTests.test_image_field fails under Oracle

[Aymeric Augustin]

See ​http://ci.django-cms.org/job/Django/database=oracle,python=python2.5/lastCompletedBuild/testReport/modeltests.model_forms.tests/OldFormForXTests/test_image_field/

The error is '' != None. It may be related to Oracle handling empty values differently from Django: ​https://docs.djangoproject.com/en/dev/ref/databases/#null-and-empty-strings

[Aymeric Augustin]

I can't reproduce this under Oracle 10. The CI server is running Oracle 11, it may explain the difference.

[Aymeric Augustin]

The problem is in modeltests/model_forms/tests.py, in a test for ImageField, at line 1320:

        self.assertEqual(instance.image.name, None)

In the same file, there's an equivalent test for FileField, at line 1192 — but it's written slightly differently:

        self.assertEqual(instance.file.name, '')

I suggest to make the test for ImageField similar to the test for FileField. See attached patch.

With this patch, the tests still pass under SQLite, MySQL, PostgreSQL and Oracle 10. I don't have access to Oracle 11, but it's very likely that the change will fix the failing test; the CI will tell us.

[Aymeric Augustin]

Also, this patch enforces a best practice, namely not using null=True on text-based fields.

[Aymeric Augustin]

IRC discussion about this patch:

2011-08-28 15:36:35 <cramm> about the test failure with Oracle 11 related to a nullable filefield
2011-08-28 15:37:18 <cramm> I've been reviewing it. I don't feel comfortable with adapting the tests to workaround that
2011-08-28 15:38:37 <cramm> I konw there is some recommendation in the charfield docs discouraging using null=True, but we have a handful of models with FileField's and ImageField with null=tru in out tests, and they work correctly
2011-08-28 15:39:57 <cramm> ... and a bunch of models with other CharField-derived fields also with null=True. In some tests we even test for ORM behavior when confronted with the null=Tree
2011-08-28 15:40:42 <mYk> cramm: the problem here is that we test that an ImageField with no value has its name set to ''
2011-08-28 15:40:47 <cramm> So, I'd prefer to see first why things don't fail in 10g and fail in 11g
2011-08-28 15:41:03 <mYk> but since the field is nullable the result should be None
2011-08-28 15:41:47 <mYk> the docs don't say exactly how <file>.name is supposed to behave, but basically, it's the string stored in the db
2011-08-28 15:42:46 <mYk> in short, I agree that I didn't nail the root cause, and the expected behavior of <file>.name is underdefined
2011-08-28 15:43:09 <cramm> I gree with the latter :)

[Erin Kelly]

I am able to reproduce the failure using Oracle 10g. This kind of thing has come up before in the tests. The usual fix is to predicate the expected output on the "interprets_empty_strings_as_nulls" feature.

[Erin Kelly]

In [16919]:

Fixed #16645: fixed a broken test to work in Oracle.