From f67916883926a539b65ae035a93f4577939c7b23 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Mon, 22 Sep 2025 16:50:33 -0400
Subject: [PATCH 01/14] Fix --directory flag initialization bug
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Move client initialization from init() to PersistentPreRun to ensure
flags are parsed before creating the client. This fixes the --directory
flag which was being ignored because the client was created with an
empty directory string before flag parsing occurred.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 cmd/root.go  | 18 ++++++++++--------
 cmd/start.go |  3 +--
 2 files changed, 11 insertions(+), 10 deletions(-)

diff --git a/cmd/root.go b/cmd/root.go
index 0be0557..782e578 100644
--- a/cmd/root.go
+++ b/cmd/root.go
@@ -44,16 +44,18 @@ func init() {
 
 	viper.AutomaticEnv()
 
-	var err error
+	RootCmd.PersistentPreRun = func(cmd *cobra.Command, args []string) {
+		var err error
 
-	client, err = openpomodoro.NewClient(directoryFlag)
-	if err != nil {
-		log.Fatalf("Could not create client: %v", err)
-	}
+		client, err = openpomodoro.NewClient(directoryFlag)
+		if err != nil {
+			log.Fatalf("Could not create client: %v", err)
+		}
 
-	settings, err = client.Settings()
-	if err != nil {
-		log.Fatalf("Could not retrieve settings: %v", err)
+		settings, err = client.Settings()
+		if err != nil {
+			log.Fatalf("Could not retrieve settings: %v", err)
+		}
 	}
 }
 
diff --git a/cmd/start.go b/cmd/start.go
index b2a34e5..3ff97a2 100644
--- a/cmd/start.go
+++ b/cmd/start.go
@@ -27,8 +27,7 @@ func init() {
 		"time ago this Pomodoro started")
 
 	command.Flags().IntVarP(
-		&durationFlag, "duration", "d",
-		int(settings.DefaultPomodoroDuration.Minutes()),
+		&durationFlag, "duration", "d", 25,
 		"duration for this Pomodoro")
 
 	command.Flags().StringArrayVarP(

From 1409054d20c56a9c108fe38d12bb7ec41fa441c6 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Mon, 22 Sep 2025 16:50:45 -0400
Subject: [PATCH 02/14] Add comprehensive BATS black box testing framework
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add BATS test infrastructure with test_helper.bash utilities
- Add Makefile with `make test` target for colored test output
- Add 34 focused black box tests covering all core commands:
  - start.bats (7 tests) - command creation and file writing
  - finish.bats (6 tests) - completion and history management
  - clear.bats (4 tests) - current pomodoro removal
  - cancel.bats (5 tests) - cancellation without history
  - amend.bats (6 tests) - current pomodoro modification
  - repeat.bats (6 tests) - last pomodoro repetition

Tests verify CLI behavior through actual command execution and file
system assertions, providing end-to-end validation of the user
experience. All tests use temporary directories for isolation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 Makefile              |  3 +++
 test/amend.bats       | 47 +++++++++++++++++++++++++++++++++++++
 test/cancel.bats      | 43 ++++++++++++++++++++++++++++++++++
 test/clear.bats       | 35 ++++++++++++++++++++++++++++
 test/finish.bats      | 54 +++++++++++++++++++++++++++++++++++++++++++
 test/repeat.bats      | 49 +++++++++++++++++++++++++++++++++++++++
 test/start.bats       | 48 ++++++++++++++++++++++++++++++++++++++
 test/test_helper.bash | 44 +++++++++++++++++++++++++++++++++++
 8 files changed, 323 insertions(+)
 create mode 100644 Makefile
 create mode 100644 test/amend.bats
 create mode 100644 test/cancel.bats
 create mode 100644 test/clear.bats
 create mode 100644 test/finish.bats
 create mode 100644 test/repeat.bats
 create mode 100644 test/start.bats
 create mode 100644 test/test_helper.bash

diff --git a/Makefile b/Makefile
new file mode 100644
index 0000000..a3e827d
--- /dev/null
+++ b/Makefile
@@ -0,0 +1,3 @@
+.PHONY: test
+test:
+	bats test/
diff --git a/test/amend.bats b/test/amend.bats
new file mode 100644
index 0000000..8504790
--- /dev/null
+++ b/test/amend.bats
@@ -0,0 +1,47 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "amend changes description of current pomodoro" {
+    pomodoro start "Original task"
+    run pomodoro amend "Amended task"
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "Amended task"
+}
+
+@test "amend adds tags to current pomodoro" {
+    pomodoro start "Task"
+    run pomodoro amend -t "work,urgent"
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "tags=work,urgent"
+}
+
+@test "amend changes duration of current pomodoro" {
+    pomodoro start "Task"
+    run pomodoro amend -d 45
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "duration=45"
+}
+
+
+@test "amend creates new current when no current exists" {
+    pomodoro start "Task" --ago 5m
+    pomodoro finish
+    run pomodoro amend "New task"
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "New task"
+}
+
+@test "amend outputs current pomodoro status" {
+    pomodoro start "Task"
+    run pomodoro amend "Amended task"
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "Amended task" ]]
+}
+
+@test "amend with no arguments succeeds" {
+    pomodoro start "Task"
+    run pomodoro amend
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "Task"
+}
\ No newline at end of file
diff --git a/test/cancel.bats b/test/cancel.bats
new file mode 100644
index 0000000..ad3211c
--- /dev/null
+++ b/test/cancel.bats
@@ -0,0 +1,43 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "cancel empties current file" {
+    pomodoro start "Work session"
+    run pomodoro cancel
+    [ "$status" -eq 0 ]
+    assert_file_empty "current"
+}
+
+@test "cancel removes current pomodoro from history" {
+    pomodoro start "Task to cancel"
+    run pomodoro cancel
+    [ "$status" -eq 0 ]
+    assert_file_empty "current"
+    assert_file_empty "history"
+}
+
+@test "cancel preserves existing history but removes current" {
+    pomodoro start "First task" --ago 5m
+    pomodoro finish
+    pomodoro start "Task to cancel"
+    run pomodoro cancel
+    [ "$status" -eq 0 ]
+
+    assert_file_contains "history" "First task"
+    assert_file_empty "current"
+}
+
+@test "cancel with no current pomodoro succeeds" {
+    run pomodoro cancel
+    [ "$status" -eq 0 ]
+    assert_file_empty "current"
+}
+
+@test "cancel produces no output" {
+    pomodoro start "Test task"
+    run pomodoro cancel
+    [ "$status" -eq 0 ]
+    [ -z "$output" ]
+}
+
diff --git a/test/clear.bats b/test/clear.bats
new file mode 100644
index 0000000..6af3728
--- /dev/null
+++ b/test/clear.bats
@@ -0,0 +1,35 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "clear empties current file" {
+    pomodoro start "Work session"
+    run pomodoro clear
+    [ "$status" -eq 0 ]
+    assert_file_empty "current"
+}
+
+@test "clear does not affect history" {
+    pomodoro start "First task" --ago 5m
+    pomodoro finish
+    pomodoro start "Second task"
+    run pomodoro clear
+    [ "$status" -eq 0 ]
+
+    assert_file_contains "history" "First task"
+    assert_file_empty "current"
+}
+
+@test "clear with no current pomodoro succeeds" {
+    run pomodoro clear
+    [ "$status" -eq 0 ]
+    assert_file_empty "current"
+}
+
+@test "clear produces no output" {
+    pomodoro start "Test task"
+    run pomodoro clear
+    [ "$status" -eq 0 ]
+    [ -z "$output" ]
+}
+
diff --git a/test/finish.bats b/test/finish.bats
new file mode 100644
index 0000000..453622a
--- /dev/null
+++ b/test/finish.bats
@@ -0,0 +1,54 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "finish moves current pomodoro to history" {
+    pomodoro start "Work session"
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+
+    assert_file_empty "current"
+    assert_file_contains "history" "Work session"
+}
+
+@test "finish preserves pomodoro description in history" {
+    pomodoro start "Important task"
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+
+    assert_file_contains "history" 'description="Important task"'
+}
+
+
+@test "finish records actual elapsed time in history" {
+    pomodoro start "Work session" -d 30 --ago 10m
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+
+    assert_file_contains "history" "Work session"
+    assert_file_contains "history" "duration=10"
+}
+
+@test "finish appends to existing history" {
+    pomodoro start "First task" --ago 5m
+    pomodoro finish
+    pomodoro start "Second task" --ago 3m
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+
+    assert_file_contains "history" "First task"
+    assert_file_contains "history" "Second task"
+}
+
+@test "finish outputs elapsed time" {
+    pomodoro start "Test task" --ago 5m
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "5:" ]]
+}
+
+@test "finish with no current pomodoro succeeds" {
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+    assert_file_empty "current"
+}
\ No newline at end of file
diff --git a/test/repeat.bats b/test/repeat.bats
new file mode 100644
index 0000000..70f3663
--- /dev/null
+++ b/test/repeat.bats
@@ -0,0 +1,49 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "repeat copies description from last history entry" {
+    pomodoro start "Original task" --ago 5m
+    pomodoro finish
+    run pomodoro repeat
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "Original task"
+}
+
+@test "repeat copies tags from last history entry" {
+    pomodoro start "Task with tags" -t "work,urgent" --ago 5m
+    pomodoro finish
+    run pomodoro repeat
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "work,urgent"
+}
+
+@test "repeat uses default duration" {
+    pomodoro start "Task" -d 45 --ago 10m
+    pomodoro finish
+    run pomodoro repeat
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "duration=25"
+}
+
+@test "repeat creates new timestamp" {
+    pomodoro start "Task" --ago 5m
+    pomodoro finish
+    run pomodoro repeat
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "$(date '+%Y-%m-%d')"
+}
+
+
+@test "repeat outputs current pomodoro status" {
+    pomodoro start "Repeated task" --ago 5m
+    pomodoro finish
+    run pomodoro repeat
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "Repeated task" ]]
+}
+
+@test "repeat with no history fails" {
+    run pomodoro repeat
+    [ "$status" -ne 0 ]
+}
\ No newline at end of file
diff --git a/test/start.bats b/test/start.bats
new file mode 100644
index 0000000..0b961c5
--- /dev/null
+++ b/test/start.bats
@@ -0,0 +1,48 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "start creates current file" {
+    run pomodoro start
+    [ "$status" -eq 0 ]
+    assert_file_exists "current"
+}
+
+@test "start with description writes description to current file" {
+    run pomodoro start "Important work"
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" 'description="Important work"'
+}
+
+@test "start with tags writes tags to current file" {
+    run pomodoro start -t "work,urgent"
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" 'tags=work,urgent'
+}
+
+@test "start with custom duration writes duration to current file" {
+    run pomodoro start --duration 30
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" 'duration=30'
+}
+
+
+@test "start writes timestamp to current file" {
+    run pomodoro start
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "$(date '+%Y-%m-%d')"
+}
+
+@test "start replaces existing current pomodoro" {
+    pomodoro start "First task"
+    run pomodoro start "Second task"
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "Second task"
+    ! grep -q "First task" "$TEST_DIR/current"
+}
+
+@test "start outputs current pomodoro status" {
+    run pomodoro start "Test task"
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "Test task" ]]
+}
\ No newline at end of file
diff --git a/test/test_helper.bash b/test/test_helper.bash
new file mode 100644
index 0000000..5bb7e11
--- /dev/null
+++ b/test/test_helper.bash
@@ -0,0 +1,44 @@
+#!/usr/bin/env bash
+
+export POMODORO_BIN="${BATS_TEST_DIRNAME}/../main.go"
+
+setup() {
+    export TEST_DIR="$(mktemp -d)"
+}
+
+teardown() {
+    rm -rf "$TEST_DIR"
+}
+
+pomodoro() {
+    go run "$POMODORO_BIN" --directory "$TEST_DIR" "$@"
+}
+
+assert_file_exists() {
+    local file="$TEST_DIR/$1"
+    [ -f "$file" ] || {
+        echo "File $file does not exist"
+        return 1
+    }
+}
+
+assert_file_contains() {
+    local file="$TEST_DIR/$1"
+    local content="$2"
+    grep -q "$content" "$file" || {
+        echo "File $file does not contain: $content"
+        echo "File contents:"
+        cat "$file"
+        return 1
+    }
+}
+
+assert_file_empty() {
+    local file="$TEST_DIR/$1"
+    [ ! -s "$file" ] || {
+        echo "File $file is not empty"
+        echo "Contents:"
+        cat "$file"
+        return 1
+    }
+}
\ No newline at end of file

