#2232861: Create BrowserTestBase for web-testing on top of Mink just landed let's use the brand new BrowserTestBase to extend DynamicEntityReferenceTest instead of WebTestBase.
| Comment | File | Size | Author |
|---|---|---|---|
| #41 | convert_all_webtestbase-2469609-41.patch | 15.31 KB | jibran |
Comments
Comment #1
jibranLet's try this.
Comment #2
larowlanShould there be a namespace update here too?
Still commented out?
Comment #3
jibranThis fails locally. Perhaps, it has something to do with BTB.
Comment #4
jibranI think we can improve conversion documentation. Here are some suggestions.
We can add this to https://www.drupal.org/node/2469723
We can add this to https://www.drupal.org/node/2469723
We can add this to https://www.drupal.org/node/2469723
Perhaps this can also be added to https://www.drupal.org/node/2469723
We can add this to https://www.drupal.org/node/2469723
Comment #11
jibranPostpone on #2503547: Contrib can't run Functional tests.
Comment #16
jibranReroll.
Comment #19
jibranLet's test on new CI.
Comment #26
jibranRe-uploading for Drupal CI.
Comment #29
jibranRe-uploading the patch to test #2503547: Contrib can't run Functional tests is fixed or not.
Comment #31
jibranWe have four tests now which are extending WebTestBase.
Comment #32
anavarreI'm curious: what is blocking this issue, if anything? I don't see
BrowserTestBasebeing used in core except in\Drupal\Tests\simpletest\Functional\BrowserTestBaseTestand can hardly find any contrib module using them. Is there any reason for that, besides awareness?Comment #33
jibranI think you haven't seen #2735005: Convert all Simpletest web tests to BrowserTestBase (or UnitTestBase/KernelTestBase). We are making great progress in conversion over there in the child issues.
Comment #34
anavarreExcellent, thanks.
Comment #35
jibranLet's start fresh.
Comment #39
jibranFound a core bug #2751363: Don't allow test entities to share base table druing conversion.
Comment #41
jibranComment #42
jibranLet's fix the deprecated function calls in follow up.
Comment #43
dawehnerYou could use
$this->assertContainshereIf this is an ajax form, how does it work without the JavascripTestBase?
there is responseContains
Comment #44
dawehnerFeel free to fix those points after commit
Comment #45
jibranI'm planning to fix it 1 and 3 in follow up. For 2 we have a JTB already but if you are not using JTB then clicking the button will do the magic. Thanks for the RTBC.
Comment #47
jibranCommitted and pushed to 8.x-1.x and created #2751409: Remove the deprecated function calls from tests as follow up.