Closed (fixed)
Project:
Dynamic Entity Reference
Version:
8.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
12 Apr 2015 at 13:53 UTC
Updated:
2 Jul 2016 at 10:54 UTC
Jump to comment: Most recent, Most recent file
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.