From 2d97c937c214d347e7e7f88b2ce4f2f3dfa025e7 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Mon, 22 Sep 2025 17:09:18 -0400
Subject: [PATCH 03/14] Add comprehensive hook testing with BATS
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add hook testing infrastructure to test_helper.bash:
  - create_hook() - creates executable hook scripts
  - assert_hook_executed() - verifies hook execution
  - assert_hook_contains() - checks hook log contents

- Add hooks.bats with 9 tests covering all hook scenarios:
  - start/stop hook execution for all commands
  - non-executable hook error handling
  - missing hook graceful handling
  - hook failure behavior
  - multiple hook sequence execution

Total test coverage now: 43 comprehensive black box tests

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 test/hooks.bats       | 92 +++++++++++++++++++++++++++++++++++++++++++
 test/test_helper.bash | 29 ++++++++++++++
 2 files changed, 121 insertions(+)
 create mode 100644 test/hooks.bats

diff --git a/test/hooks.bats b/test/hooks.bats
new file mode 100644
index 0000000..d439da1
--- /dev/null
+++ b/test/hooks.bats
@@ -0,0 +1,92 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "start hook executes when starting pomodoro" {
+    create_hook "start" 'echo "START_HOOK_EXECUTED" >> "$TEST_DIR/hook_log"'
+
+    run pomodoro start "Test task"
+    [ "$status" -eq 0 ]
+
+    assert_hook_contains "START_HOOK_EXECUTED"
+}
+
+@test "stop hook executes when finishing pomodoro" {
+    create_hook "stop" 'echo "STOP_HOOK_EXECUTED" >> "$TEST_DIR/hook_log"'
+
+    pomodoro start "Test task" --ago 5m
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+
+    assert_hook_contains "STOP_HOOK_EXECUTED"
+}
+
+@test "stop hook executes when cancelling pomodoro" {
+    create_hook "stop" 'echo "CANCEL_HOOK_EXECUTED" >> "$TEST_DIR/hook_log"'
+
+    pomodoro start "Test task"
+    run pomodoro cancel
+    [ "$status" -eq 0 ]
+
+    assert_hook_contains "CANCEL_HOOK_EXECUTED"
+}
+
+@test "stop hook executes when clearing pomodoro" {
+    create_hook "stop" 'echo "CLEAR_HOOK_EXECUTED" >> "$TEST_DIR/hook_log"'
+
+    pomodoro start "Test task"
+    run pomodoro clear
+    [ "$status" -eq 0 ]
+
+    assert_hook_contains "CLEAR_HOOK_EXECUTED"
+}
+
+@test "start hook executes when repeating pomodoro" {
+    pomodoro start "Original task" --ago 5m
+    pomodoro finish
+
+    create_hook "start" 'echo "REPEAT_HOOK_EXECUTED" >> "$TEST_DIR/hook_log"'
+
+    run pomodoro repeat
+    [ "$status" -eq 0 ]
+
+    assert_hook_contains "REPEAT_HOOK_EXECUTED"
+}
+
+@test "non-executable hook causes command to fail" {
+    mkdir -p "$TEST_DIR/hooks"
+    echo 'echo "SHOULD_NOT_RUN" >> "$TEST_DIR/hook_log"' > "$TEST_DIR/hooks/start"
+
+    run pomodoro start "Test task"
+    [ "$status" -ne 0 ]
+
+    [ ! -f "$TEST_DIR/hook_log" ]
+}
+
+@test "missing hook does not cause error" {
+    run pomodoro start "Test task"
+    [ "$status" -eq 0 ]
+
+    [ ! -f "$TEST_DIR/hook_log" ]
+}
+
+@test "hook failure does not prevent command from succeeding" {
+    create_hook "start" 'echo "HOOK_RAN" >> "$TEST_DIR/hook_log"; exit 1'
+
+    run pomodoro start "Test task"
+    [ "$status" -ne 0 ]
+
+    assert_hook_contains "HOOK_RAN"
+}
+
+@test "multiple hooks execute in sequence" {
+    create_hook "start" 'echo "START_EXECUTED" >> "$TEST_DIR/hook_log"'
+    create_hook "stop" 'echo "STOP_EXECUTED" >> "$TEST_DIR/hook_log"'
+
+    pomodoro start "Test task" --ago 5m
+    run pomodoro finish
+    [ "$status" -eq 0 ]
+
+    assert_hook_contains "START_EXECUTED"
+    assert_hook_contains "STOP_EXECUTED"
+}
\ No newline at end of file
diff --git a/test/test_helper.bash b/test/test_helper.bash
index 5bb7e11..fa35ce5 100644
--- a/test/test_helper.bash
+++ b/test/test_helper.bash
@@ -41,4 +41,33 @@ assert_file_empty() {
         cat "$file"
         return 1
     }
