Skip to content

Conversation

@JiviteshT
Copy link
Contributor

Update zip creation logic and related tests

Modified FulfilmentDataService to include a new call to
CreateExchangeSetZip ensuring zip creation after extracting
and updating files. Updated tests in FulfilmentDataServiceTests
to verify multiple calls to zip creation and adjusted logging
tests to focus on "started" messages. Removed test for
"completed" logging message.

Modified FulfilmentDataService to include a new call to
`CreateExchangeSetZip` ensuring zip creation after extracting
and updating files. Updated tests in FulfilmentDataServiceTests
to verify multiple calls to zip creation and adjusted logging
tests to focus on "started" messages. Removed test for
"completed" logging message.
@JiviteshT JiviteshT requested review from a team and rockydevnet as code owners June 10, 2025 14:27
@snyk-io-eu
Copy link

snyk-io-eu bot commented Jun 10, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

code/snyk check is complete. No issues have been found. (View Details)

Copy link
Contributor

@hukho hukho left a comment

Choose a reason for hiding this comment

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

Looks good

@JiviteshT JiviteshT merged commit a4d3731 into main Jun 10, 2025
21 checks passed
@JiviteshT JiviteshT deleted the Feature/Abzu-220298-serial-aio-week-update-fix branch June 10, 2025 16:24
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.

3 participants