Follow up for #1862512-15: Convert drupal_http_request usage in xmlrpc.module to Guzzle
Problem/Motivation
Frequently
Invalid values generate a list of form errors. Other EntityTranslationUITest.php 40 Drupal\translation_entity\Tests\EntityTranslationUITest->testTranslationUI()
is a random test failure... too frequent to be really random.
Proposed resolution
Let's look for other issues that have seen this and also see if we can find the cause.
Remaining tasks
TBD
User interface changes
No UI changes.
API changes
No API changes aniticipated.
Original report by @Berdir
in #1862512-15: Convert drupal_http_request usage in xmlrpc.module to Guzzle
Invalid values generate a list of form errors. Other EntityTranslationUITest.php 40 Drupal\translation_entity\Tests\EntityTranslationUITest->testTranslationUI()
Seen that one a lot recently, in various translation UI tests.
Comment | File | Size | Author |
---|---|---|---|
#2 | drupal-1965048-2-fail.patch | 976 bytes | olli |
#2 | drupal-1965048-2.patch | 978 bytes | olli |
Comments
Comment #1
BerdirComment #2
olli CreditAttribution: olli commentedI've seen this too.
Comment #3
olli CreditAttribution: olli commentedComment #4
BerdirNice find. Fix looks good to me, although it's not 100% clear to me what exactly causes it to work/fail but the old value certainly didn't make sense and the fail test shows that you can make it fail with the wrong random string :)
Testbot doesn't return the results somehow, checked with @rfay and re-tested the one that should pass. But if you click on it you can see that it's green/red as it should be.
Comment #5
ParisLiakos CreditAttribution: ParisLiakos commentedyes please!
and bumping to critical cause i ve seen it twice today
Comment #6
alexpottLovely...
Committed 7e7b0f0 and pushed to 8.x. Thanks!
Comment #7
tim.plunkettEnded up being http://drupalcode.org/project/drupal.git/commit/7389325