#35226: Copying a connection with a new alias fails -----------------------------------+-------------------------------------- Reporter: Florian Apolloner | Owner: nobody Type: Bug | Status: new Component: Testing framework | Version: 5.0 Severity: Release blocker | Resolution: Keywords: | Triage Stage: Unreviewed Has patch: 0 | Needs documentation: 0 Needs tests: 0 | Patch needs improvement: 0 Easy pickings: 0 | UI/UX: 0 -----------------------------------+--------------------------------------
Old description: > While working on connection pooling for postgres I ran into the following > regression (since > https://github.com/django/django/commit/8fb0be3500cc7519a56985b1b6f415d75ac6fedb > ): > > {{{ > from django.db import connection > from django.test import TestCase > > class Test(TestCase): > def test_something(self): > new_connection = connection.copy("asd") > with new_connection.cursor() as cursor: > print(cursor.execute("SELECT 1").fetchall()) > }}} > > On 5.0 this works fine, on main this fails with: > > django.test.testcases.DatabaseOperationForbidden: Database threaded > connections to 'asd' are not allowed in this test. Add 'asd' to > test_regression.Test.databases to ensure proper test isolation and > silence this failure. > > I cannot really add this to `databases` since this is a dynamically > created database. I am opening this as release blocker so it doesn't get > lost (I might be holding it wrong though) New description: While working on connection pooling for postgres I ran into the following regression (since 8fb0be3500cc7519a56985b1b6f415d75ac6fedb): {{{ from django.db import connection from django.test import TestCase class Test(TestCase): def test_something(self): new_connection = connection.copy("asd") with new_connection.cursor() as cursor: print(cursor.execute("SELECT 1").fetchall()) }}} On 5.0 this works fine, on main this fails with: > django.test.testcases.DatabaseOperationForbidden: Database threaded connections to 'asd' are not allowed in this test. Add 'asd' to test_regression.Test.databases to ensure proper test isolation and silence this failure. I cannot really add this to `databases` since this is a dynamically created database. I am opening this as release blocker so it doesn't get lost (I might be holding it wrong though) -- Comment (by Mariusz Felisiak): TBH, I don't see how we could handle this and keep 8fb0be3500cc7519a56985b1b6f415d75ac6fedb. The main question is, do we want to support something like this? Creating a copy of the connection copy is tricky, and if you change the alias in the meantime it becomes even more complicated. -- Ticket URL: <https://code.djangoproject.com/ticket/35226#comment:1> Django <https://code.djangoproject.com/> The Web framework for perfectionists with deadlines. -- You received this message because you are subscribed to the Google Groups "Django updates" group. To unsubscribe from this group and stop receiving emails from it, send an email to django-updates+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/0107018db85733bb-e57304f2-c124-4aa8-8ee2-0c001d7b8b2c-000000%40eu-central-1.amazonses.com.