Make pytest.mark.django_db(reset_sequences=True) imply transaction=True again

Regressed in `4.5.0`. Though it would have been better if it hadn't,
changing it now is a breaking change so needs to be fixed.

Author: bluetech

docs/changelog.rst

@@ -1,6 +1,16 @@
 Changelog
 =========
 
+v4.5.2 (2021-12-07)
+-------------------
+
+Bugfixes
+^^^^^^^^
+
+* Fix regression in v4.5.0 - ``pytest.mark.django_db(reset_sequence=True)`` now
+  implies ``transaction=True`` again.
+
+
 v4.5.1 (2021-12-02)
 -------------------
 

pytest_django/fixtures.py

@@ -167,7 +167,7 @@ def _django_db_helper(
             serialized_rollback,
         ) = False, False, None, False
 
-    transactional = transactional or (
+    transactional = transactional or reset_sequences or (
         "transactional_db" in request.fixturenames
         or "live_server" in request.fixturenames
     )

tests/test_database.py

@@ -287,11 +287,16 @@ def test_reset_sequences_disabled(self, request) -> None:
         marker = request.node.get_closest_marker("django_db")
         assert not marker.kwargs
 
-    @pytest.mark.django_db(transaction=True, reset_sequences=True)
+    @pytest.mark.django_db(reset_sequences=True)
     def test_reset_sequences_enabled(self, request) -> None:
         marker = request.node.get_closest_marker("django_db")
         assert marker.kwargs["reset_sequences"]
 
+    @pytest.mark.django_db(transaction=True, reset_sequences=True)
+    def test_transaction_reset_sequences_enabled(self, request) -> None:
+        marker = request.node.get_closest_marker("django_db")
+        assert marker.kwargs["reset_sequences"]
+
     @pytest.mark.django_db(databases=['default', 'replica', 'second'])
     def test_databases(self, request) -> None:
         marker = request.node.get_closest_marker("django_db")