Problem/Motivation

Add test or the payment transaction source plugin.

Proposed resolution

Write the tests.

Remaining tasks

Patch
review
commit

CommentFileSizeAuthor
#2 2966207-2.patch3.97 KBquietone

Comments

quietone created an issue. See original summary.

quietone’s picture

quietone commented
Status: Active » Needs review
StatusFileSize
new 2966207-2.patch3.97 KB

Patch and small change to source plugin.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Needs review » Reviewed & tested by the community 
+++ b/modules/commerce/tests/src/Kernel/Plugin/migrate/source/d7/PaymentTransactionTest.php
@@ -0,0 +1,97 @@
+        'message_variables' => 'a:3:{s:7:"@number";s:16:"4111111111111111";s:6:"@month";s:2:"06";s:5:"@year";s:4:"2012";}',

CC# is actually in the database? Seems like a bad thing for PCI compliance.

heddn’s picture

heddn
Primary language English
Location Nicaragua
commented
Status: Reviewed & tested by the community » Fixed

  • heddn committed 2e88375 on 8.x-2.x authored by quietone 
    Issue #2966207 by quietone, heddn: Add payment transaction source plugin...

Status: Fixed » Closed (fixed)

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