From ff5a91153f5ab3770f8691f7372f5b74aab2166f Mon Sep 17 00:00:00 2001 From: Chris Ge Date: Mon, 24 Jun 2024 17:09:51 -0400 Subject: [PATCH] Was failing the CI tests on github, maybe the issue is that extra in LogAssert is passed in by reference? --- cls/TestCoverage/Manager.cls | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cls/TestCoverage/Manager.cls b/cls/TestCoverage/Manager.cls index 20ae2d8..8f395f8 100644 --- a/cls/TestCoverage/Manager.cls +++ b/cls/TestCoverage/Manager.cls @@ -178,10 +178,10 @@ Method CoverageTargetsSet(%value) As %Status [ Internal, Private ] Quit $$$OK } -Method LogAssert(success, action, description, extra, location) +Method LogAssert(success, action, description, ByRef extra, location) { set ^IRIS.TEMPCG($i(^IRIS.TEMPCG)) = "We're in LogAssert" - do ##super(success, action, description, extra, location) + do ##super(success, action, description, .extra, location) if (..InteroperabilityProcesses) { // Collect the list of PIDs used by Interoperability processes in the current namespace // Run unit tests collecting coverage in that specific list of processes (plus the current process)