added test cases, remove activeClass from output if no target set

src/View/Helper/ActiveLinkHelper.php

@@ -42,6 +42,8 @@ class ActiveLinkHelper extends Helper {
 	public function link(array|string $title, array|string|null $url = null, array $options = []): string {
 		$currentUrl = $options['current'] ?? Router::parseRequest($this->getView()->getRequest());
 		if (!array_key_exists('target', $options) || !$currentUrl) {
+			unset($options['activeClass']);
+
 			return $this->Html->link($title, $url, $options);
 		}
 

tests/TestCase/View/Helper/ActiveLinkHelperTest.php

@@ -25,6 +25,10 @@ class ActiveLinkHelperTest extends TestCase {
 				'/',
 				['controller' => 'Tests', 'action' => 'index'],
 			);
+			$routes->get(
+				'/home',
+				['controller' => 'Tests', 'action' => 'index'],
+			);
 			$routes->get(
 				'/admin/users',
 				['prefix' => 'Admin', 'controller' => 'Users', 'action' => 'index'],
@@ -39,6 +43,20 @@ class ActiveLinkHelperTest extends TestCase {
 
 	/**
      * @return void
+     */
+	public function testLinkNoTargetSet(): void {
+		$request = new ServerRequest(['url' => '/']);
+		$view = new View($request);
+		$helper = new ActiveLinkHelper($view);
+
+		$result = $helper->link('goto', '/', [
+			'activeClass' => 'awesome-active-class',
+		]);
+		$this->assertStringNotContainsString('awesome-active-class', $result);
+	}
+
+	/**
+     * @return void
      */
 	public function testLinkMatchesString(): void {
 		$request = new ServerRequest(['url' => '/']);
@@ -50,6 +68,19 @@ class ActiveLinkHelperTest extends TestCase {
 			'activeClass' => 'awesome-active-class',
 		]);
 		$this->assertStringContainsString('awesome-active-class', $result);
+		$this->assertStringNotContainsString('activeClass', $result);
+
+		$request = new ServerRequest(['url' => '/home']);
+		$view = new View($request);
+		$helper = new ActiveLinkHelper($view);
+
+		$result = $helper->link('goto', '/', [
+			'target' => '/',
+			'activeClass' => 'awesome-active-class',
+		]);
+		$this->assertStringContainsString('awesome-active-class', $result);
+		$this->assertStringNotContainsString('activeClass', $result);
+
 	}
 
 	/**