Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(postman): Fix bluesnap refund status and folder rename to respect consecutive order #2392

Merged
merged 2 commits into from
Sep 30, 2023

Conversation

pixincreate
Copy link
Member

@pixincreate pixincreate commented Sep 29, 2023

Type of Change

  • Bugfix
  • CI/CD

Description

This PR fixes the Bluesnap refunds assertion issue where refunds now points to succeeded instead of Pending state.
Also, folder has been renamed from Scenario9 to Scenario8 and Scenario10 to Scenario9

Motivation and Context

With fe43458 (#2374) going in, Refund behaviour for Bluesnap changed from Pending state to actual status of the Refund, i.e., if the Payment succeeds, status is updated as Succeeded.

How did you test it?

Ran the collection:
image

Checklist

  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@pixincreate pixincreate added A-CI-CD Area: Continuous Integration/Deployment C-bug Category: Bug S-ready-for-merge C-test Category: Tests labels Sep 29, 2023
@pixincreate pixincreate added this to the September 2023 Milestone milestone Sep 29, 2023
@pixincreate pixincreate requested a review from a team as a code owner September 29, 2023 08:16
@pixincreate pixincreate self-assigned this Sep 29, 2023
@SanchithHegde SanchithHegde changed the title ci(postman): Fix bluesnap refund status and folder rename to respsect consequtive order ci(postman): Fix bluesnap refund status and folder rename to respsect consecutive order Sep 29, 2023
@SanchithHegde SanchithHegde changed the title ci(postman): Fix bluesnap refund status and folder rename to respsect consecutive order ci(postman): Fix bluesnap refund status and folder rename to respect consecutive order Sep 29, 2023
@SanchithHegde SanchithHegde added this pull request to the merge queue Sep 30, 2023
Merged via the queue into main with commit d8a1bdd Sep 30, 2023
10 checks passed
@SanchithHegde SanchithHegde deleted the collection-update branch September 30, 2023 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-CI-CD Area: Continuous Integration/Deployment C-bug Category: Bug C-test Category: Tests
Projects
No open projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

3 participants