+}
+
+create_hook() {
+    local hook_name="$1"
+    local hook_content="$2"
+
+    mkdir -p "$TEST_DIR/hooks"
+    cat > "$TEST_DIR/hooks/$hook_name" << EOF
+#!/bin/bash
+$hook_content
+EOF
+    chmod +x "$TEST_DIR/hooks/$hook_name"
+}
+
+assert_hook_executed() {
+    [ -f "$TEST_DIR/hook_log" ] || {
+        echo "Hook log file not found"
+        return 1
+    }
+}
+
+assert_hook_contains() {
+    assert_hook_executed
+    grep -q "$1" "$TEST_DIR/hook_log" || {
+        echo "Hook log does not contain: $1"
+        echo "Hook log contents:"
+        cat "$TEST_DIR/hook_log"
+        return 1
+    }
 }
\ No newline at end of file

From b8a9cad57feceb49a7c6c7e209e11aa3b2facbfd Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Mon, 22 Sep 2025 19:58:50 -0400
Subject: [PATCH 04/14] Add comprehensive tests for remaining commands
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add status.bats (5 tests) - current pomodoro display and state
- Add break.bats (3 tests) - break timer hook execution and validation
- Add history.bats (5 tests) - history display and --limit flag functionality

Complete black box test coverage: 56 tests across all CLI commands
- Core workflow: start, finish, clear, cancel, amend, repeat (34 tests)
- Hook system: comprehensive hook execution testing (9 tests)
- Display commands: status and history output validation (10 tests)
- Break command: timer and hook integration (3 tests)

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 test/break.bats   | 26 +++++++++++++++++++++
 test/history.bats | 59 +++++++++++++++++++++++++++++++++++++++++++++++
 test/status.bats  | 37 +++++++++++++++++++++++++++++
 3 files changed, 122 insertions(+)
 create mode 100644 test/break.bats
 create mode 100644 test/history.bats
 create mode 100644 test/status.bats

