Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.Simpletest has changed it's style a bit for 7.x which was then implemented in 6.x-2.x.
I believe this patch brings the current pathauto.test to the point where it matches this new style for tests. However, when I run the test I get redirected back to the parent screen without any report on the status. For the core block tests I get a little message "58 passes, 0 fails, 0 exceptions".
So, it appears there is still a bit more work to do somewhere.
Also, after patching it the pathauto.test file should be moved into the base level of the pathauto directory.
| Comment | File | Size | Author |
|---|---|---|---|
| #9 | 296460-update_simpletest_6x2x_style-9-d6.patch | 5.33 KB | Freso |
| #8 | 296460_update_2.patch | 5.32 KB | naxoc |
| #7 | 296460-update_simpletest_6x2x_style-7-d6.patch | 5.42 KB | Freso |
| 6x2x_style_tests_pathauto.patch | 4.56 KB | greggles |
Comments
Comment #1
Freso commentedSubscribing! (Even if I'm already getting announcements for all movements in the queue, I thought I'd add this one to my tracker as well. =))
Comment #2
ezra-g commentedBefore applying this patch simpletest.module didn't recognize pathauto's tests. After applying and testing pathauto, I get 21 passes and 1 fail:
Created permissions: administer pathauto, create type_vun content, edit own type_vun content, administer url aliases, create url aliases Role drupal_web_test_case.php 476 _drupalCreateRole
Comment #3
ezra-g commentedAlso, I get the same results for the core block test as greggles reported.
Comment #4
Freso commentedI assume we'll want this for Pathauto 6.x-1.x as well?
Comment #5
gregglesYes.
Comment #6
Freso commentedI'm currently getting "18 passes, 10 fails, and 0 exceptions" right now with SimpleTest 6.x-2.x. The old tests with SimpleTest 6.x-1.x gave all passes.
Comment #7
Freso commentedHowever. I'm also experiencing several errors on the SimpleTest side of things, so the Pathauto tests may not be responsible for all of it.
Anyway, I've updated the previous patch a slight bit.
Comment #8
naxoc commentedI made some changes to the test, and it runs with no errors now. I did not add any new tests, but I moved some code to the setUp function so that more test functions can be written easier.
Comment #9
Freso commentedCode looks sound and all tests pass. Committed (with some slight code style changes) to 6.x-2.x and 6.x-1.x. Tak for hjælpen, Camilla! :)