try to fix tests

[wolflu05]

This PR fixes the failing migrations tests first discovered after the merge of #7684 in: https://github.com/inventree/InvenTree/actions/runs/10069548588/job/27836706327
https://github.com/inventree/InvenTree/actions/runs/10069548597/job/27838227324

Somehow this seems to not be directly related to that PR, but was first caught after that merge. It may be related to wemake-services/django-test-migrations#294 ? Because when upgrading to 1.4.0 from 1.3.0 the issue is gone.

[netlify]

✅ Deploy Preview for inventree-web-pui-preview canceled.

Name	Link
🔨 Latest commit	71d9ab6
🔍 Latest deploy log	https://app.netlify.com/sites/inventree-web-pui-preview/deploys/66a2787f756b720008a88b8b

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.67%. Comparing base (96abd08) to head (71d9ab6).
Report is 3 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7730      +/-   ##
==========================================
- Coverage   83.70%   83.67%   -0.04%     
==========================================
  Files        1118     1118              
  Lines       49542    49542              
  Branches     1625     1625              
==========================================
- Hits        41469    41452      -17     
- Misses       7641     7662      +21     
+ Partials      432      428       -4     

Flag	Coverage Δ
backend	85.37% <ø> (ø)
migrations	42.47% <ø> (+0.02%)	⬆️
pui	64.47% <ø> (-0.37%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sonarcloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
1.8% Duplication on New Code

See analysis details on SonarCloud

[wolflu05]

Im trying to debug this issue. What I already found out:

1. when reverting back to the commit prior to my icons merge commit, the migration tests pass
2. when removing the separate database and state operations and place the unnecessary migrations directly in the operations list, it's still not working
3. somehow this seems to not be the the correct instance of part, was trying to debug that with my last run

[matmair]

Maybe the issue is in the testing code itself?

[SchrodingersGat]

I also found that updating to the latest version of django-test-migrations "fixed" the tests too....

[wolflu05]

You're right, updating "fixed" the issues. (See my last run). Feel free to change this pr if I did something wrong upgrading the package.

[matmair]

LGTM; great that we figured that out

[SchrodingersGat]

Nice work, thanks @wolflu05