From b005b331bd398890c4d17e055e10170765841ebf Mon Sep 17 00:00:00 2001
From: DevForge Engineer <engineer@devforge.dev>
Date: Mon, 18 May 2026 17:42:03 -0400
Subject: [PATCH] fix: remove duplicated TestChangeStr class in test_diff.py
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The TestChangeStr class was defined twice in the same file. The first
definition was entirely dead code — its test methods were overridden
by the second definition's methods with the same names.

This fixes ruff F811 (redefinition of unused variable) when linting
tests/.
---
 tests/test_diff.py | 27 ---------------------------
 1 file changed, 27 deletions(-)

diff --git a/tests/test_diff.py b/tests/test_diff.py
index aa3f91c..79ea278 100644
--- a/tests/test_diff.py
+++ b/tests/test_diff.py
@@ -10,33 +10,6 @@
 )
 
 
-class TestChangeStr:
-    def test_str_added(self):
-        """Change.__str__ for added key."""
-        c = Change(key="port", change_type=ChangeType.ADDED, new_value=8080, env="prod")
-        s = str(c)
-        assert "[+]" in s
-        assert "port" in s
-        assert "8080" in s
-        assert "prod" in s
-
-    def test_str_removed(self):
-        """Change.__str__ for removed key."""
-        c = Change(key="debug", change_type=ChangeType.REMOVED, old_value=True, env="dev")
-        s = str(c)
-        assert "[-]" in s
-        assert "debug" in s
-        assert "True" in s or "true" in s
-
-    def test_str_changed(self):
-        """Change.__str__ for changed key."""
-        c = Change(key="host", change_type=ChangeType.CHANGED, old_value="localhost", new_value="prod.example.com", env="prod")
-        s = str(c)
-        assert "[~]" in s
-        assert "localhost" in s
-        assert "prod.example.com" in s
-
-
 class TestDiffConfigs:
     def test_no_changes(self):
         base = {"host": "localhost", "port": 8080}