Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Attached patch adding tests for the product node inline form UI.
Comment | File | Size | Author |
---|---|---|---|
ief_testing.patch | 5.33 KB | itamar | |
Comments
Comment #1
amitaibuWork in progress, test currently only includes testing the commerce product <-> node relation. Setting to needs review, for the testbot to trigger.
Comment #2
joachim CreditAttribution: joachim commentedThe base test for this module shouldn't require Commerce.
We should probably wait till #1658056: Add node integration gets in and use that as the base test, with a second class for Commerce integration.
Comment #3
bojanz CreditAttribution: bojanz commentedI'm fine with having a test that requires Commerce, we should just have one that doesn't as well.
I've yet to review these tests, will do so later.
Comment #4
joachim CreditAttribution: joachim commentedSure, but then the test class that requires Commerce probably shouldn't be called InlineEntityFormTestCase ...
Comment #5
bojanz CreditAttribution: bojanz commentedYep, that's true.
Comment #6
amitaibuCorrect status
Comment #6.0
amitaibuUpdated issue summary.