I found that locale tests are uses 'somehow works' approach to test export of strings.
Also there are some useless drupalLogout() at the end of a test
This patch could seriously help to continue work on #532512: Plural string storage is broken, editing UI is missing
| Comment | File | Size | Author |
|---|---|---|---|
| #22 | interdiff_21-22.txt | 369 bytes | sourabhjain |
| #22 | 848966-22.patch | 4.11 KB | sourabhjain |
| #21 | interdiff_848966_20-21.txt | 1.02 KB | ankithashetty |
| #21 | 848966-21.patch | 4.13 KB | ankithashetty |
| #20 | reroll_diff_19-20.txt | 2.4 KB | immaculatexavier |
Comments
Comment #1
andypostExport page have same forms so this fixes tests with change done in #601398: Simpletest does not allow assigning actions to triggers
Comment #2
plachsubscribe
Comment #3
andypostlocale-tests-d7.patch queued for re-testing.
Comment #4
thedavidmeister commentedI don't 100% understand the problem here, but looks like it might still be a problem in d8 as the code in the test is very similar, even if the comments have been updated to be less scary.
Comment #5
thedavidmeister commentedComment #16
quietone commentedThe patch is combing two Functional tests into one which makes sense in this case. However, this is a task.
The patch needs to be updated, adding tag.
Comment #17
immaculatexavier commentedRerolled patch against 10.0.x
Comment #18
immaculatexavier commentedComment #19
immaculatexavier commentedRerolled patch against #17 with diff
Comment #20
immaculatexavier commentedRerolled patch against #19 with diff
Comment #21
ankithashettyFixed the custom command errors in #20, thanks!
Comment #22
sourabhjainFixed the custom command errors in #21, thanks!