The 2026 Drupal Business Survey is open. Your response will help shape the future of the Drupal business ecosystem.Provide some Simpletest for the module on D7 to help with regression testing.
| Comment | File | Size | Author |
|---|---|---|---|
| #10 | eck-n2889325-10.interdiff.txt | 372 bytes | damienmckenna |
| #10 | eck-n2889325-10.patch | 24.16 KB | damienmckenna |
Comments
Comment #2
damienmckennaTests written for #1798646.
Comment #3
damienmckennaThe tests cover some basic CRUD actions, translation using Entity Translation, and then translation when attached to a node using EntityReference. So far the basic functionality appears to be working well, even with translations. There's a whole lot more to do to build these out, but this should help with regression testing and create a platform to build upon for the additional changes that are to be added to the 7.x-2.x branch.
Comment #4
damienmckennaICBW but I think if we also had tests for the permissions structure we'd then be able to ditch the behat tests entirely.
Comment #5
damienmckennaWrong filenames. Duh.
Comment #6
damienmckennaOnly one test file was picked up that time, lets see if this fixes the problem.
Comment #7
damienmckennaFurther improvements.
Comment #9
damienmckennaDuh, Damien, OOP basics.
Comment #10
damienmckennaIt isn't picking up the translation tests because of a missing test dependency.
Comment #11
damienmckennaOnce the test_dependencies[] lines are committed it'll then run the translation tests too.
Comment #12
damienmckennaRerunning the tests to make sure it still all works.
Comment #13
pifagor commented+1
Comment #14
pifagor commentedComment #16
pifagor commentedComment #17
damienmckennaExcellent! Thanks.