More Unit Tests

dotnet/typeagent/src/knowproStorage/Sqlite/SqliteMessageCollection.cs

@@ -480,7 +480,7 @@ public static IEnumerable<MessageRow> GetSlice(SqliteDatabase db, int startOrdin
         ArgumentVerify.ThrowIfGreaterThan(startOrdinal, endOrdinal, nameof(startOrdinal));
 
         return db.Enumerate(@"
-SELECT chunks, chunk_uri, message_length, start_timestamp, tags, metadata, extra
+SELECT msg_id, chunks, chunk_uri, message_length, start_timestamp, tags, metadata, extra
 FROM Messages WHERE msg_id >= @start_id AND msg_id < @end_id
 ORDER BY msg_id",
             (cmd) =>

dotnet/typeagent/src/typechat/TypeChat.csproj

@@ -19,4 +19,8 @@
     <ProjectReference Include="..\typechat.schema\TypeChat.Schema.csproj" />
   </ItemGroup>
 
+  <ItemGroup>
+    <InternalsVisibleTo Include="Microsoft.TypeChat.Tests" />
+  </ItemGroup>  
+
 </Project>

dotnet/typeagent/tests/conversationMemory.test/PodcastTests.cs

@@ -27,8 +27,6 @@ private class TestTranscriptInfo
         public string name { get; set; } = string.Empty;
         public System.DateTime date { get; set; } = System.DateTime.Now;
         public uint length { get; set; } = 0;
-        public uint? participantCount { get; set; } = null;
-        public uint? messageCount { get; set; } = null;
     }
 
     private static TestTranscriptInfo GetTransscriptSmall()
@@ -39,8 +37,6 @@ private static TestTranscriptInfo GetTransscriptSmall()
             name = "Test",
             date = System.DateTime.Parse("March 2024"),
             length = 15,
-            messageCount = 7,
-            participantCount = 5,
         };
     }
 
@@ -62,7 +58,7 @@ public async Task BuildIndexAsync()
         Assert.Equal(["hamlet", "lady bracknell", "macbeth", "richard", "sherlock holmes"], participants);
 
         var terms = await podcast.SemanticRefIndex.LookupTermAsync("misfortune");
-        Assert.True(terms?.Count > 0);
+        Assert.True(terms!.Count > 0);
     }
 
     private async Task<Podcast> ImportTestPodcastAsync(TestTranscriptInfo podcastDetails, bool online)

dotnet/typeagent/tests/testLib/MockModels.cs

@@ -0,0 +1,40 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Text;
+using System.Threading.Tasks;
+using Microsoft.TypeChat;
+using TypeAgent.AIClient;
+
+namespace TypeAgent.TestLib;
+
+public class MockModel_No_JSON_Response : IChatModel
+{
+    public Task<string> CompleteAsync(Prompt prompt, TranslationSettings? settings, CancellationToken cancelToken)
+    {
+
+        return Task.Run(() => "Mock response", cancelToken);
+    }
+
+    public Task<string> CompleteTextAsync(Prompt prompt, TranslationSettings? settings, CancellationToken cancelToken)
+    {
+        return Task.Run(() => "Mock response", cancelToken);
+    }
+}
+
+public class MockModel_Partial_JSON_Response : IChatModel
+{
+    public Task<string> CompleteAsync(Prompt prompt, TranslationSettings? settings, CancellationToken cancelToken)
+    {
+
+        return Task.Run(() => "{ \"text\": \"partial json\"", cancelToken);
+    }
+
+    public Task<string> CompleteTextAsync(Prompt prompt, TranslationSettings? settings, CancellationToken cancelToken)
+    {
+        return Task.Run(() => "{ \"text\": \"partial json\"", cancelToken);
+    }
+}

dotnet/typeagent/tests/typeChat.test/ArgumentVerifyTests.cs

@@ -0,0 +1,72 @@
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
+using System;
+using System.Collections.Generic;
+using Xunit;
+
+namespace Microsoft.TypeChat.Tests;
+
+public class ArgumentVerifyTests
+{
+    [Fact]
+    public void Throw_ThrowsArgumentException_WithMessage()
+    {
+        var ex = Assert.Throws<ArgumentException>(() => ArgumentVerify.Throw("test message"));
+        Assert.Equal("test message", ex.Message);
+    }
+
+    [Fact]
+    public void ThrowIfNull_WithNull_ThrowsArgumentNullException()
+    {
+        var ex = Assert.Throws<ArgumentNullException>(() => ArgumentVerify.ThrowIfNull(null, "param"));
+        Assert.Equal("param", ex.ParamName);
+    }
+
+    [Fact]
+    public void ThrowIfNull_WithNonNull_DoesNotThrow()
+    {
+        ArgumentVerify.ThrowIfNull(new object(), "param");
+    }
+
+    [Fact]
+    public void ThrowIfNullOrEmpty_StringNull_ThrowsArgumentNullException()
+    {
+        var ex = Assert.Throws<ArgumentNullException>(() => ArgumentVerify.ThrowIfNullOrEmpty((string)null, "param"));
+        Assert.Equal("param", ex.ParamName);
+    }
+
+    [Fact]
+    public void ThrowIfNullOrEmpty_StringEmpty_ThrowsArgumentException()
+    {
+        var ex = Assert.Throws<ArgumentException>(() => ArgumentVerify.ThrowIfNullOrEmpty("", "param"));
+        Assert.Equal("The value cannot be an empty string. (Parameter 'param')", ex.Message);
+    }
+
+    [Fact]
+    public void ThrowIfNullOrEmpty_StringValid_DoesNotThrow()
+    {
+        ArgumentVerify.ThrowIfNullOrEmpty("valid", "param");
+    }
+
+    [Fact]
+    public void ThrowIfNullOrEmpty_ListNull_ThrowsArgumentNullException()
+    {
+        List<int> list = null;
+        var ex = Assert.Throws<ArgumentNullException>(() => ArgumentVerify.ThrowIfNullOrEmpty(list, "param"));
+        Assert.Equal("param", ex.ParamName);
+    }
+
+    [Fact]
+    public void ThrowIfNullOrEmpty_ListEmpty_ThrowsArgumentException()
+    {
+        var ex = Assert.Throws<ArgumentException>(() => ArgumentVerify.ThrowIfNullOrEmpty(new List<int>(), "param"));
+        Assert.Equal("The list cannot be empty. (Parameter 'param')", ex.Message);
+    }
+
+    [Fact]
+    public void ThrowIfNullOrEmpty_ListValid_DoesNotThrow()
+    {
+        ArgumentVerify.ThrowIfNullOrEmpty(new List<int> { 1 }, "param");
+    }
+}