diff --git a/test/break.bats b/test/break.bats
new file mode 100644
index 0000000..386e72e
--- /dev/null
+++ b/test/break.bats
@@ -0,0 +1,26 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "break executes break hook before starting timer" {
+    create_hook "break" 'echo "BREAK_HOOK" >> "$TEST_DIR/hook_log"; exit 1'
+
+    run pomodoro break
+    [ "$status" -ne 0 ]
+
+    assert_hook_contains "BREAK_HOOK"
+}
+
+@test "break with custom duration parses correctly" {
+    create_hook "break" 'echo "BREAK_STARTED" >> "$TEST_DIR/hook_log"; exit 1'
+
+    run pomodoro break "10"
+    [ "$status" -ne 0 ]
+
+    assert_hook_contains "BREAK_STARTED"
+}
+
+@test "break with invalid duration fails" {
+    run pomodoro break "invalid"
+    [ "$status" -ne 0 ]
+}
\ No newline at end of file
diff --git a/test/history.bats b/test/history.bats
new file mode 100644
index 0000000..4da6b96
--- /dev/null
+++ b/test/history.bats
@@ -0,0 +1,59 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "history shows nothing when no history exists" {
+    run pomodoro history
+    [ "$status" -eq 0 ]
+    [ -z "$output" ]
+}
+
+@test "history shows completed pomodoros" {
+    pomodoro start "First task" --ago 10m
+    pomodoro finish
+    pomodoro start "Second task" --ago 5m
+    pomodoro finish
+
+    run pomodoro history
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "First task" ]]
+    [[ "$output" =~ "Second task" ]]
+}
+
+@test "history limit flag restricts output" {
+    pomodoro start "Task 1" --ago 15m
+    pomodoro finish
+    pomodoro start "Task 2" --ago 10m
+    pomodoro finish
+    pomodoro start "Task 3" --ago 5m
+    pomodoro finish
+
+    run pomodoro history --limit 2
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "Task 2" ]]
+    [[ "$output" =~ "Task 3" ]]
+    [[ ! "$output" =~ "Task 1" ]]
+}
+
+@test "history shows timestamps and durations" {
+    pomodoro start "Test task" --ago 10m
+    pomodoro finish
+
+    run pomodoro history
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "Test task" ]]
+    [[ "$output" =~ "$(date '+%Y-%m-%d')" ]]
+    [[ "$output" =~ "duration=10" ]]
+}
+
+@test "history with zero limit shows all entries" {
+    pomodoro start "Task 1" --ago 10m
+    pomodoro finish
+    pomodoro start "Task 2" --ago 5m
+    pomodoro finish
+
+    run pomodoro history --limit 0
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "Task 1" ]]
+    [[ "$output" =~ "Task 2" ]]
+}
\ No newline at end of file
diff --git a/test/status.bats b/test/status.bats
new file mode 100644
index 0000000..19e75d7
--- /dev/null
+++ b/test/status.bats
@@ -0,0 +1,37 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "status shows nothing when no current pomodoro" {
+    run pomodoro status
+    [ "$status" -eq 0 ]
+    [ -z "$output" ]
+}
+
+@test "status shows current pomodoro description" {
+    pomodoro start "Current task"
+    run pomodoro status
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "Current task" ]]
+}
+
+@test "status shows current pomodoro tags" {
+    pomodoro start "Task" -t "work,urgent"
+    run pomodoro status
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "work, urgent" ]]
+}
+
+@test "status shows remaining time for active pomodoro" {
+    pomodoro start "Task" --ago 5m
+    run pomodoro status
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "19:" ]] || [[ "$output" =~ "20:" ]]
+}
+
+@test "status shows exclamation for overdue pomodoro" {
+    pomodoro start "Task" --ago 30m
+    run pomodoro status
+    [ "$status" -eq 0 ]
+    [[ "$output" =~ "❗️" ]]
+}
\ No newline at end of file

From d9d0644c4d7dd9aa58a7093fc47573df4f468780 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Mon, 22 Sep 2025 21:01:35 -0400
Subject: [PATCH 05/14] Add GitHub Actions CI workflow for automated testing
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Run BATS test suite on push and pull requests
- Test on both Ubuntu and macOS platforms
- Install BATS via package manager (apt/brew)
- Execute full test suite with `make test`

