[Tests]: Increase PHPUnit coverage.

[costdev]

This is a tracking issue for increasing PHPUnit test coverage of the codebase.

Below is an outline of each class, its assignee, its dependencies, and its methods.

---

Admin_Settings ✅

• Assigned to @costdev
• Depends on [Tests] Introduce AdminSettings_UnitTestCase. #212

Click to view the methods

• ::__construct() - See Add tests for Admin_Settings::__construct() #229
• ::get_instance() - See Add tests for Admin_Settings::get_instance() #230
• ::get_default_settings() - See Add more tests for Admin_Settings::get_setting() #234
  • This is a private method.
  • It can be covered by ::get_setting() tests.
• ::reset_settings() - See Add tests for Admin_Settings::reset_settings() #231
• ::delete_all_settings() - See Add tests for Admin_Settings::delete_all_settings() #232
• ::admin_notices() - See Add tests for Admin_Settings::admin_notices() #233
• ::get_setting() - See Add more tests for Admin_Settings::get_setting() #234
  • This is partially covered by existing tests.
• ::get_settings_from_config_file() - See Add more tests for Admin_Settings::get_setting() #234
  • This is a private method.
  • It can be covered by ::get_setting() tests.
• ::update_settings() - See Add tests for Admin_Settings::update_settings() #235
• ::register_admin_menu() - See Add tests for Admin_Settings::register_admin_menu() #236
• ::admin_enqueue_scripts() - See Add tests for Admin_Settings::admin_enqueue_scripts() #237
• ::the_settings_page() - See Add tests for Admin_Settings::the_settings_page() #238
• ::register_settings() - See Add tests for Admin_Settings::register_settings() #241
• ::add_settings_field_callback() - See Code Coverage: Ignore Admin_Settings::add_settings_field_callback() #251
  • This method outputs only HTML and has no conditions.
  • This should not be covered by PHPUnit tests. We can test with E2E instead when we get that set up.
• ::sanitize_settings() - See Add tests for Admin_Settings::sanitize_settings() #239

---

API_Rewrite ✅

• Assigned to @costdev

Click to view the methods

• ::__construct() - See Add tests for API_Rewrite::__construct() #217
• ::pre_http_request() - See Add tests for API_Rewrite::pre_http_request() #218

---

AspireCloud_API

• Not implemented at this time.
• No PHPUnit tests needed.

---

Branding ✅

• Already covered.

Click to view the methods

• ::__construct()
• ::get_instance()
• ::admin_enqueue_scripts()
• ::output_admin_notice()

---

Controller ✅

• Assigned to @costdev

Click to view the methods

• ::__construct() - See Add tests for Controller::__construct() #219
• ::api_rewrite() - See Add tests for Controller::__construct() #219
  • This is a private method.
  • Cannot be tested without testing other class' methods which will already be tested.
  • This should not be covered by PHPUnit tests.
• ::clear_log() - See Add tests for Controller::__construct() #219
  • This is an AJAX callback which calls wp_send_json_success|error(), which call die() internally.
  • It cannot be tested because script execution is terminated.
  • This cannot be covered by PHPUnit tests.
• ::read_log() - See Add tests for Controller::__construct() #219
  • This is an AJAX callback which calls wp_send_json_success|error(), which call die() internally.
  • It cannot be tested because script execution is terminated.
  • This cannot be covered by PHPUnit tests.

---

Debug ✅

• Assigned to @costdev
• Want to handle this one? Drop a comment on this issue!

Click to view the methods

• ::get_file_path()
  • This is a private method.
  • It can be covered by ::read(), ::clear() or ::log() tests.
  • See Add tests for Debug::read() #245
  • See Add tests for Debug::clear() #246
  • See Add tests for Debug::log() #247
• ::init_filesystem()
  • This is a private method.
  • It can be covered by ::read(), ::clear() or ::log() tests.
  • See Add tests for Debug::read() #245
  • See Add tests for Debug::clear() #246
  • See Add tests for Debug::log() #247
• ::verify_filesystem()
  • This is a private method.
  • It can be covered by ::read(), ::clear() or ::log() tests.
  • See Add tests for Debug::read() #245
  • See Add tests for Debug::clear() #246
  • See Add tests for Debug::log() #247
• ::read() - See Add tests for Debug::read() #245
• ::clear() - See Add tests for Debug::clear() #246
• ::log() - See Add tests for Debug::log() #247
• ::format_message() - See Add tests for Debug::log() #247
  • This is a private method.
  • It can be covered by ::log() tests.
• ::log_string() - See Add tests for Debug::log_string() #248
• ::log_request() - See Add tests for Debug::log_request() #249
• ::log_response() - See Add tests for Debug::log_response() #250

---

Plugins_Screens ✅

• Assigned to @costdev

Click to view the methods

• ::__construct() - See Add tests for Plugins_Screens::__construct() #220
• ::get_instance() - See Add tests for Plugins_Screens::get_instance() #221
• ::remove_unused_filter_tabs() - See Add tests for Plugins_Screens::remove_unused_filter_tabs() #222

---

Themes_Screens ✅

• Assigned to @costdev

Click to view the methods

• ::__construct() - See Add tests for Themes_Screens::__construct() #223
• ::get_instance() - See Add tests for Themes_Screens::get_instance() #224
• ::admin_enqueue_scripts() - See Add tests for Themes_Screens::admin_enqueue_scripts() #225
• ::redirect_to_theme_install() - See Add tests for Themes_Screens::redirect_to_theme_install() #226

---

Utilities ✅

• Assigned to @costdev
• Depends on [Bug]: Utilities::get_top_level_domain() only works for single-part TLDs. #194

Click to view the methods

• ::get_site_domain() - See Add tests for Utilities::get_site_domain(). #228
  • This method is currently named ::get_top_level_domain().
  • It will be replaced by [Bug]: Utilities::get_top_level_domain() only works for single-part TLDs. #194.
• ::include_file() - See Add tests for Utilities::include_file() #227

[costdev]

Line and branch coverage of the codebase at the time of writing is now at 100%.

Here's a merged single-site and multisite coverage report showing line coverage. Merging coverage reports doesn't support branch coverage unfortunately.

Closing out this issue.