Fixed unit test after accessibility changes.

colinmurphy · colinmurphy · commit 34a25f1b89ae · 2025-11-14T16:44:24.000Z
diff --git a/plugins/wpgraphql-logging/src/Logger/Database/WordPressDatabaseEntity.php b/plugins/wpgraphql-logging/src/Logger/Database/WordPressDatabaseEntity.php
@@ -283,7 +283,6 @@ public static function get_schema(): string {
 	 */
 	public static function get_table_name(): string {
 		global $wpdb;
-		// @TODO - Check for multisite
 		return $wpdb->prefix . 'wpgraphql_logging';
 	}
 
diff --git a/plugins/wpgraphql-logging/tests/wpunit/Admin/View/List/ListTableTest.php b/plugins/wpgraphql-logging/tests/wpunit/Admin/View/List/ListTableTest.php
@@ -96,7 +96,9 @@ public function test_column_cb_returns_checkbox_for_valid_item(): void {
 
 		$result = $this->list_table->column_cb($entity);
 
-		$this->assertStringContainsString('<input type="checkbox"', $result);
+		$this->assertStringContainsString('<label class="screen-reader-text"', $result);
+		$this->assertStringContainsString('Select log entry 123', $result);
+		$this->assertStringContainsString('<input id="cb-select-123" type="checkbox"', $result);
 		$this->assertStringContainsString('value="123"', $result);
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,6 @@ public static function get_schema(): string {`
`283`	`283`	`*/`
`284`	`284`	`public static function get_table_name(): string {`
`285`	`285`	`global $wpdb;`
`286`		`- // @TODO - Check for multisite`
`287`	`286`	`return $wpdb->prefix . 'wpgraphql_logging';`
`288`	`287`	`}`
`289`	`288`
Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,9 @@ public function test_column_cb_returns_checkbox_for_valid_item(): void {`
`96`	`96`
`97`	`97`	`$result = $this->list_table->column_cb($entity);`
`98`	`98`
`99`		`- $this->assertStringContainsString('<input type="checkbox"', $result);`
	`99`	`+ $this->assertStringContainsString('<label class="screen-reader-text"', $result);`
	`100`	`+ $this->assertStringContainsString('Select log entry 123', $result);`
	`101`	`+ $this->assertStringContainsString('<input id="cb-select-123" type="checkbox"', $result);`
`100`	`102`	`$this->assertStringContainsString('value="123"', $result);`
`101`	`103`	`}`
`102`	`104`