fix: Rename duplicate test method to testPrepareCardAssignedUsesRedirectUrl

jacksbox-cassandra · jacksbox-cassandra · commit b56448becd92 · 2026-02-25T22:48:35.000+01:00
Addresses review feedback: the added test method had the same name
as an existing method with a data provider, causing a duplicate
function declaration.

Renamed to testPrepareCardAssignedUsesRedirectUrl to clarify its
purpose (verifying that card-assigned notifications use the
redirect URL pattern).
diff --git a/tests/unit/Notification/NotifierTest.php b/tests/unit/Notification/NotifierTest.php
@@ -355,7 +355,7 @@ public function testPrepareBoardShared($withUserFound = true) {
 		$this->assertEquals($notification, $actualNotification);
 	}
 
-	public function testPrepareCardAssigned() {
+	public function testPrepareCardAssignedUsesRedirectUrl() {
 		/** @var INotification|MockObject $notification */
 		$notification = $this->createMock(INotification::class);
 		$notification->expects($this->once())

Original file line number	Diff line number	Diff line change
`@@ -355,7 +355,7 @@ public function testPrepareBoardShared($withUserFound = true) {`
`355`	`355`	`$this->assertEquals($notification, $actualNotification);`
`356`	`356`	`}`
`357`	`357`
`358`		`- public function testPrepareCardAssigned() {`
	`358`	`+ public function testPrepareCardAssignedUsesRedirectUrl() {`
`359`	`359`	`/** @var INotification\|MockObject $notification */`
`360`	`360`	`$notification = $this->createMock(INotification::class);`
`361`	`361`	`$notification->expects($this->once())`