| Comment | File | Size | Author |
|---|---|---|---|
| #11 | testperf.search-SearchCommentTest.patch | 5.63 KB | sun |
| #10 | interdiff.txt | 4.52 KB | sun |
| #10 | testperf.search-SearchCommentTest.patch | 6.02 KB | sun |
| #3 | testperf.search-SearchCommentTest.patch | 1.54 KB | sun |
| testperf.search-SearchCommentTest.patch | 1003 bytes | sun |
Comments
Comment #2
jhodgdonThis test would need more changes than that to work. It is currently relying on the existence of the "Full HTML" text format and the "page" content type. That would need to be replicated. See in setUp():
Comment #3
sunComment #5
sunCreated #2256317: Remove static caching from WebTestBase::checkPermissions()
Comment #6
jhodgdonHm. The permission for the full HTML format is not working still with this patch. May need some kind of a cache clear after creating the format, to rebuild the permission names?
Ah, I see your other issue (cross-post as I was adding this comment). Sounds like we reached the same conclusion.
Comment #7
sun3: testperf.search-SearchCommentTest.patch queued for re-testing.
Comment #9
jhodgdonIt looks like it's now failing on:
It looks like the test is also relying on the "Basic HTML" format being present. In the Standard install profile, this format is defined, is automatically set up for any anonymous/authenticated user to use, and is the default text format. So I guess we'll need to set it up like this, and give the test user permission to use it.
Or... maybe just leave this using the Standard install profile?
Comment #10
sunComment #11
sunVictory! :-) Now without the run-tests.sh tweak.
Comment #12
jhodgdonGreat! Thanks sun! I've reviewed all the test changes, and they all look good to me.
Comment #13
catchCommitted/pushed to 8.x, thanks!