Add unit coverage for graph_buttons and page_head glyph changes

The theme-glyph change touched quicktree_graph_buttons() (setup.php:219) and quicktree_page_head() (setup.php:265-268), which the patch-coverage gate flagged as uncovered. Add QuicktreeGraphButtonsTest and QuicktreePageHeadTest, plus get_md5_include_js/get_md5_include_css/get_selected_theme stubs in the unit bootstrap, to exercise both hooks.

Author: Cacti Group

CHANGELOG.md

@@ -9,6 +9,7 @@
 - issue: Fix quicktree_show_tab() building an invalid `?location=tabtab`/`?location=tabconsole` URL, an undefined-variable text-domain typo in the QuickTree page's "no graph specified" message, and a misspelled `qucktree` text domain
 - security: Migrate quicktree SQL helpers to prepared statements, normalize location handling, and ensure redirects terminate execution
 - test: Migrate test suite to Pest with a Cacti-provided test harness and CI workflow
+- feature: Give the QuickTree add/remove graph glyphs their own theme-appropriate colors across every packaged Cacti theme, instead of reusing the core deviceUp/deviceDown classes
 
 
 --- 2.0 ---

tests/Unit/QuicktreeGraphButtonsTest.php

@@ -0,0 +1,40 @@
+<?php
+/*
+ +-------------------------------------------------------------------------+
+ | Copyright (C) 2004-2026 The Cacti Group                                 |
+ +-------------------------------------------------------------------------+
+*/
+
+/*
+ * Coverage for quicktree_graph_buttons(): the 'graph_buttons' hook prints an
+ * "Add this graph to QuickTree" glyph for the current graph, but only when
+ * the user is authorized for the quicktree realm.
+ */
+
+beforeAll(function () {
+	require_once __DIR__ . '/../../setup.php';
+});
+
+beforeEach(function () {
+	$GLOBALS['__test_realm_auth'] = true;
+});
+
+it('prints an Add-to-QuickTree glyph for the current graph when authorized', function () {
+	ob_start();
+	quicktree_graph_buttons(array(1 => array('local_graph_id' => 42, 'rra' => 7)));
+	$output = ob_get_clean();
+
+	expect($output)->toContain('quicktreeAdd');
+	expect($output)->toContain('fa-plus-circle');
+	expect($output)->toContain('addQuickTree(42, 7)');
+});
+
+it('prints nothing when the user is not authorized for the quicktree realm', function () {
+	$GLOBALS['__test_realm_auth'] = false;
+
+	ob_start();
+	quicktree_graph_buttons(array(1 => array('local_graph_id' => 42, 'rra' => 7)));
+	$output = ob_get_clean();
+
+	expect($output)->toBe('');
+});

tests/Unit/QuicktreePageHeadTest.php

@@ -0,0 +1,77 @@
+<?php
+/*
+ +-------------------------------------------------------------------------+
+ | Copyright (C) 2004-2026 The Cacti Group                                 |
+ +-------------------------------------------------------------------------+
+*/
+
+/*
+ * Coverage for quicktree_page_head(): the 'page_head' hook includes the
+ * plugin JavaScript and base stylesheet on every page, and additionally
+ * includes a theme-specific stylesheet when one exists for the selected
+ * theme.
+ */
+
+beforeAll(function () {
+	require_once __DIR__ . '/../../setup.php';
+});
+
+beforeEach(function () {
+	$GLOBALS['__qt_orig_base']        = $GLOBALS['config']['base_path'];
+	$GLOBALS['__test_selected_theme'] = 'modern';
+});
+
+afterEach(function () {
+	$GLOBALS['config']['base_path'] = $GLOBALS['__qt_orig_base'];
+
+	if (!empty($GLOBALS['__qt_tmp_base']) && is_dir($GLOBALS['__qt_tmp_base'])) {
+		@unlink($GLOBALS['__qt_tmp_base'] . '/plugins/quicktree/css/modern.css');
+		@rmdir($GLOBALS['__qt_tmp_base'] . '/plugins/quicktree/css');
+		@rmdir($GLOBALS['__qt_tmp_base'] . '/plugins/quicktree');
+		@rmdir($GLOBALS['__qt_tmp_base'] . '/plugins');
+		@rmdir($GLOBALS['__qt_tmp_base']);
+	}
+
+	$GLOBALS['__qt_tmp_base'] = null;
+});
+
+it('includes the plugin JavaScript and base stylesheet', function () {
+	ob_start();
+	quicktree_page_head();
+	$output = ob_get_clean();
+
+	expect($output)->toContain('plugins/quicktree/js/quicktree.js');
+	expect($output)->toContain('plugins/quicktree/css/quicktree.css');
+});
+
+it('includes the theme stylesheet when a matching theme file exists', function () {
+	$base = sys_get_temp_dir() . '/qt_theme_' . uniqid();
+	mkdir($base . '/plugins/quicktree/css', 0777, true);
+	file_put_contents($base . '/plugins/quicktree/css/modern.css', '/* test */');
+
+	$GLOBALS['__qt_tmp_base']          = $base;
+	$GLOBALS['config']['base_path']    = $base;
+	$GLOBALS['__test_selected_theme']  = 'modern';
+
+	ob_start();
+	quicktree_page_head();
+	$output = ob_get_clean();
+
+	expect($output)->toContain('plugins/quicktree/css/modern.css');
+});
+
+it('omits the theme stylesheet when no matching theme file exists', function () {
+	$base = sys_get_temp_dir() . '/qt_theme_' . uniqid();
+	mkdir($base . '/plugins/quicktree/css', 0777, true);
+
+	$GLOBALS['__qt_tmp_base']          = $base;
+	$GLOBALS['config']['base_path']    = $base;
+	$GLOBALS['__test_selected_theme']  = 'no-such-theme';
+
+	ob_start();
+	quicktree_page_head();
+	$output = ob_get_clean();
+
+	expect($output)->not->toContain('no-such-theme');
+	expect($output)->toContain('plugins/quicktree/css/quicktree.css');
+});

tests/bootstrap-unit.php

@@ -388,3 +388,29 @@ function form_end() {
 		$GLOBALS['__form_wrapper_events'][] = 'form_end';
 	}
 }
+
+/*
+ * quicktree_page_head() emits the plugin's JS/CSS includes and selects a
+ * theme-specific stylesheet. These stubs return the asset paths verbatim so
+ * specs can assert which includes were printed, and let the selected theme be
+ * controlled per test.
+ */
+if (!function_exists('get_md5_include_js')) {
+	function get_md5_include_js($path) {
+		return "<script src='$path'></script>";
+	}
+}
+
+if (!function_exists('get_md5_include_css')) {
+	function get_md5_include_css($path) {
+		return "<link rel='stylesheet' href='$path'>";
+	}
+}
+
+$GLOBALS['__test_selected_theme'] = 'modern';
+
+if (!function_exists('get_selected_theme')) {
+	function get_selected_theme() {
+		return $GLOBALS['__test_selected_theme'];
+	}
+}