Add test to ensure that AbstractFunctionParameterSniff::is_targetted_token() handles function calls with variable unpacking correctly

Author: rodrigoprimo

WordPress/Tests/Utils/I18nTextDomainFixerUnitTest.4.inc

@@ -256,6 +256,10 @@ esc_html_x(
 use function __;
 use function __ as my_function;
 add_action('my_action', __(...));
+// The test below ensures that the AbstractFunctionParameterSniff does not incorrectly ignore
+// function calls with variable unpacking. But it is a false positive in the context of the
+// I18nTextDomainFixer sniff and will be addressed in a future update.
+__(...$args);
 
 // phpcs:set WordPress.Utils.I18nTextDomainFixer old_text_domain[]
 // phpcs:set WordPress.Utils.I18nTextDomainFixer new_text_domain false

WordPress/Tests/Utils/I18nTextDomainFixerUnitTest.4.inc.fixed

@@ -261,6 +261,10 @@ esc_html_x(
 use function __;
 use function __ as my_function;
 add_action('my_action', __(...));
+// The test below ensures that the AbstractFunctionParameterSniff does not incorrectly ignore
+// function calls with variable unpacking. But it is a false positive in the context of the
+// I18nTextDomainFixer sniff and will be addressed in a future update.
+__(...$args, 'something-else');
 
 // phpcs:set WordPress.Utils.I18nTextDomainFixer old_text_domain[]
 // phpcs:set WordPress.Utils.I18nTextDomainFixer new_text_domain false

WordPress/Tests/Utils/I18nTextDomainFixerUnitTest.php

@@ -148,6 +148,7 @@ public function getErrorList( $testFile = '' ) {
 					241 => 1,
 					242 => 1,
 					245 => 1,
+					262 => 1,
 				);
 
 			default: