Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.