Add tests for Controller::__construct() (#219)

includes/class-controller.php

@@ -27,6 +27,8 @@ public function __construct() {
 	/**
 	 * Enable API Rewrites based on the Users settings.
 	 *
+	 * @codeCoverageIgnore Side-effects are from other methods already covered by tests.
+	 *
 	 * @return void
 	 */
 	private function api_rewrite() {
@@ -52,6 +54,8 @@ private function api_rewrite() {
 	/**
 	 * Ajax action to clear the Log file.
 	 *
+	 * @codeCoverageIgnore Cannot be tested. Results in script termination.
+	 *
 	 * @return void
 	 */
 	public function clear_log() {
@@ -82,6 +86,8 @@ public function clear_log() {
 	/**
 	 * Ajax action to read the Log file.
 	 *
+	 * @codeCoverageIgnore Cannot be tested. Results in script termination.
+	 *
 	 * @return void
 	 */
 	public function read_log() {

tests/phpunit/tests/Controller/Controller_ConstructTest.php

@@ -0,0 +1,44 @@
+<?php
+/**
+ * Class Controller_ConstructTest
+ *
+ * @package AspireUpdate
+ */
+
+/**
+ * Tests for Controller::__construct()
+ *
+ * @covers \AspireUpdate\Controller::__construct
+ */
+class Controller_ConstructTest extends WP_UnitTestCase {
+	/**
+	 * Test that hooks are added.
+	 *
+	 * @dataProvider data_hooks_and_methods
+	 *
+	 * @string $hook   The hook's name.
+	 * @string $method The method to hook.
+	 */
+	public function test_should_add_hooks( $hook, $method ) {
+		$controller = new AspireUpdate\Controller();
+		$this->assertIsInt( has_action( $hook, [ $controller, $method ] ) );
+	}
+
+	/**
+	 * Data provider.
+	 *
+	 * @return array[]
+	 */
+	public function data_hooks_and_methods() {
+		return [
+			'wp_ajax_aspireupdate_clear_log -> clear_log' => [
+				'hook'   => 'wp_ajax_aspireupdate_clear_log',
+				'method' => 'clear_log',
+			],
+			'wp_ajax_aspireupdate_read_log -> read_log'   => [
+				'hook'   => 'wp_ajax_aspireupdate_read_log',
+				'method' => 'read_log',
+			],
+		];
+	}
+}