Eliminate repated sync of developer/users with no reason

[mxr576]

Closes #1152

TODO

• Code clean up would be nice but the code as-is should also work

[mxr576]

Okay, so the same trick is not going to work inside \Drupal\apigee_edge\Job\DeveloperCreateUpdate::executeRequest() because the last modified date on the developer is a immutable and calculated on the server (as it should), so we either store the last sync date:

• in an attribute, which could lead to sync issues when multiple Drupal instances syncs from the same Apigee instance 🛑
• in the Drupal's database - this may work ✔️

or else...?

[kedarkhaire]

Hi @mxr576
Testcases are not passing - https://github.com/kedarkhaire/apigee-edge-drupal/actions/runs/15880850942/job/44780502524#step:13:43

Please check & update the test cases also as per the changes.

Thanks!

[codecov]

Codecov Report

❌ Patch coverage is 0% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 27.23%. Comparing base (f11c080) to head (54db20e).
⚠️ Report is 1 commits behind head on 4.x.

Files with missing lines	Patch %	Lines
src/Job/DeveloperSync.php	0.00%	19 Missing ⚠️
src/Job/UserCreateUpdate.php	0.00%	5 Missing ⚠️
src/Job/DeveloperCreateUpdate.php	0.00%	4 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##                4.x    #1153       +/-   ##
=============================================
- Coverage     43.55%   27.23%   -16.32%     
  Complexity     3112     3112               
=============================================
  Files           345      345               
  Lines         11382    11382               
=============================================
- Hits           4957     3100     -1857     
- Misses         6425     8282     +1857     

Files with missing lines	Coverage Δ
src/Job/DeveloperCreateUpdate.php	0.00% <0.00%> (-36.00%)	⬇️
src/Job/UserCreateUpdate.php	0.00% <0.00%> (-75.68%)	⬇️
src/Job/DeveloperSync.php	0.00% <0.00%> (-100.00%)	⬇️

... and 116 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[mxr576]

Thanks for the notice. I need to investigate what happens and what would be a proper fix

[kedarkhaire]

Thanks for the notice. I need to investigate what happens and what would be a proper fix

Marking this PR to draft as testcase fix need to be implemented.

[mxr576]

Hm, previously failing tests got fixed in HEAD? 🤔

This looks unrelated:

1) Drupal\Tests\apigee_edge_apiproduct_rbac\FunctionalJavascript\ApiProductRoleBasedAccessAuthenticatedInternalTest::testApiProductAccess
WebDriver\Exception\JavaScriptError: javascript error: jQuery is not defined