DM Code coverage

.github/workflows/build-test.yml

@@ -43,3 +43,7 @@ jobs:
       run: |
         $env:COMPlus_gcServer=1
         dotnet test --no-build Content.IntegrationTests/Content.IntegrationTests.csproj -v n
+    - name: Upload coverage reports to Codecov
+      uses: codecov/codecov-action@v3
+      with:
+        token: ${{ secrets.CODECOV_TOKEN }}

Content.Tests/DMTests.cs

@@ -5,6 +5,7 @@
 using NUnit.Framework;
 using OpenDreamRuntime;
 using OpenDreamRuntime.Objects;
+using OpenDreamRuntime.Procs.DebugAdapter;
 using OpenDreamRuntime.Rendering;
 using OpenDreamShared.Rendering;
 using Robust.Shared.Asynchronous;
@@ -29,6 +30,7 @@ public sealed class DMTests : ContentUnitTest {
         [Dependency] private readonly DreamManager _dreamMan = default!;
         [Dependency] private readonly DreamObjectTree _objectTree = default!;
         [Dependency] private readonly ITaskManager _taskManager = default!;
+        [Dependency] private readonly IDreamDebugManager _debugManager = default!;
 
         [Flags]
         public enum DMTestFlags {
@@ -71,7 +73,7 @@ private static void Cleanup(string? compiledFile) {
         }
 
         [Test, TestCaseSource(nameof(GetTests))]
-        public void TestFiles(string sourceFile, DMTestFlags testFlags) {
+        public void TestFiles(string sourceFile, string coverageFile, DMTestFlags testFlags) {
             string initialDirectory = Directory.GetCurrentDirectory();
             TestContext.WriteLine($"--- TEST {sourceFile} | Flags: {testFlags}");
             try {
@@ -85,10 +87,15 @@ public void TestFiles(string sourceFile, DMTestFlags testFlags) {
 
                 Assert.IsTrue(compiledFile is not null && File.Exists(compiledFile), "Failed to compile DM source file");
                 Assert.IsTrue(_dreamMan.LoadJson(compiledFile), $"Failed to load {compiledFile}");
+
+                _debugManager.InitializeCoverage(coverageFile);
+
                 _dreamMan.StartWorld();
 
                 (bool successfulRun, DreamValue? returned, Exception? exception) = RunTest();
 
+                _debugManager.Shutdown();
+
                 if (testFlags.HasFlag(DMTestFlags.NoReturn)) {
                     Assert.IsFalse(returned.HasValue, "proc returned unexpectedly");
                 } else {
@@ -137,8 +144,11 @@ public void TestFiles(string sourceFile, DMTestFlags testFlags) {
             var watch = new Stopwatch();
             watch.Start();
 
+            // hack to hopefully force spawned calls to finish
+            var iterationsRemaining = 100;
+
             // Tick until our inner call has finished
-            while (!callTask.IsCompleted) {
+            while (!callTask.IsCompleted || iterationsRemaining-- > 0) {
                 _dreamMan.Update();
                 _taskManager.ProcessPendingTasks();
 
@@ -163,6 +173,7 @@ private static IEnumerable<object[]> GetTests()
 
                 yield return new object[] {
                     sourceFile2,
+                    Path.GetFullPath($"{sourceFile[..^2]}coverage.xml"),
                     testFlags
                 };
             }

OpenDreamRuntime/EntryPoint.cs

@@ -60,13 +60,18 @@ public override void Init() {
         public override void PostInit() {
             _commandSystem = _entitySystemManager.GetEntitySystem<DreamCommandSystem>();
 
+            var codeCoverageOutputFile = _configManager.GetCVar(OpenDreamCVars.CodeCoverage);
+            if (!String.IsNullOrWhiteSpace(codeCoverageOutputFile)) {
+                _debugManager.InitializeCoverage(codeCoverageOutputFile);
+            }
+
             int debugAdapterPort = _configManager.GetCVar(OpenDreamCVars.DebugAdapterLaunched);
             if (debugAdapterPort == 0) {
                 _dreamManager.PreInitialize(_configManager.GetCVar<string>(OpenDreamCVars.JsonPath));
                 _dreamManager.StartWorld();
             } else {
                 // The debug manager is responsible for running _dreamManager.PreInitialize() and .StartWorld()
-                _debugManager.Initialize(debugAdapterPort);
+                _debugManager.InitializeDebugging(debugAdapterPort);
             }
         }
 

OpenDreamRuntime/Objects/DreamObjectTree.cs

@@ -80,16 +80,20 @@
 
         Strings = json.Strings ?? new();
 
+        var types = json.Types ?? Array.Empty<DreamTypeJson>();
+        var procs = json.Procs;
+        var globalProcs = json.GlobalProcs;
+
+        // Load procs first so types can set their init proc's super proc
+        LoadProcsFromJson(types, procs, globalProcs);
+        LoadTypesFromJson(types);
+
         if (json.GlobalInitProc is { } initProcDef) {
-            GlobalInitProc = new DMProc(0, Root, initProcDef, "<global init>", _dreamManager, _atomManager, _dreamMapManager, _dreamDebugManager, _dreamResourceManager, this, _procScheduler);
+            GlobalInitProc = new DMProc(Procs.Count, Root, initProcDef, "<global init>", _dreamManager, _atomManager, _dreamMapManager, _dreamDebugManager, _dreamResourceManager, this, _procScheduler);
+            Procs.Add(GlobalInitProc);
         } else {
             GlobalInitProc = null;
         }
-
-        var procs = json.Procs;
-        var globalProcs = json.GlobalProcs;
-
-        LoadTypesFromJson(types, procs, globalProcs);
     }
 
     public TreeEntry GetTreeEntry(string path) {