Skip to content

Conversation

@Gagaro
Copy link

@Gagaro Gagaro commented Apr 5, 2022

I had an issue when upgrading to Django 3.2, probably because of the memory isolation of setUpTestData objects.

The issue was that I had a StateWrapper instead of a State object during _pre_transition_checks, which caused an error even if I had the correct state for the transition.

Another issuer was that State instance were not considered equal, probably because they were copied.

The fixes I did are to compare State by name, and to get the State instance inside the StateWrapper if we need to.

Thanks for reviewing this PR 👍 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant