Because of the recent change in commerce 8.2.12, the order_email condition now assumes that all order customer email addresses are stored lowercase.
See https://cgit.drupalcode.org/commerce/commit/?id=245839b

This patch changes the email address given to the user at setup() to be lowercase.

Marked critical because this bug is causing the main branch to fail.

CommentFileSizeAuthor
#2 PASS-3032807-2.commerce_shipping.ShippingMethodStorageTest-needs-strtolower-user-email.patch1011 bytesNils Loewen
#2 3032807-2.commerce_shipping.ShippingMethodStorageTest-needs-strtolower-user-email.patch670 bytesNils Loewen

Comments

nlz created an issue. See original summary.

Nils Loewen’s picture

Nils Loewen
Primary language English
Location Kelowna, BC
commented
Issue summary: View changes
StatusFileSize
new 3032807-2.commerce_shipping.ShippingMethodStorageTest-needs-strtolower-user-email.patch670 bytes
new PASS-3032807-2.commerce_shipping.ShippingMethodStorageTest-needs-strtolower-user-email.patch1011 bytes

This failing patch has a dummy comment line to trigger the branch tests.

Nils Loewen’s picture

Nils Loewen
Primary language English
Location Kelowna, BC
commented
Issue summary: View changes

The last submitted patch, 2: 3032807-2.commerce_shipping.ShippingMethodStorageTest-needs-strtolower-user-email.patch, failed testing. View results

  • bojanz committed 67e0814 on 8.x-2.x authored by nlz 
    Issue #3032807 by nlz: ShippingMethodStorageTest needs strtolower user...
bojanz’s picture

bojanz commented
Status: Needs review » Fixed

Let's get the tests passing with 2.12 before researching future changes to Commerce.

Thanks!

Status: Fixed » Closed (fixed)

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