Skip to content

AMP-272 & AMP-170 changes#35

Merged
srivanimuddineni merged 10 commits intomainfrom
dev/AMP-272
Feb 6, 2026
Merged

AMP-272 & AMP-170 changes#35
srivanimuddineni merged 10 commits intomainfrom
dev/AMP-272

Conversation

@srivanimuddineni
Copy link
Contributor

@srivanimuddineni srivanimuddineni commented Feb 5, 2026 •
edited
Loading

AMP-272 & AMP-170 changes as follows

  1. Added Awaitility by removing RestTemplate for Material calls
  2. For Material-api-OpenAPI-Spec Java resttemplate generator doesn’t add builder() methods to the model classes, even with the extra config options. So Material.builder() and
    MaterialMetadata.builder() don’t exist, hence added TestMaterialBuilder for immutable
    useLombok : 'true',
    modelLombok : 'true',
    builderPattern : 'true'
  3. Added test coverage
  4. Added mapping to the outbound payload of callback

@srivanimuddineni srivanimuddineni requested a review from a team as a code owner February 5, 2026 14:01
@srivanimuddineni srivanimuddineni changed the title AMP-272 changes AMP-272 & AMP-170 changes Feb 5, 2026
@coling01
feature: amp-272 material service retry
7d40d52 
Make the retry intervals properties so we can inject into tests in the usual way
@coling01
feature: amp-272 material service retry
41156ba 
Make the retry intervals properties so we can inject into tests in the usual way
@srivanimuddineni srivanimuddineni merged commit dc4ce0f into main Feb 6, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coling01 coling01 coling01 approved these changes

@samirgarg samirgarg Awaiting requested review from samirgarg

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@srivanimuddineni @coling01