Ensures all 56 black box tests pass before merge and catches
regressions across different environments.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .github/workflows/test.yml | 35 +++++++++++++++++++++++++++++++++++
 1 file changed, 35 insertions(+)
 create mode 100644 .github/workflows/test.yml

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
new file mode 100644
index 0000000..aa17d23
--- /dev/null
+++ b/.github/workflows/test.yml
@@ -0,0 +1,35 @@
+name: Test
+
+on:
+  push:
+    branches: [ main ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, macos-latest]
+        go-version: [1.21]
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Set up Go
+      uses: actions/setup-go@v5
+      with:
+        go-version: ${{ matrix.go-version }}
+
+    - name: Install BATS
+      run: |
+        if [ "$RUNNER_OS" == "Linux" ]; then
+          sudo apt-get update
+          sudo apt-get install -y bats
+        elif [ "$RUNNER_OS" == "macOS" ]; then
+          brew install bats-core
+        fi
+
+    - name: Run tests
+      run: make test
\ No newline at end of file

From 747b3fd4bf2dec47b089edfbe7f566a9ea32d96a Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 21:40:20 -0400
Subject: [PATCH 06/14] Use user's default pomodoro duration from settings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

When no --duration flag is provided, read DefaultPomodoroDuration from
settings file instead of hardcoding 25 minutes. This restores the
intended behavior that was broken by the --directory flag fix.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 cmd/start.go | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/cmd/start.go b/cmd/start.go
index 3ff97a2..18f8d7c 100644
--- a/cmd/start.go
+++ b/cmd/start.go
@@ -27,7 +27,7 @@ func init() {
 		"time ago this Pomodoro started")
 
 	command.Flags().IntVarP(
-		&durationFlag, "duration", "d", 25,
+		&durationFlag, "duration", "d", 0,
 		"duration for this Pomodoro")
 
 	command.Flags().StringArrayVarP(
@@ -42,7 +42,11 @@ func startCmd(cmd *cobra.Command, args []string) error {
 
 	p := openpomodoro.NewPomodoro()
 	p.Description = description
-	p.Duration = time.Duration(durationFlag) * time.Minute
+	if durationFlag == 0 {
+		p.Duration = settings.DefaultPomodoroDuration
+	} else {
+		p.Duration = time.Duration(durationFlag) * time.Minute
+	}
 	p.StartTime = time.Now().Add(-agoFlag)
 	p.Tags = tagsFlag
 

From 9af74b646b7e1adf430d9455d3309093b35d57f6 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 21:41:26 -0400
Subject: [PATCH 07/14] Add just-in-time BATS download to Makefile
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Download bats-core automatically if not installed, eliminating the need
for users to install BATS separately. Includes clean target to remove
downloaded BATS installation.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .gitignore |  1 +
 Makefile   | 18 ++++++++++++++++--
 2 files changed, 17 insertions(+), 2 deletions(-)

diff --git a/.gitignore b/.gitignore
index 849ddff..1cf9798 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1 +1,2 @@
 dist/
+.bats/
diff --git a/Makefile b/Makefile
index a3e827d..90792a9 100644
--- a/Makefile
+++ b/Makefile
@@ -1,3 +1,17 @@
+BATS_VERSION := v1.11.0
+BATS_DIR := .bats
+BATS := $(BATS_DIR)/bin/bats
+
 .PHONY: test
-test:
-	bats test/
+test: $(BATS)
+	$(BATS) test/
+
+$(BATS):
+	@mkdir -p $(BATS_DIR)
+	@echo "Downloading bats-core $(BATS_VERSION)..."
+	@curl -sSL https://github.com/bats-core/bats-core/archive/$(BATS_VERSION).tar.gz | tar xz -C $(BATS_DIR) --strip-components=1
+	@chmod +x $(BATS_DIR)/bin/bats
+
+.PHONY: clean
+clean:
+	rm -rf $(BATS_DIR)

From 7a5b3324f2e00ef1ea1e266b9829806ae3535806 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 21:44:31 -0400
Subject: [PATCH 08/14] Build binary for tests with dependency tracking
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Add binary build target to Makefile that tracks Go source files
- Use compiled binary instead of 'go run' in tests for better performance
- Only rebuild when Go sources or module files change
- Add pomodoro binary to .gitignore
- Update clean target to remove binary

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .gitignore            |  1 +
 Makefile              | 10 ++++++++--
 test/test_helper.bash |  4 ++--
 3 files changed, 11 insertions(+), 4 deletions(-)

diff --git a/.gitignore b/.gitignore
index 1cf9798..047d873 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,2 +1,3 @@
 dist/
 .bats/
+pomodoro
diff --git a/Makefile b/Makefile
index 90792a9..69e7382 100644
--- a/Makefile
+++ b/Makefile
@@ -1,11 +1,17 @@
 BATS_VERSION := v1.11.0
 BATS_DIR := .bats
 BATS := $(BATS_DIR)/bin/bats
+BINARY := pomodoro
+
+GO_SOURCES := $(shell find . -name '*.go' -not -path './vendor/*')
 
 .PHONY: test
-test: $(BATS)
+test: $(BINARY) $(BATS)
 	$(BATS) test/
 
+$(BINARY): $(GO_SOURCES) go.mod go.sum
+	go build -o $(BINARY) .
+
 $(BATS):
 	@mkdir -p $(BATS_DIR)
 	@echo "Downloading bats-core $(BATS_VERSION)..."
@@ -14,4 +20,4 @@ $(BATS):
 
 .PHONY: clean
 clean:
-	rm -rf $(BATS_DIR)
+	rm -rf $(BATS_DIR) $(BINARY)
diff --git a/test/test_helper.bash b/test/test_helper.bash
index fa35ce5..c86e6f1 100644
--- a/test/test_helper.bash
+++ b/test/test_helper.bash
@@ -1,6 +1,6 @@
 #!/usr/bin/env bash
 
-export POMODORO_BIN="${BATS_TEST_DIRNAME}/../main.go"
+export POMODORO_BIN="${BATS_TEST_DIRNAME}/../pomodoro"
 
 setup() {
     export TEST_DIR="$(mktemp -d)"
@@ -11,7 +11,7 @@ teardown() {
 }
 
 pomodoro() {
-    go run "$POMODORO_BIN" --directory "$TEST_DIR" "$@"
+    "$POMODORO_BIN" --directory "$TEST_DIR" "$@"
 }
 
 assert_file_exists() {

From 31657987932aeb8886d08d397900ee31a41203d7 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 21:49:56 -0400
Subject: [PATCH 09/14] Add comprehensive settings tests
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Test coverage for:
- Default pomodoro duration from settings
- Default break duration from settings
- Explicit duration flags overriding settings
- --directory flag using settings from specified directory
- Multiple settings values parsing
- Fallback to 25 minute default when no settings

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 test/settings.bats | 67 ++++++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 67 insertions(+)
 create mode 100644 test/settings.bats

diff --git a/test/settings.bats b/test/settings.bats
new file mode 100644
index 0000000..ebe545f
--- /dev/null
+++ b/test/settings.bats
@@ -0,0 +1,67 @@
+#!/usr/bin/env bats
+
+load test_helper
+
+@test "start uses default pomodoro duration from settings" {
+    echo "default_pomodoro_duration=45" > "$TEST_DIR/settings"
+    pomodoro start "Task with custom default"
+    assert_file_contains "current" "duration=45"
+}
+
+@test "start uses 25 minutes when no settings file exists" {
+    pomodoro start "Task with system default"
+    assert_file_contains "current" "duration=25"
+}
+
+@test "start explicit duration overrides settings default" {
+    echo "default_pomodoro_duration=45" > "$TEST_DIR/settings"
+    pomodoro start "Task" -d 30
+    assert_file_contains "current" "duration=30"
+}
+
+@test "repeat uses default pomodoro duration from settings" {
+    echo "default_pomodoro_duration=50" > "$TEST_DIR/settings"
+    pomodoro start "Original task" --ago 5m
+    pomodoro finish
+    run pomodoro repeat
+    [ "$status" -eq 0 ]
+    assert_file_contains "current" "duration=50"
+}
+
+@test "break uses default break duration from settings" {
+    echo "default_break_duration=10" > "$TEST_DIR/settings"
+    create_hook "break" 'echo "BREAK_HOOK_RAN" >> "$TEST_DIR/hook_log"; exit 1'
+
+    run pomodoro break
+    [ "$status" -ne 0 ]
+    assert_hook_contains "BREAK_HOOK_RAN"
+}
+
+@test "--directory flag uses settings from specified directory" {
+    ALT_DIR="$(mktemp -d)"
+    echo "default_pomodoro_duration=60" > "$ALT_DIR/settings"
+
+    run "$POMODORO_BIN" --directory "$ALT_DIR" start "Task in alt dir"
+    [ "$status" -eq 0 ]
+
+    grep -q "duration=60" "$ALT_DIR/current" || {
+        echo "Expected duration=60 in $ALT_DIR/current"
+        echo "File contents:"
+        cat "$ALT_DIR/current"
+        rm -rf "$ALT_DIR"
+        return 1
+    }
+
+    rm -rf "$ALT_DIR"
+}
+
+@test "settings with multiple values are parsed correctly" {
+    cat > "$TEST_DIR/settings" << EOF
+default_pomodoro_duration=35
+default_break_duration=7
+daily_goal=10
+EOF
+
+    pomodoro start "Multi-setting task"
+    assert_file_contains "current" "duration=35"
+}
\ No newline at end of file

From 00ff41eea9b7f1aa9393aa5ddd56fbcbd3ffe85c Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 21:51:37 -0400
Subject: [PATCH 10/14] Add create_settings helper for cleaner test code
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Add create_settings() helper that:
- Accepts multiple setting key=value pairs as arguments
- Optionally takes custom file path as first arg (if contains /)
- Defaults to $TEST_DIR/settings
- Makes multi-line settings more readable

Updated all settings tests to use the new helper.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 test/settings.bats    | 19 +++++++++----------
 test/test_helper.bash | 16 ++++++++++++++++
 2 files changed, 25 insertions(+), 10 deletions(-)

diff --git a/test/settings.bats b/test/settings.bats
index ebe545f..1e33182 100644
--- a/test/settings.bats
+++ b/test/settings.bats
@@ -3,7 +3,7 @@
 load test_helper
 
 @test "start uses default pomodoro duration from settings" {
-    echo "default_pomodoro_duration=45" > "$TEST_DIR/settings"
+    create_settings "default_pomodoro_duration=45"
     pomodoro start "Task with custom default"
     assert_file_contains "current" "duration=45"
 }
@@ -14,13 +14,13 @@ load test_helper
 }
 
 @test "start explicit duration overrides settings default" {
-    echo "default_pomodoro_duration=45" > "$TEST_DIR/settings"
+    create_settings "default_pomodoro_duration=45"
     pomodoro start "Task" -d 30
     assert_file_contains "current" "duration=30"
 }
 
 @test "repeat uses default pomodoro duration from settings" {
-    echo "default_pomodoro_duration=50" > "$TEST_DIR/settings"
+    create_settings "default_pomodoro_duration=50"
     pomodoro start "Original task" --ago 5m
     pomodoro finish
     run pomodoro repeat
@@ -29,7 +29,7 @@ load test_helper
 }
 
 @test "break uses default break duration from settings" {
-    echo "default_break_duration=10" > "$TEST_DIR/settings"
+    create_settings "default_break_duration=10"
     create_hook "break" 'echo "BREAK_HOOK_RAN" >> "$TEST_DIR/hook_log"; exit 1'
 
     run pomodoro break
@@ -39,7 +39,7 @@ load test_helper
 
 @test "--directory flag uses settings from specified directory" {
     ALT_DIR="$(mktemp -d)"
-    echo "default_pomodoro_duration=60" > "$ALT_DIR/settings"
+    create_settings "$ALT_DIR/settings" "default_pomodoro_duration=60"
 
     run "$POMODORO_BIN" --directory "$ALT_DIR" start "Task in alt dir"
     [ "$status" -eq 0 ]
@@ -56,11 +56,10 @@ load test_helper
 }
 
 @test "settings with multiple values are parsed correctly" {
-    cat > "$TEST_DIR/settings" << EOF
-default_pomodoro_duration=35
-default_break_duration=7
-daily_goal=10
-EOF
+    create_settings \
+        "default_pomodoro_duration=35" \
+        "default_break_duration=7" \
+        "daily_goal=10"
 
     pomodoro start "Multi-setting task"
     assert_file_contains "current" "duration=35"
diff --git a/test/test_helper.bash b/test/test_helper.bash
index c86e6f1..b7619ed 100644
--- a/test/test_helper.bash
+++ b/test/test_helper.bash
@@ -70,4 +70,20 @@ assert_hook_contains() {
         cat "$TEST_DIR/hook_log"
         return 1
     }
+}
+
+create_settings() {
+    local settings_file="$TEST_DIR/settings"
+
+    if [[ "$1" == *"/"* ]]; then
+        settings_file="$1"
+        shift
+    fi
+
+    > "$settings_file"
+
+    while [ $# -gt 0 ]; do
+        echo "$1" >> "$settings_file"
+        shift
+    done
 }
\ No newline at end of file

From c1a48fc03e4f45f59a02012ba126df243e9b06c4 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 21:54:40 -0400
Subject: [PATCH 11/14] Use explicit create_settings_in for custom paths
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Replace magic path detection with two explicit functions:
- create_settings: writes to $TEST_DIR/settings
- create_settings_in <path>: writes to specified path

This makes the intent clear without relying on / detection.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 test/settings.bats    |  2 +-
 test/test_helper.bash | 19 ++++++++++++-------
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/test/settings.bats b/test/settings.bats
index 1e33182..851dfe8 100644
--- a/test/settings.bats
+++ b/test/settings.bats
@@ -39,7 +39,7 @@ load test_helper
 
 @test "--directory flag uses settings from specified directory" {
     ALT_DIR="$(mktemp -d)"
-    create_settings "$ALT_DIR/settings" "default_pomodoro_duration=60"
+    create_settings_in "$ALT_DIR/settings" "default_pomodoro_duration=60"
 
     run "$POMODORO_BIN" --directory "$ALT_DIR" start "Task in alt dir"
     [ "$status" -eq 0 ]
diff --git a/test/test_helper.bash b/test/test_helper.bash
index b7619ed..fc4110a 100644
--- a/test/test_helper.bash
+++ b/test/test_helper.bash
@@ -74,16 +74,21 @@ assert_hook_contains() {
 
 create_settings() {
     local settings_file="$TEST_DIR/settings"
+    local setting_line
 
-    if [[ "$1" == *"/"* ]]; then
-        settings_file="$1"
-        shift
-    fi
+    for setting_line in "$@"; do
+        echo "$setting_line" >> "$settings_file"
+    done
+}
+
+create_settings_in() {
+    local settings_file="$1"
+    shift
+    local setting_line
 
     > "$settings_file"
 
-    while [ $# -gt 0 ]; do
-        echo "$1" >> "$settings_file"
-        shift
+    for setting_line in "$@"; do
+        echo "$setting_line" >> "$settings_file"
     done
 }
\ No newline at end of file

From 35bdb6da2732bf7d7ca32bb4ba3d6e79017765f5 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 21:56:57 -0400
Subject: [PATCH 12/14] Use Makefile BATS installation in CI
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Remove manual BATS installation from GitHub Actions workflow.
The Makefile now handles downloading BATS automatically, ensuring
consistent versions across local dev and CI environments.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .github/workflows/test.yml | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index aa17d23..ff22d03 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -22,14 +22,5 @@ jobs:
       with:
         go-version: ${{ matrix.go-version }}
 
-    - name: Install BATS
-      run: |
-        if [ "$RUNNER_OS" == "Linux" ]; then
-          sudo apt-get update
-          sudo apt-get install -y bats
-        elif [ "$RUNNER_OS" == "macOS" ]; then
-          brew install bats-core
-        fi
-
     - name: Run tests
       run: make test
\ No newline at end of file

From ff6a0ffe1536f35976464adfc2dc9885bdde568a Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 22:01:49 -0400
Subject: [PATCH 13/14] Fix file endings - add newlines to end of files
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Ensure all files end with a single newline character.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 .github/workflows/test.yml | 2 +-
 test/amend.bats            | 2 +-
 test/break.bats            | 2 +-
 test/cancel.bats           | 1 -
 test/clear.bats            | 1 -
 test/finish.bats           | 2 +-
 test/history.bats          | 2 +-
 test/hooks.bats            | 2 +-
 test/repeat.bats           | 2 +-
 test/settings.bats         | 2 +-
 test/start.bats            | 2 +-
 test/status.bats           | 2 +-
 test/test_helper.bash      | 2 +-
 13 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
index ff22d03..bcb4231 100644
--- a/.github/workflows/test.yml
+++ b/.github/workflows/test.yml
@@ -23,4 +23,4 @@ jobs:
         go-version: ${{ matrix.go-version }}
 
     - name: Run tests
-      run: make test
\ No newline at end of file
+      run: make test
diff --git a/test/amend.bats b/test/amend.bats
index 8504790..f9f9da7 100644
--- a/test/amend.bats
+++ b/test/amend.bats
@@ -44,4 +44,4 @@ load test_helper
     run pomodoro amend
     [ "$status" -eq 0 ]
     assert_file_contains "current" "Task"
-}
\ No newline at end of file
+}
diff --git a/test/break.bats b/test/break.bats
index 386e72e..6e8f8c0 100644
--- a/test/break.bats
+++ b/test/break.bats
@@ -23,4 +23,4 @@ load test_helper
 @test "break with invalid duration fails" {
     run pomodoro break "invalid"
     [ "$status" -ne 0 ]
-}
\ No newline at end of file
+}
diff --git a/test/cancel.bats b/test/cancel.bats
index ad3211c..36727d1 100644
--- a/test/cancel.bats
+++ b/test/cancel.bats
@@ -40,4 +40,3 @@ load test_helper
     [ "$status" -eq 0 ]
     [ -z "$output" ]
 }
-
diff --git a/test/clear.bats b/test/clear.bats
index 6af3728..1050968 100644
--- a/test/clear.bats
+++ b/test/clear.bats
@@ -32,4 +32,3 @@ load test_helper
     [ "$status" -eq 0 ]
     [ -z "$output" ]
 }
-
diff --git a/test/finish.bats b/test/finish.bats
index 453622a..b22d46d 100644
--- a/test/finish.bats
+++ b/test/finish.bats
@@ -51,4 +51,4 @@ load test_helper
     run pomodoro finish
     [ "$status" -eq 0 ]
     assert_file_empty "current"
-}
\ No newline at end of file
+}
diff --git a/test/history.bats b/test/history.bats
index 4da6b96..8ed713b 100644
--- a/test/history.bats
+++ b/test/history.bats
@@ -56,4 +56,4 @@ load test_helper
     [ "$status" -eq 0 ]
     [[ "$output" =~ "Task 1" ]]
     [[ "$output" =~ "Task 2" ]]
-}
\ No newline at end of file
+}
diff --git a/test/hooks.bats b/test/hooks.bats
index d439da1..e7af5c6 100644
--- a/test/hooks.bats
+++ b/test/hooks.bats
@@ -89,4 +89,4 @@ load test_helper
 
     assert_hook_contains "START_EXECUTED"
     assert_hook_contains "STOP_EXECUTED"
