Skip to content

ignore originalEmail attribute and fix test cases | 3.x#463

Merged
shailesh-google merged 1 commit intoapigee:3.xfrom
shailesh-google:developer-payload-original-email-mismatch-3.x
Feb 5, 2026
Merged

ignore originalEmail attribute and fix test cases | 3.x#463
shailesh-google merged 1 commit intoapigee:3.xfrom
shailesh-google:developer-payload-original-email-mismatch-3.x

Conversation

@shailesh-google
Copy link
Collaborator

@shailesh-google shailesh-google commented Feb 5, 2026

The 'originalEmail' attribute was causing a bad attribute error in Apigee X api during user creation from drupal side. This commit adds 'originalEmail' to the ignored attributes list during Developer entity serialization.

It also updates the test helper methods (alterArrays and alterObjects) to account for changes in Symfony Serializer 6.4.33+, ensuring test cases pass against the new behavior.

Fix: apigee/apigee-edge-drupal#1240

@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.63%. Comparing base (cd25d96) to head (2253c9c).
⚠️ Report is 1 commits behind head on 3.x.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##                3.x     #463      +/-   ##
============================================
+ Coverage     86.58%   86.63%   +0.05%     
- Complexity     1770     1772       +2     
============================================
  Files           349      349              
  Lines          4659     4663       +4     
============================================
+ Hits           4034     4040       +6     
+ Misses          625      623       -2     
Flag Coverage Δ
unittests 86.63% <100.00%> (+0.05%) ⬆️

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

Files with missing lines Coverage Δ
.../Api/Management/Controller/DeveloperController.php 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@shailesh-google shailesh-google changed the title ignore originalEmail attribute and fix test cases ignore originalEmail attribute and fix test cases | 3.x Feb 5, 2026
Copy link
Collaborator

@shishir-intelli shishir-intelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@shailesh-google shailesh-google merged commit 9ccf791 into apigee:3.x Feb 5, 2026
6 checks passed
@shailesh-google shailesh-google added this to the 3.0.11 milestone Feb 5, 2026
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.

2 participants