Debounce checkLoginState (testing reliability)

[fuzeman]

Had a closer look at the issue I was experiencing with tests stalling in #8992 and tracked it down to a redirection issue during login. I'm not sure if this is the best way to go about fixing this issue, ideally fixing this without a time delay would probably be better. Let me know what you think.

As displayed in the screenshots below: two checkLoginState calls are being fired as of 1b5019b, this can intermittently cause an unexpected redirection back to /home after playwright has clicked a tab on the header which in-turn stalls the test until timeout.

---

master / 1b5019b (test successful)

master / 1b5019b (test stalled)

---

PR / c180469 (test successful)

[netlify]

✅ Deploy Preview for inventree-web-pui-preview ready!

Name	Link
🔨 Latest commit	c180469
🔍 Latest deploy log	https://app.netlify.com/sites/inventree-web-pui-preview/deploys/679f5412872fee000812f406
😎 Deploy Preview	https://deploy-preview-9012--inventree-web-pui-preview.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 96 (no change from production) Accessibility: 89 (no change from production) Best Practices: 100 (no change from production) SEO: 78 (no change from production) PWA: - View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.50%. Comparing base (1b5019b) to head (c180469).
Report is 2 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #9012      +/-   ##
==========================================
- Coverage   85.55%   85.50%   -0.05%     
==========================================
  Files        1177     1177              
  Lines       52014    52015       +1     
  Branches     2114     2114              
==========================================
- Hits        44502    44478      -24     
- Misses       6978     7004      +26     
+ Partials      534      533       -1     

Flag	Coverage Δ
pui	68.90% <100.00%> (-0.25%)	⬇️

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.

[SchrodingersGat]

@fuzeman thanks for looking into this, I really appreciate it. The fix looks pretty clean to me, and having tested it locally it does appear to be more "stable". Fingers crossed that this addresses a good part of our race condition related issues!

@matmair @wolflu05 any thoughts on this one before merge?

[SchrodingersGat]

@fuzeman thanks for the contribution 😁

[github-actions]

💚 All backports created successfully

Status	Branch	Result
✅	0.17.x

Questions ?

Please refer to the Backport tool documentation and see the Github Action logs for details

[fuzeman]

I'll take a look at the other playwright test failures as well. I noticed some test failures had 401 Unauthorized ("Invalid token.") logged in the console, not sure if that's relevant yet though.

[matmair]

@fuzeman that is normal for some runs (login tests for example)