From 7769522c632ccaf0ba37b64d0c48d58e958b0c8f Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Sat, 8 Nov 2025 16:58:30 -0600 Subject: test: smoke: reorganize test suite following quality-engineer.org naming conventions - Create test-wttrin-smoke.el with 15 smoke tests for package sanity checks - Delete test-wttrin-integration.el (misnamed, contained smoke tests not integration tests) - Remove redundant functional tests already covered in dedicated unit test files - URL building: covered by test-wttrin--build-url.el (12 tests) - Cache operations: covered by test-wttrin--get-cached-or-fetch.el (11 tests) - Mode-line display: covered by test-wttrin--mode-line-map.el Smoke tests verify: - Package loads without errors - Dependencies available (xterm-color, url) - Public API exists and is interactive - Configuration variables defined - Keymaps exist and are valid - Package metadata correct (version, autoload cookies) Final test organization: 18 files, 162 tests (15 smoke + 142 unit + 5 integration) --- tests/test-wttrin-integration.el | 148 --------------------------------------- 1 file changed, 148 deletions(-) delete mode 100644 tests/test-wttrin-integration.el (limited to 'tests/test-wttrin-integration.el') diff --git a/tests/test-wttrin-integration.el b/tests/test-wttrin-integration.el deleted file mode 100644 index a79d7b3..0000000 --- a/tests/test-wttrin-integration.el +++ /dev/null @@ -1,148 +0,0 @@ -;;; test-wttrin-integration.el --- Integration tests for wttrin package loading -*- lexical-binding: t; -*- - -;; Copyright (C) 2024 Craig Jennings - -;;; Commentary: -;; Integration tests to verify wttrin package loads correctly and -;; key functionality is available. - -;;; Code: - -(require 'ert) - -;;; Package Loading Tests - -(ert-deftest test-wttrin-integration-package-loads () - "Test that wttrin package loads without errors." - ;; This test runs AFTER wttrin.el is loaded by the test runner - ;; If we got here, the package loaded successfully - (should t)) - -(ert-deftest test-wttrin-integration-provide-statement () - "Test that wttrin feature is provided." - (should (featurep 'wttrin))) - -;;; Dependency Tests - -(ert-deftest test-wttrin-integration-xterm-color-available () - "Test that xterm-color dependency is available. -This is a REQUIRED dependency - wttrin cannot function without it." - (should (featurep 'xterm-color))) - -(ert-deftest test-wttrin-integration-url-available () - "Test that url library is available (built-in)." - (should (require 'url nil t))) - -;;; Function Availability Tests - -(ert-deftest test-wttrin-integration-main-command-defined () - "Test that main wttrin command is defined." - (should (fboundp 'wttrin))) - -(ert-deftest test-wttrin-integration-main-command-is-interactive () - "Test that main wttrin command is interactive." - (should (commandp 'wttrin))) - -(ert-deftest test-wttrin-integration-wttrin-mode-defined () - "Test that wttrin-mode is defined." - (should (fboundp 'wttrin-mode))) - -(ert-deftest test-wttrin-integration-mode-line-mode-defined () - "Test that wttrin-mode-line-mode is defined." - (should (fboundp 'wttrin-mode-line-mode))) - -(ert-deftest test-wttrin-integration-clear-cache-defined () - "Test that cache clear command is defined." - (should (fboundp 'wttrin-clear-cache))) - -;;; Configuration Variable Tests - -(ert-deftest test-wttrin-integration-defcustoms-exist () - "Test that key defcustom variables are defined." - (should (boundp 'wttrin-default-locations)) - (should (boundp 'wttrin-unit-system)) - (should (boundp 'wttrin-cache-ttl)) - (should (boundp 'wttrin-cache-max-entries)) - (should (boundp 'wttrin-mode-line-favorite-location)) - (should (boundp 'wttrin-mode-line-refresh-interval)) - (should (boundp 'wttrin-mode-line-startup-delay))) - -(ert-deftest test-wttrin-integration-internal-vars-exist () - "Test that internal variables are defined." - (should (boundp 'wttrin--cache)) - (should (boundp 'wttrin--mode-line-map)) - (should (boundp 'wttrin-mode-line-string))) - -;;; Keymap Tests - -(ert-deftest test-wttrin-integration-wttrin-mode-map-exists () - "Test that wttrin-mode keymap exists." - (should (boundp 'wttrin-mode-map)) - (should (keymapp wttrin-mode-map))) - -(ert-deftest test-wttrin-integration-mode-line-map-exists () - "Test that mode-line keymap exists." - (should (boundp 'wttrin--mode-line-map)) - (should (keymapp wttrin--mode-line-map))) - -;;; Package Metadata Tests - -(ert-deftest test-wttrin-integration-package-version () - "Test that package has version information." - ;; Check if package-get-version would work (not calling it as it's not always available) - ;; Just verify the Version header exists in the file - (let ((version-line (with-temp-buffer - (insert-file-contents "wttrin.el" nil 0 500) - (goto-char (point-min)) - (re-search-forward "^;; Version: \\([0-9.]+\\)" nil t) - (match-string 1)))) - (should version-line) - (should (string-match-p "^[0-9]+\\.[0-9]+\\.[0-9]+$" version-line)))) - -;;; Autoload Tests - -(ert-deftest test-wttrin-integration-autoload-cookies () - "Test that key functions have autoload cookies." - ;; Verify autoload cookies exist in source (they become effective after package install) - (let ((source (with-temp-buffer - (insert-file-contents "wttrin.el") - (buffer-string)))) - ;; Main command should have autoload - (should (string-match-p ";;;###autoload\n(defun wttrin" source)) - ;; Mode-line mode should have autoload - (should (string-match-p ";;;###autoload\n(define-minor-mode wttrin-mode-line-mode" source)))) - -;;; Smoke Test - Basic Functionality - -(ert-deftest test-wttrin-integration-url-building () - "Test that URL building works correctly." - (let ((wttrin-unit-system "m")) - (should (string= "https://wttr.in/Paris?mA" - (wttrin--build-url "Paris"))))) - -(ert-deftest test-wttrin-integration-cache-operations () - "Test that cache operations don't error." - ;; Clear cache - (wttrin-clear-cache) - (should (= 0 (hash-table-count wttrin--cache))) - - ;; Add an entry - (puthash "test-location" (cons (float-time) "test-data") wttrin--cache) - (should (= 1 (hash-table-count wttrin--cache))) - - ;; Clear again - (wttrin-clear-cache) - (should (= 0 (hash-table-count wttrin--cache)))) - -(ert-deftest test-wttrin-integration-mode-line-display () - "Test that mode-line display function doesn't error." - (let ((wttrin-mode-line-favorite-location "Test, CA") - (wttrin--mode-line-tooltip-data "Test weather")) - ;; Should not error (function returns nil but sets wttrin-mode-line-string) - (wttrin--mode-line-update-display "☀️") - ;; Should set the mode-line string - (should wttrin-mode-line-string) - (should (stringp wttrin-mode-line-string)))) - -(provide 'test-wttrin-integration) -;;; test-wttrin-integration.el ends here -- cgit v1.2.3