From 16186a344a7b61633cb7342aac37ac56ad83d261 Mon Sep 17 00:00:00 2001
From: Adam Mathes <adam@adammathes.com>
Date: Thu, 12 Feb 2026 19:55:05 -0800
Subject: =?UTF-8?q?Add=20comprehensive=20test=20suite=20=E2=80=94=2081%=20?=
 =?UTF-8?q?cross-package=20coverage?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Bug fixes:
- config: remove unused log import
- item: fix Printf format %d->%t for boolean ReadState
- util: update stale config.Read -> config.Init, remove config.Config.DBServer

Test files added:
- config/config_test.go: Init, readConfig, addDefaults (100%)
- vlog/vlog_test.go: Printf, Println verbose/silent (100%)
- models/db_test.go: InitDB tests
- models/feed/feed_test.go: CRUD, filter, Categories, NewFeed, ResolveFeedURL (87%)
- models/item/item_test.go: CRUD, Filter with category/search/starred, rewriteImages (71%)
- exporter/exporter_test.go: all export formats (91%)
- importer/importer_test.go: InsertIItem, ImportJSON (90%)
- crawler/crawler_test.go: GetFeedContent, CrawlFeed, CrawlWorker, Crawl (89%)
- web/web_test.go: auth, login/logout, stream, item, feed, category,
  export, crawl, imageProxy handlers (77%)

Remaining 0% functions require HTTP/rice.MustFindBox/main entry and
can't be unit tested without refactoring (see tickets NK-gqkh96, NK-6q9nyg).
---
 vlog/vlog_test.go | 78 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 78 insertions(+)
 create mode 100644 vlog/vlog_test.go

(limited to 'vlog')

diff --git a/vlog/vlog_test.go b/vlog/vlog_test.go
new file mode 100644
index 0000000..9def0f0
--- /dev/null
+++ b/vlog/vlog_test.go
@@ -0,0 +1,78 @@
+package vlog
+
+import (
+	"bytes"
+	"fmt"
+	"os"
+	"testing"
+)
+
+func captureStdout(f func()) string {
+	old := os.Stdout
+	r, w, _ := os.Pipe()
+	os.Stdout = w
+
+	f()
+
+	w.Close()
+	os.Stdout = old
+
+	var buf bytes.Buffer
+	buf.ReadFrom(r)
+	return buf.String()
+}
+
+func TestPrintfVerbose(t *testing.T) {
+	VERBOSE = true
+	defer func() { VERBOSE = false }()
+
+	output := captureStdout(func() {
+		Printf("hello %s", "world")
+	})
+	expected := fmt.Sprintf("hello %s", "world")
+	if output != expected {
+		t.Errorf("expected %q, got %q", expected, output)
+	}
+}
+
+func TestPrintfSilent(t *testing.T) {
+	VERBOSE = false
+
+	output := captureStdout(func() {
+		Printf("hello %s", "world")
+	})
+	if output != "" {
+		t.Errorf("expected empty output when not verbose, got %q", output)
+	}
+}
+
+func TestPrintlnVerbose(t *testing.T) {
+	VERBOSE = true
+	defer func() { VERBOSE = false }()
+
+	output := captureStdout(func() {
+		Println("hello", "world")
+	})
+	expected := fmt.Sprintln("hello", "world")
+	if output != expected {
+		t.Errorf("expected %q, got %q", expected, output)
+	}
+}
+
+func TestPrintlnSilent(t *testing.T) {
+	VERBOSE = false
+
+	output := captureStdout(func() {
+		Println("hello", "world")
+	})
+	if output != "" {
+		t.Errorf("expected empty output when not verbose, got %q", output)
+	}
+}
+
+func TestInit(t *testing.T) {
+	// init() sets VERBOSE to false
+	if VERBOSE != false {
+		t.Error("VERBOSE should default to false")
+	}
+}
-- 
cgit v1.2.3