Type hint "\Drupal\views\Plugin\views\ViewsHandlerInterface" missing for $handler

class was missing implementing interface. Just fixed that in this patch.

Patch updated.

Updated the last submitted patch.

Type hint "\Drupal\views\Plugin\views\ViewsHandlerInterface" missing for $handler. This is not correct as $handler is a stdClass object containing the value and operator properties. There is a "breakString" method in the "ViewsHandlerInterface" which is used to break x,y,z, and x+y+z into an array and $handler object is the result of this method.

Updated the code accordingly. Please review.

Adding interdiff file.

Added summary, I bet the issue is to add interface for assert, let's see this scope (how many tests will fail)

Probably we can't change argument but

Working on it

@andypost could you please review this patch .

The scope here has become very unclear. This could do with a title and IS update to make it clearer what we are trying to fix, a lot of unrelated clean up seems to have snuck into the patch

Sorry the changes #23 is not working

I agree with @sja112

 $handler = HandlerBase::breakString('word1 word2+word');

its returning object of Drupal\Tests\views\Functional\Handler\HandlerTest
so its good to remove @param \Drupal\views\Plugin\views\ViewsHandlerInterface $handler from protected function assertEqualValue

@kishor_kolekar that's why assert should use interface

Updated doc block and added ref to interface that defines result of handler

Added extra assertions for it,

@pavnish the problem with your patch is a lot of unrelated changes when issue exactly about doc block, please check the guide to clarify https://www.drupal.org/core/scope#creep

PS: interdiff from #19, fixed IS

@andypost I agree with your comment .

This has become outdated. The method for which the documentation is being updated was removed in #3231781: [D9.3 beta - w/c Nov 8, 2021] Remove remaining uses of t() in tests

Thanks!