Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.For example, https://qa.drupal.org/pifr/test/588203:
| Message | Group | Filename | Line | Function | Status |
|---|---|---|---|---|---|
| HTTP response expected 204, actual 400 | Browser | NodeTest.php | 70 | Drupal\rest\Tests\NodeTest->testNodes() |  |
| Value 'a#^>Rn;g' is equal to value '?j6s;"{y'. | Other | NodeTest.php | 74 | Drupal\rest\Tests\NodeTest->testNodes() | |
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | drupal-2056713-3.will-fail.patch | 635 bytes | alexpott |
| #3 | drupal-2056713-3.will-pass.patch | 1.27 KB | alexpott |
| #1 | drupal-2056713-1.patch | 1.24 KB | olli |
Comments
Comment #1
olli CreditAttribution: olli commentedComment #2
BerdirAwesome, probably deserves a higher priority.
Can you create a patch that fails consistently, by changing the random string to a string that's known to fail?
Comment #3
alexpottHere a couple of patches that show that the fix in #1 is correct an is rtbc. The $new_title string is hard coded to 'jQ+X`T"6' in both instances. One patch has the fix form #1 and one does not.
The rtbc is for #1.
Comment #4
tim.plunkett+1, not doing the serialization manually is better anyway, regardless of the fail.
Comment #6
alexpottDrupal\taxonomy\Tests\VocabularyTest failed randomly.
The test did not complete due to a fatal error. Completion check VocabularyTest.php 82 Drupal\taxonomy\Tests\VocabularyTest->testTaxonomyAdminChangingWeights()
Comment #7
alexpott#3: drupal-2056713-3.will-pass.patch queued for re-testing.
Comment #8
alexpottSo back to rtbc
Comment #9
alexpottthanks olli, nice fix!
Committed 2c02cd8 and pushed to 8.x. Thanks!