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 85.71429% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 43.63%. Comparing base (1f0f840) to head (e335e96).

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

Additional details and impacted files

@@              Coverage Diff              @@
##                4.x    #1153       +/-   ##
=============================================
+ Coverage     27.28%   43.63%   +16.34%     
  Complexity     3067     3067               
=============================================
  Files           342      342               
  Lines         11228    11228               
=============================================
+ Hits           3064     4899     +1835     
+ Misses         8164     6329     -1835     

Files with missing lines	Coverage Δ
src/Job/DeveloperSync.php	100.00% <100.00%> (+100.00%)	⬆️
src/Job/UserCreateUpdate.php	75.67% <100.00%> (+75.67%)	⬆️
src/Job/DeveloperCreateUpdate.php	36.00% <0.00%> (+36.00%)	⬆️

... and 113 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.