-}
\ No newline at end of file
+}
diff --git a/test/repeat.bats b/test/repeat.bats
index 70f3663..713c2c3 100644
--- a/test/repeat.bats
+++ b/test/repeat.bats
@@ -46,4 +46,4 @@ load test_helper
 @test "repeat with no history fails" {
     run pomodoro repeat
     [ "$status" -ne 0 ]
-}
\ No newline at end of file
+}
diff --git a/test/settings.bats b/test/settings.bats
index 851dfe8..c2dd87e 100644
--- a/test/settings.bats
+++ b/test/settings.bats
@@ -63,4 +63,4 @@ load test_helper
 
     pomodoro start "Multi-setting task"
     assert_file_contains "current" "duration=35"
-}
\ No newline at end of file
+}
diff --git a/test/start.bats b/test/start.bats
index 0b961c5..433ddfd 100644
--- a/test/start.bats
+++ b/test/start.bats
@@ -45,4 +45,4 @@ load test_helper
     run pomodoro start "Test task"
     [ "$status" -eq 0 ]
     [[ "$output" =~ "Test task" ]]
-}
\ No newline at end of file
+}
diff --git a/test/status.bats b/test/status.bats
index 19e75d7..8755e18 100644
--- a/test/status.bats
+++ b/test/status.bats
@@ -34,4 +34,4 @@ load test_helper
     run pomodoro status
     [ "$status" -eq 0 ]
     [[ "$output" =~ "❗️" ]]
