dict get for TEST_SERIALIZE in django.test.runner.setup_databases() always return True

[Guido Kollerie]

Given the following in settings.py:

DATABASES = {
    'default': {
        'ENGINE': 'django.db.backends.postgresql_psycopg2',
        # ....
        'TEST_SERIALIZE': False,
        }
}

The call serialize=connection.settings_dict.get("TEST_SERIALIZE", True) on line 299 in django.test.runner.setup_databases will always be set to True as TEST_SERIALIZE is somewhere transformed into:

settings_dict = { 
    'TEST': {
        'SERIALIZE`: True,
        # ...
    },
    # ...
}

Hence the key lookup always fails, and get() will use the supplied default value of True. A possible fix will be to rewrite that call to something like:

serialize=connection.settings_dict["TEST"].get("SERIALIZE", True)

[Tim Graham]

Added in [8c12d51e]. Not sure what the intention was, but documenting that it should be set in the test dictionary seems okay. It needs documentation as well.

[Tim Graham]

Andrew says, "I think it was meant to be a way to turn off the full serialization of data using loaddata during test runs; however it's useless due to TransactionTestCases, so perhaps should be excised."

Was there a use case where you needed to set it to False?

[Tim Graham]

I found how the option could be useful: ​PR

[Markus Holtermann]

Pull request looks good to me.

[Tim Graham <timograham@…>]

In a4f23eba2e690f1331fe35d6f29af42899e80395:

Fixed #23421 -- Corrected TEST SERIALIZE setting.

Thanks gkoller for the report.

[Tim Graham <timograham@…>]

In 02aa3e30e9cb8e5be5c33082c3548e2a6e1b91cb:

[1.7.x] Fixed #23421 -- Corrected TEST SERIALIZE setting.

Thanks gkoller for the report and Markus Holtermann for review.