There's a method NodeBlocktest::testSearchFormBlock() which actually tests the syndicate block, not the search form block. It really should be renamed. This is in core/modules/node/lib/Drupal/node/Tests/NodeBlockTest.php.

Files: 
CommentFileSizeAuthor
#9 confusing_name-1824016-9.patch644 bytesdermario
PASSED: [[SimpleTest]]: [MySQL] 39,536 pass(es).
[ View ]
#6 confusing_name-1824016-6.patch746 bytesdermario
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch confusing_name-1824016-6.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
#2 confusing_name-1824016-2.patch746 bytesdermario
PASSED: [[SimpleTest]]: [MySQL] 46,248 pass(es).
[ View ]

Comments

dermario’s picture

Assigned:Unassigned» dermario

Let me give a try to fix it.

dermario’s picture

Status:Active» Needs review
StatusFileSize
new746 bytes
PASSED: [[SimpleTest]]: [MySQL] 46,248 pass(es).
[ View ]

I renamed the method to

function testSyndicateBlock() {
..
jhodgdon’s picture

Status:Needs review» Reviewed & tested by the community

That seems like a much more reasonable name. Thanks! This should be fine assuming the test bot agrees.

webchick’s picture

Status:Reviewed & tested by the community» Fixed

Uh, yes! Those two things aren't the same thing at all, are they? :)

Committed and pushed to 8.x. Thanks!

jhodgdon’s picture

Version:8.x-dev» 7.x-dev
Status:Fixed» Patch (to be ported)

Actually, it looks like this one could use a backport to d7 -- node.test has the NodeBlockTestCase class with the same method name in it.

dermario’s picture

Status:Patch (to be ported)» Needs review
StatusFileSize
new746 bytes
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch confusing_name-1824016-6.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

I fixed that for d7 too. Created that patch on 7.x-dev version.

Status:Needs review» Needs work

The last submitted patch, confusing_name-1824016-6.patch, failed testing.

jhodgdon’s picture

That's not a 7.x patch -- there is no core directory in 7.x.

dermario’s picture

Status:Needs work» Needs review
StatusFileSize
new644 bytes
PASSED: [[SimpleTest]]: [MySQL] 39,536 pass(es).
[ View ]

Yes obviously i was wrong. I am still learning git but i should have seen that by myself. Heres another patch for d7.

jhodgdon’s picture

Assigned:dermario» jhodgdon
Status:Needs review» Reviewed & tested by the community

Thanks! That patch looks right to me. I'll get it committed this week (avoiding non-essential core commits this week due to a big Views sprint going on).

jhodgdon’s picture

Assigned:jhodgdon» Unassigned
Status:Reviewed & tested by the community» Fixed

Thanks! 7.x is now one word clearer. :) Oops. I guess this wasn't strictly documentation, but it's a clean-up and backport of a committed 8.x patch. Hope it's OK that I committed it. ...

Automatically closed -- issue fixed for 2 weeks with no activity.