-}
\ No newline at end of file
+}
diff --git a/test/test_helper.bash b/test/test_helper.bash
index fc4110a..5522f8e 100644
--- a/test/test_helper.bash
+++ b/test/test_helper.bash
@@ -91,4 +91,4 @@ create_settings_in() {
     for setting_line in "$@"; do
         echo "$setting_line" >> "$settings_file"
     done
-}
\ No newline at end of file
+}

From c436128dbf4f2278b96f4064b694c8e9dd865cf1 Mon Sep 17 00:00:00 2001
From: Justin Campbell <justin@justincampbell.me>
Date: Tue, 23 Sep 2025 22:06:35 -0400
Subject: [PATCH 14/14] Remove redundant tests and improve test names
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

- Removed duplicate 'cancel empties current file' test
- Removed duplicate 'clear empties current file' test
- Removed duplicate 'finish preserves description' test
- Updated test names to include 'and' for combined assertions

Reduced test count from 63 to 60 while maintaining coverage.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
---
 test/cancel.bats | 9 +--------
 test/clear.bats  | 9 +--------
 test/finish.bats | 9 ---------
 3 files changed, 2 insertions(+), 25 deletions(-)

diff --git a/test/cancel.bats b/test/cancel.bats
index 36727d1..5638c73 100644
--- a/test/cancel.bats
+++ b/test/cancel.bats
@@ -2,14 +2,7 @@
 
 load test_helper
 
