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
FileSize
746 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
FileSize
746 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
FileSize
644 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.