From 57ce78ab067c897ac792e59142fe6f0eab4800e2 Mon Sep 17 00:00:00 2001
From: Tomas Slusny <slusnucky@gmail.com>
Date: Tue, 26 Aug 2025 23:01:46 +0200
Subject: [PATCH 1/2] test: add unit tests for class, orderedmap, stringbuffer

Add new unit tests for CopilotChat.utils.class, orderedmap, and stringbuffer
modules. These tests cover class creation, inheritance, ordered map behavior,
and string buffer operations to improve code reliability and coverage.

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
---
 tests/class_spec.lua        | 29 +++++++++++++++++++++++++++++
 tests/orderedmap_spec.lua   | 28 ++++++++++++++++++++++++++++
 tests/stringbuffer_spec.lua | 23 +++++++++++++++++++++++
 3 files changed, 80 insertions(+)
 create mode 100644 tests/class_spec.lua
 create mode 100644 tests/orderedmap_spec.lua
 create mode 100644 tests/stringbuffer_spec.lua

diff --git a/tests/class_spec.lua b/tests/class_spec.lua
new file mode 100644
index 00000000..73b32d58
--- /dev/null
+++ b/tests/class_spec.lua
@@ -0,0 +1,29 @@
+local class = require('CopilotChat.utils.class')
+
+describe('CopilotChat.utils.class', function()
+  it('creates a simple class', function()
+    local Foo = class(function(self, x)
+      self.x = x
+    end)
+    local obj = Foo(42)
+    assert.equals(42, obj.x)
+  end)
+
+  it('supports init method', function()
+    local Bar = class(function(self, y)
+      self.y = y
+    end)
+    local obj = Bar.new(7)
+    assert.equals(7, obj.y)
+    obj:init(8)
+    assert.equals(8, obj.y)
+  end)
+
+  it('supports inheritance', function()
+    local Parent = class(function(self) self.val = 1 end)
+    local Child = class(function(self) self.val = 2 end, Parent)
+    local obj = Child()
+    assert.equals(2, obj.val)
+    assert.equals(Parent, getmetatable(Child).__index)
+  end)
+end)
diff --git a/tests/orderedmap_spec.lua b/tests/orderedmap_spec.lua
new file mode 100644
index 00000000..9000915c
--- /dev/null
+++ b/tests/orderedmap_spec.lua
@@ -0,0 +1,28 @@
+local orderedmap = require('CopilotChat.utils.orderedmap')
+
+describe('CopilotChat.utils.orderedmap', function()
+  it('sets and gets values', function()
+    local map = orderedmap()
+    map:set('a', 1)
+    map:set('b', 2)
+    assert.equals(1, map:get('a'))
+    assert.equals(2, map:get('b'))
+  end)
+
+  it('preserves insertion order', function()
+    local map = orderedmap()
+    map:set('x', 10)
+    map:set('y', 20)
+    map:set('z', 30)
+    assert.are.same({ 'x', 'y', 'z' }, map:keys())
+    assert.are.same({ 10, 20, 30 }, map:values())
+  end)
+
+  it('overwrites value but not order', function()
+    local map = orderedmap()
+    map:set('a', 1)
+    map:set('a', 2)
+    assert.are.same({ 'a' }, map:keys())
+    assert.are.same({ 2 }, map:values())
+  end)
+end)
diff --git a/tests/stringbuffer_spec.lua b/tests/stringbuffer_spec.lua
new file mode 100644
index 00000000..d491fd43
--- /dev/null
+++ b/tests/stringbuffer_spec.lua
@@ -0,0 +1,23 @@
+local stringbuffer = require('CopilotChat.utils.stringbuffer')
+
+describe('CopilotChat.utils.stringbuffer', function()
+  it('concatenates strings with put', function()
+    local buf = stringbuffer()
+    buf:put('hello')
+    buf:put(' ')
+    buf:put('world')
+    assert.equals('hello world', buf:tostring())
+  end)
+
+  it('sets buffer with set', function()
+    local buf = stringbuffer()
+    buf:put('foo')
+    buf:set('bar')
+    assert.equals('bar', buf:tostring())
+  end)
+
+  it('handles empty buffer', function()
+    local buf = stringbuffer()
+    assert.equals('', buf:tostring())
+  end)
+end)

From 5a1207d18e19a80bac8139d8cbe5e9df1838cbdd Mon Sep 17 00:00:00 2001
From: "pre-commit-ci[bot]"
 <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Date: Tue, 26 Aug 2025 21:02:16 +0000
Subject: [PATCH 2/2] [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci
---
 tests/class_spec.lua | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/tests/class_spec.lua b/tests/class_spec.lua
index 73b32d58..ef2f1657 100644
--- a/tests/class_spec.lua
+++ b/tests/class_spec.lua
@@ -20,8 +20,12 @@ describe('CopilotChat.utils.class', function()
   end)
 
   it('supports inheritance', function()
-    local Parent = class(function(self) self.val = 1 end)
-    local Child = class(function(self) self.val = 2 end, Parent)
+    local Parent = class(function(self)
+      self.val = 1
+    end)
+    local Child = class(function(self)
+      self.val = 2
+    end, Parent)
     local obj = Child()
     assert.equals(2, obj.val)
     assert.equals(Parent, getmetatable(Child).__index)