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

Apply credit card and food voucher test on gatewayio account #115

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

carolkrroo
Copy link
Contributor

@carolkrroo carolkrroo commented Sep 29, 2021

What is the purpose of this pull request?

Adding the tests for the Credit Card + Food Voucher payment methods combination.
Re-reverting this PR, that was reverted here because it was testing something that wasn't in production yet.

What problem is this solving?

Covering the new scenario for purchase (Payment Method Association)

How should this be manually tested?

You can run it locally, check in the readme how to do it.

Screenshots or example usage

Screen.Recording.2021-09-28.at.1.15.59.PM.mov

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which has been updated accordingly.

@carolkrroo carolkrroo requested a review from a team as a code owner September 29, 2021 14:46
@cypress
Copy link

cypress bot commented Sep 29, 2021



Test summary

214 13 30 0Flakiness 1


Run details

Project Checkout UI (v5/v6)
Status Failed
Commit 4f6102f
Started Sep 29, 2021 7:48 PM
Ended Sep 29, 2021 7:57 PM
Duration 09:09 💡
OS Linux Ubuntu - 20.04
Browser Electron 87

View run in Cypress Dashboard ➡️


Failures

shipping/Delivery - No number - vtexgame1geo.test.js Failed
1 Delivery - Address with no number - vtexgame1geo > with only delivery
shipping/Delivery - No zipcode - vtexgame1geo.test.js Failed
1 Delivery - No zipcode - vtexgame1geo > with only delivery
shipping-preview/Delivery Only_Delivery Pickup - vtexgame1nolean.test.js Failed
1 Delivery Only + Delivery/Pickup - vtexgame1nolean > with only pickup
shipping/Delivery - Geolocation Input - vtexgame1geo.test.js Failed
1 Delivery - Geolocation Input - vtexgame1geo > with only delivery
shipping/Delivery - Peru - vtexgame1.test.js Failed
1 Delivery - Peru - vtexgame1 > with only delivery
shipping/Delivery - Paraguay Geolocation - vtexgame1geo.test.js Failed
1 Delivery - Paraguay - vtexgame1geo > with only delivery
shipping/Delivery - Second Purchase - Without Geolocation and PostalCode - Boleto - vtexgame1.test.js Failed
1 Delivery - 2P - Without Geolocation and PostalCode - Boleto - vtexgame1 > delivery with second purchase email
shipping/Delivery - Peru - vtexgame1geo.test.js Failed
1 Delivery - Peru - vtexgame1geo > with only delivery
shipping/Delivery - Second Purchase - Without Geolocation and PostalCode - Boleto - vtexgame1geo.test.js Failed
1 Delivery - 2P - Without Geolocation and PostalCode - Boleto - vtexgame1geo > delivery with second purchase email
shipping/Delivery - No city - vtexgame1geo.test.js Failed
1 Delivery - No city - vtexgame1geo > with only delivery
This comment includes only the first 10 test failures. See all 13 failures in the Cypress Dashboard.

Flakiness

tests/shipping/Pickup - Invoice no Zipcode - vtexgame1geoinvoice.test.js Flakiness
1 Pickup - Invoice no Zipcode - vtexgame1geoinvoice > with only pickup

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

Copy link
Contributor

@paul0vinicius paul0vinicius left a comment

Choose a reason for hiding this comment

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

LGTM ✨
Nice job!!!

Copy link

@ericbreno ericbreno left a comment

Choose a reason for hiding this comment

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

LGTM 👏

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