Comments

harivenu_zyxware created an issue. See original summary.

harivenuv’s picture

Status: Active » Needs work
StatusFileSize
new4.78 KB
harivenuv’s picture

Status: Needs work » Needs review

Hi,

This test case is checking the checkout process.

Process flow listed below.

  1. Enable the uc_affirm module.
  2. Create admin user for testing.
  3. Enable the affirm payment method in payment configuration.
  4. Configure the setting in uc_affirm.
  5. Create an order.
  6. Fill the checkout pane information.
anish.a’s picture

Status: Needs review » Reviewed & tested by the community
StatusFileSize
new4.61 KB

There was some indentation issues in the code. Fixed it.

anish_zyxware’s picture

Status: Reviewed & tested by the community » Fixed

Merged. Thanks.

harivenuv’s picture

Status: Fixed » Needs work

Hi,

There is an issue in affirm unit test some of the test case breaks during testing. Most of the function are using API calls. so we can't directly create the test case without a mock function. So I will try to find another method to do the unit test more stable. Kindly pause the testing during this time.

anish_zyxware’s picture

Can you create a patch to remove test code?

anish.a’s picture

Status: Needs work » Needs review
StatusFileSize
new4.61 KB

Patch for removing tests.

  • anish.a committed 6fd73e6 on 7.x-1.x
    Issue #2730043 by anish.a, harivenu_zyxware: Remove tests
    
anish_zyxware’s picture

Status: Needs review » Fixed

  • anish.a committed ae5747a on 8.x-1.x authored by harivenu_zyxware
    Issue #2730043 by harivenu_zyxware, anish.a: Added a checkout test case
    
  • anish.a committed 6fd73e6 on 8.x-1.x
    Issue #2730043 by anish.a, harivenu_zyxware: Remove tests
    

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.