Code

Opened 4 years ago

Closed 4 years ago

#13638 closed (fixed)

Serializer regress tests have uncessary transaction management that is very slow

Reported by: Alex Owned by: nobody
Component: Testing framework Version: 1.1
Severity: Keywords:
Cc: Triage Stage: Unreviewed
Has patch: yes Needs documentation:
Needs tests: Patch needs improvement:
Easy pickings: UI/UX:

Description

We can just delete this and it still pases for me (tested under postgres and sqlite).

Attachments (1)

django-serializer-regress-tests.diff (5.4 KB) - added by Alex 4 years ago.

Download all attachments as: .zip

Change History (2)

Changed 4 years ago by Alex

comment:1 Changed 4 years ago by russellm

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

(In [13308]) Fixed #13638 -- Refactored the serializers_regress tests to avoid the use of flush, and make better use of the transactional capabilities of django.test.TestCase. Thanks to Alex Gaynor for the patch.

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.