Include the large source test

Rekkonnect · Aug 10, 2024 · f0993d8 · f0993d8
1 parent 94eaff5
commit f0993d8
Show file tree

Hide file tree

Showing 6 changed files with 68 additions and 41 deletions.
diff --git a/Syndiesis.Tests/BaseProjectCodeTests.cs b/Syndiesis.Tests/BaseProjectCodeTests.cs
@@ -7,11 +7,23 @@ namespace Syndiesis.Tests;
 public abstract class BaseProjectCodeTests
 {
     protected static readonly ProjectSourceProvider SourceProvider
-        = ProjectSourceProvider.Get();
+        = Syndiesis.ProjectSourceProviderGetter.Get();
 
-    protected static readonly ImmutableArray<FileInfo> FilesToTest
+    protected static readonly ProjectSourceProvider TestSourceProvider
+        = Syndiesis.Tests.ProjectSourceProviderGetter.Get();
+
+    protected static readonly ImmutableArray<FileInfo> MainFilesToTest
         = SourceProvider.GetFilePaths();
 
+    protected static readonly ImmutableArray<FileInfo> TestFilesToTest
+        = TestSourceProvider.GetFilePaths();
+
+    protected static readonly ImmutableArray<FileInfo> FilesToTest =
+    [
+        .. MainFilesToTest,
+        .. TestFilesToTest,
+    ];
+
     [Test]
     public async Task TestAllFilesIndependently()
     {

diff --git a/Syndiesis.Tests/NodeViewDetailsHandlerTests.cs b/Syndiesis.Tests/NodeViewDetailsHandlerTests.cs
@@ -1,6 +1,8 @@
-using Microsoft.CodeAnalysis;
+using Garyon.Extensions;
+using Microsoft.CodeAnalysis;
 using Microsoft.CodeAnalysis.Text;
 using Syndiesis.Core;
+using Syndiesis.Utilities;
 
 namespace Syndiesis.Tests;
 
@@ -18,16 +20,29 @@ protected override async Task TestSource(string text)
     private static async Task TestEntireHybridCompilationTree(
         HybridSingleTreeCompilationSource hybridCompilation)
     {
-        var tree = hybridCompilation.CurrentSource.Tree;
-        Assert.That(tree, Is.Not.Null);
-        var root = await tree.GetRootAsync();
-        Assert.That(root, Is.Not.Null);
+        var profiling = new SimpleProfiling();
+        int nodeCount = 0;
+        int length = 0;
+        using (var _ = profiling.BeginProcess())
+        {
+            var tree = hybridCompilation.CurrentSource.Tree;
+            Assert.That(tree, Is.Not.Null);
+            var root = await tree.GetRootAsync();
+            Assert.That(root, Is.Not.Null);
+            length = root.FullSpan.Length;
+
+            var nodes = root.DescendantNodesAndSelf(descendIntoTrivia: true)
+                .ToList();
+            nodeCount = nodes.Count;
+            await Parallel.ForEachAsync(
+                nodes,
+                TestNodeLocal);
+        }
 
-        var nodes = root.DescendantNodesAndSelf(descendIntoTrivia: true)
-            .ToList();
-        await Parallel.ForEachAsync(
-            nodes,
-            TestNodeLocal);
+        var seconds = profiling.SnapshotResults!.Time.TotalSeconds;
+        TestContext.Progress.WriteLine($"""
+            Finished testing all {nodeCount} nodes from {length} characters in {seconds:N3}s
+            """);
 
         async ValueTask TestNodeLocal(SyntaxNode node, CancellationToken cancellationToken)
         {
@@ -55,6 +70,7 @@ private static async Task TestNode(
         {
             Assert.That(rootNode?.FullSpan, Is.EqualTo(node.FullSpan));
         }
+
     }
 
     private static async Task<NodeViewAnalysisExecution> TestExecutingResult(
@@ -68,26 +84,8 @@ private static async Task<NodeViewAnalysisExecution> TestExecutingResult(
         var result = execution.ExecuteCore(default);
         Assert.That(result, Is.Not.Null);
 
-        bool allSuccessful = await result.AwaitAllLoaded();
+        bool allSuccessful = await result.AwaitAllLoaded(TimeSpan.FromMilliseconds(45));
         Assert.That(allSuccessful, Is.True);
         return execution;
     }
-
-    [Test]
-    public async Task TestAllFilesWithFlow()
-    {
-        TestContext.Progress.WriteLine(
-            "Began testing the node view data analysis on all files sequentially, this will take some more time.");
-
-        var hybridCompilation = new HybridSingleTreeCompilationSource();
-
-        foreach (var file in FilesToTest)
-        {
-            var text = await File.ReadAllTextAsync(file.FullName);
-            hybridCompilation.SetSource(text, default);
-            await TestEntireHybridCompilationTree(hybridCompilation);
-
-            TestContext.Progress.WriteLine($"Processed file {file.FullName}");
-        }
-    }
 }
diff --git a/Syndiesis.Tests/ProjectSourceProviderGetter.cs b/Syndiesis.Tests/ProjectSourceProviderGetter.cs
@@ -0,0 +1,10 @@
+namespace Syndiesis.Tests;
+
+public static class ProjectSourceProviderGetter
+{
+    public static ProjectSourceProvider Get()
+    {
+        var thisPath = ProjectSourceProvider.CallerFilePath();
+        return new(thisPath);
+    }
+}
diff --git a/Syndiesis/Controls/AnalysisVisualization/NodeDetailsViewData.cs b/Syndiesis/Controls/AnalysisVisualization/NodeDetailsViewData.cs
@@ -1,5 +1,6 @@
 using Garyon.Functions;
 using Syndiesis.Core.DisplayAnalysis;
+using System;
 using System.Collections.Generic;
 using System.Linq;
 using System.Threading.Tasks;
@@ -40,14 +41,16 @@ SemanticModelSection SemanticModel
         ];
     }
 
-    public async Task<bool> AwaitAllLoaded()
+    public async Task<bool> AwaitAllLoaded(TimeSpan expectedDelay = default)
     {
         var nodeLoaders = AllNodes()
             .Select(s => s.NodeLoader)
             .Where(Predicates.NotNull)
             .ToList()
             ;
 
+        await Task.Delay(expectedDelay);
+
         await Task.WhenAll(nodeLoaders!);
         return nodeLoaders.All(l => l!.IsCompletedSuccessfully);
     }

diff --git a/Syndiesis/ProjectSourceProvider.cs b/Syndiesis/ProjectSourceProvider.cs
@@ -4,10 +4,10 @@
 
 namespace Syndiesis;
 
-public sealed class ProjectSourceProvider(string? callerFilePath)
+public class ProjectSourceProvider(string? callerFilePath)
 {
     private readonly string? _callerFilePath = callerFilePath;
-    
+
     public ImmutableArray<FileInfo> GetFilePaths()
     {
         if (string.IsNullOrEmpty(_callerFilePath))
@@ -21,14 +21,8 @@ public ImmutableArray<FileInfo> GetFilePaths()
         return [.. files];
     }
 
-    private static string? ThisPath([CallerFilePath] string? callerFilePath = null)
+    public static string? CallerFilePath([CallerFilePath] string? callerFilePath = null)
     {
         return callerFilePath;
     }
-
-    public static ProjectSourceProvider Get()
-    {
-        var thisPath = ThisPath();
-        return new(thisPath);
-    }
 }
diff --git a/Syndiesis/ProjectSourceProviderGetter.cs b/Syndiesis/ProjectSourceProviderGetter.cs
@@ -0,0 +1,10 @@
+namespace Syndiesis;
+
+public static class ProjectSourceProviderGetter
+{
+    public static ProjectSourceProvider Get()
+    {
+        var thisPath = ProjectSourceProvider.CallerFilePath();
+        return new(thisPath);
+    }
+}