-@test "cancel empties current file" {
-    pomodoro start "Work session"
-    run pomodoro cancel
-    [ "$status" -eq 0 ]
-    assert_file_empty "current"
-}
-
-@test "cancel removes current pomodoro from history" {
+@test "cancel empties current file and removes from history" {
     pomodoro start "Task to cancel"
     run pomodoro cancel
     [ "$status" -eq 0 ]
diff --git a/test/clear.bats b/test/clear.bats
index 1050968..f88b0d1 100644
--- a/test/clear.bats
+++ b/test/clear.bats
@@ -2,14 +2,7 @@
 
 load test_helper
 
-@test "clear empties current file" {
-    pomodoro start "Work session"
-    run pomodoro clear
-    [ "$status" -eq 0 ]
-    assert_file_empty "current"
-}
-
-@test "clear does not affect history" {
+@test "clear empties current file and does not affect history" {
     pomodoro start "First task" --ago 5m
     pomodoro finish
     pomodoro start "Second task"
diff --git a/test/finish.bats b/test/finish.bats
index b22d46d..25b3f81 100644
--- a/test/finish.bats
+++ b/test/finish.bats
@@ -11,15 +11,6 @@ load test_helper
     assert_file_contains "history" "Work session"
 }
 
-@test "finish preserves pomodoro description in history" {
-    pomodoro start "Important task"
-    run pomodoro finish
-    [ "$status" -eq 0 ]
-
-    assert_file_contains "history" 'description="Important task"'
-}
-
-
 @test "finish records actual elapsed time in history" {
     pomodoro start "Work session" -d 30 --ago 10m
     run pomodoro finish