Improve github actions logging with grouping (#503)

kenherring · web-flow · commit e3cedb08a2cc · 2026-03-21T11:52:20.000Z
diff --git a/.gitignore b/.gitignore
@@ -3,7 +3,7 @@
 .vscode-test/
 .vscode-test-docker/
 .worktrees/
-artifacts/
+artifacts*/
 coverage/
 dist/
 docker/.rssw/
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,6 +1,16 @@
-# [1.4.9](https://github.com/kenherring/ablunit-test-runner/releases/tag/1.4.9) - 2026-03-19 (pre-release)
-
+# [1.4.11](https://github.com/kenherring/ablunit-test-runner/releases/tag/1.4.11) - 2026-03-21 (pre-release)
+
+* Improve github actions logging with grouping (#503)
+* Bump actions/download-artifact from 5 to 8 (#506)
+* Bump actions/upload-artifact from 4 to 7 (#505)
+* Bump SonarSource/sonarqube-scan-action from 6.0.0 to 7.0.0 (#507)
+* Bump esbuild from 0.27.3 to 0.27.4 (#512)
+* Bump typescript-eslint from 8.57.0 to 8.57.1 (#513)
+* Bump actions/checkout from 4 to 6 (#509)
+* Bump actions/setup-node from 4 to 6 (#504)
 * Configure dependabot for github actions ecosystem (#475)
+* Bump flatted from 3.3.3 to 3.4.2 in the npm_and_yarn group across 1 directory (#502)
+* Bump undici from 7.22.0 to 7.24.4 in the npm_and_yarn group across 1 directory (#501)
 * Bump @tootallnate/once from 1.1.2 to removed in /dummy-ext in the npm_and_yarn group across 1 directory (#495)
 * Bump PCT from v230 to v231 (#476)
 * Add OpenEdge 12.8.11 test target (#484)
@@ -17,7 +27,7 @@
 * Bump esbuild from 0.27.0 to 0.27.1 (#457)
 * Bump jws from 3.2.2 to 3.2.3 in the npm_and_yarn group across 1 directory (#455)
 
-**Full Changelog**: [1.4.4...1.4.9](https://github.com/kenherring/ablunit-test-runner/compare/1.4.7...1.4.9)
+**Full Changelog**: [1.4.4...1.4.11](https://github.com/kenherring/ablunit-test-runner/compare/1.4.9...1.4.11)
 
 # [1.4.4](https://github.com/kenherring/ablunit-test-runner/releases/tag/1.4.4) - 2025-11-26 (pre-release)
 
diff --git a/docker/run_tests.sh b/docker/run_tests.sh
@@ -99,7 +99,7 @@ initialize () {
 		exit 1
 	fi
 
-	if ${CIRCLECI:-false} && [ -z "$ABLUNIT_TEST_RUNNER_OE_VERSION" ]; then
+	if ${CI:-false} && [ -z "$ABLUNIT_TEST_RUNNER_OE_VERSION" ]; then
 		cat "$DLC/version"
 		ABLUNIT_TEST_RUNNER_OE_VERSION=$(awk '{print $3}' < "$DLC/version")
 		if [[ "$ABLUNIT_TEST_RUNNER_OE_VERSION" =~ ^[0-9]+\.[0-9]+$ ]]; then
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -2,7 +2,7 @@
 	"name": "ablunit-test-runner",
 	"displayName": "ABLUnit Test Runner",
 	"description": "OpenEdge ABLUnit test runner for VSCode",
-	"version": "1.4.9",
+	"version": "1.4.11",
 	"engineStrict": true,
 	"galleryBanner": {
 		"color": "#007ACC",
diff --git a/scripts/bump_version.sh b/scripts/bump_version.sh
@@ -20,7 +20,7 @@ main () {
         return 0
     fi
 
-    if ${CIRCLECI:-}; then
+    if ${CI:-false}; then
         if ! git config --get user.email &>/dev/null; then
             LAST_COMMITTER_EMAIL=$(git log -1 --pretty='%ae')
             git config user.email "${LAST_COMMITTER_EMAIL:-noreply@ablunit-test-runner.kenherring.com}"
diff --git a/scripts/common.sh b/scripts/common.sh
@@ -2,14 +2,7 @@
 set -eou pipefail
 
 common_init () {
-	echo "GITHUB_REF_TYPE=${GITHUB_REF_TYPE:-}"
-	echo "GITHUB_REF_NAME=${GITHUB_REF_NAME:-}"
-	echo "GITHUB_HEAD_REF=${GITHUB_HEAD_REF:-}"
-	echo "GITHUB_REF=${GITHUB_REF:-}"
-
- 	# GITHUB_EVENT_NAME=pull_request
 	GITHUB_REF_TYPE="${GITHUB_REF_TYPE:-branch}"
-	CIRCLECI=${GITHUB_ACTIONS:-false}
 	if [ "$GITHUB_REF_TYPE" = "tag" ]; then
 		CIRCLE_BRANCH=
 		CIRCLE_TAG="${GITHUB_REF_NAME:-}"
@@ -22,11 +15,7 @@ common_init () {
 	[ -z "${CIRCLE_TAG:-}" ] && [ -z "${CIRCLE_BRANCH:-}" ] && CIRCLE_TAG=$(git rev-parse --abbrev-ref HEAD)
 	[ -z "${CIRCLE_TAG:-}" ] && [ -z "${CIRCLE_BRANCH:-}" ] && CIRCLE_BRANCH=$(git branch --show-current)
 
-	echo "CIRCLE_TAG=${CIRCLE_TAG:-}"
-	echo "CIRCLE_BRANCH=${CIRCLE_BRANCH:-}"
-	echo "CIRCLECI=${CIRCLECI:-}"
-
-	export CIRCLECI CIRCLE_TAG CIRCLE_BRANCH
+	export CIRCLE_TAG CIRCLE_BRANCH
 }
 
 log_it () {
@@ -37,6 +26,19 @@ log_error () {
 	echo "[$(date +%Y-%m-%d:%H:%M:%S) $0 ${FUNCNAME[1]}] ERROR:" "$@" >&2
 }
 
+log_group_start () {
+	local GROUP_NAME=$1
+	if ${CI:-false}; then
+		echo "::group::$GROUP_NAME"
+	fi
+}
+
+log_group_end () {
+	if ${CI:-false}; then
+		echo "::endgroup::"
+	fi
+}
+
 jq () {
 	if command -v jq >/dev/null 2>&1; then
 		command jq "$@"
diff --git a/scripts/create_release.sh b/scripts/create_release.sh
@@ -35,7 +35,7 @@ main () {
     if $PRERELEASE; then
         ARGS+=(--prerelease)
         ARGS+=(--title "$PACKAGE_VERSION (prerelease)")
-        if ! ${CIRCLECI:-false}; then
+        if ! ${CI:-false}; then
             ARGS+=(--draft)
         fi
     else
diff --git a/scripts/install_and_run.sh b/scripts/install_and_run.sh
@@ -7,11 +7,10 @@ initialize() {
     echo "[$(date +%Y-%m-%d:%H:%M:%S) $0 ${FUNCNAME[0]}] pwd=$(pwd)"
     local VSIX_COUNT
 
-    CIRCLECI=${CIRCLECI:-false}
     # PACKAGE_VERSION=$(node -p "require('./package.json').version")
     export DONT_PROMPT_WSL_INSTALL=true
 
-    $CIRCLECI || ./scripts/package.sh
+    ${CI:-false} || ./scripts/package.sh
 
     echo "vsix files packaged:"
     find . -name 'ablunit-test-runner-*.vsix'
diff --git a/scripts/npm_pretest.sh b/scripts/npm_pretest.sh
@@ -16,8 +16,7 @@ initialize () {
 	if ! command -v ant &>/dev/null; then
 		PATH=$PATH:$DLC/ant/bin
 	fi
-	CIRCLECI=${CIRCLECI:-false}
-	if $CIRCLECI; then
+	if ${CI:-false}; then
 		HOME=/github/home
 	fi
 	NO_BUILD=${NO_BUILD:-false}
@@ -52,7 +51,7 @@ initialize () {
 		exit 1
 	fi
 
-	export PATH CIRCLECI ABLUNIT_TEST_RUNNER_OE_VERSION ABLUNIT_TEST_RUNNER_VSCODE_VERSION
+	export PATH ABLUNIT_TEST_RUNNER_OE_VERSION ABLUNIT_TEST_RUNNER_VSCODE_VERSION
 
 	if [ -d artifacts ]; then
 		rm -rf artifacts/*
@@ -175,8 +174,8 @@ package () {
 	else
 		PACKAGE_OUT_OF_DATE=true
 	fi
-	log_it "CIRCLECI=$CIRCLECI PACKAGE_OUT_OF_DATE=$PACKAGE_OUT_OF_DATE VSIX_COUNT=$VSIX_COUNT"
-	if $PACKAGE_OUT_OF_DATE || $CIRCLECI || [ "$VSIX_COUNT" = "0" ]; then
+	log_it "CI=${CI:-} PACKAGE_OUT_OF_DATE=$PACKAGE_OUT_OF_DATE VSIX_COUNT=$VSIX_COUNT"
+	if $PACKAGE_OUT_OF_DATE || $CI || [ "$VSIX_COUNT" = "0" ]; then
 		./scripts/package.sh
 	fi
 
diff --git a/scripts/npm_version.sh b/scripts/npm_version.sh
@@ -104,7 +104,7 @@ update_changelog () {
 	} > "changelog_$PACKAGE_VERSION.md"
 
 	mv "changelog_$PACKAGE_VERSION.md" CHANGELOG.md
-	if ! ${CIRCLECI:-false}; then
+	if ! ${CI:-false}; then
 		code --wait CHANGELOG.md
 	fi
 }
diff --git a/scripts/run_test_wrapper.sh b/scripts/run_test_wrapper.sh
@@ -4,6 +4,7 @@ set -eou pipefail
 . scripts/common.sh
 
 initialize () {
+	log_group_start "$0"
 	log_it "whoami=$(whoami)"
 
 	VERBOSE=${VERBOSE:-false}
@@ -45,7 +46,7 @@ initialize () {
 	log_it "ABLUNIT_TEST_RUNNER_REPO_DIR=$ABLUNIT_TEST_RUNNER_REPO_DIR"
 	log_it "ABLUNIT_TEST_RUNNER_RUN_SCRIPT_FLAG=$ABLUNIT_TEST_RUNNER_RUN_SCRIPT_FLAG"
 	log_it "ABLUNIT_TEST_RUNNER_UNIT_TESTING=$ABLUNIT_TEST_RUNNER_UNIT_TESTING"
-	log_it "CIRCLECI=$CIRCLECI"
+	log_it "CI=${CI:-}"
 
 	update_oe_version
 }
@@ -142,10 +143,13 @@ run_tests () {
 		export JAVA_TOOL_OPTIONS=${JAVA_TOOL_OPTIONS:-'-Dfile.encoding=UTF8'}
 	fi
 
+	log_group_end
+	log_group_start "npm $RUN_SCRIPT"
 	log_it "starting 'npm $RUN_SCRIPT'"
 	EXIT_CODE=0
 	xvfb-run -a npm run "$RUN_SCRIPT" || EXIT_CODE=$?
 	log_it "xvfb-run end (EXIT_CODE=$EXIT_CODE)"
+	log_group_end
 
 	if [ "$RUN_SCRIPT" = 'test:coverage' ]; then
 		mv coverage/lcov.info artifacts/coverage/lcov.info || true ## https://github.com/microsoft/vscode-test-cli/issues/38
diff --git a/src/ChannelLogger.ts b/src/ChannelLogger.ts
@@ -1,5 +1,5 @@
 /* eslint-disable no-console */
-import { Location, LogLevel, TestItem, TestRun, Uri, window } from 'vscode'
+import { Location, LogLevel, LogOutputChannel, TestItem, TestRun, Uri, window } from 'vscode'
 
 interface ITestInfo {
 	testRun: TestRun
@@ -13,18 +13,30 @@ enum NotificationType {
 	Error = 'Error',
 }
 
+class GroupLogger {
+
+	public start (name: string) {
+		console.info('::group::' + name)
+	}
+
+	public end () {
+		console.info('::endgroup::')
+	}
+}
+
 class Logger {
 	private static instance: Logger
 
-	private readonly logOutputChannel
+	private readonly logOutputChannel: LogOutputChannel
 	private readonly consoleLogLevel = LogLevel.Info
 	private readonly testResultsLogLevel = LogLevel.Info
 	private logLevel: LogLevel
-	private readonly consoleTimestamp = process.env['ABLUNIT_TEST_RUNNER_UNIT_TESTING'] === 'true' || process.env['CIRCLECI']
+	private readonly consoleTimestamp = process.env['ABLUNIT_TEST_RUNNER_UNIT_TESTING'] === 'true' || process.env['CI']
 	private testResultsTimestamp = false
 	private readonly extensionCodeDir = __dirname
 	private readonly baseDir = __dirname
 	notificationsEnabled = true
+	public readonly group: GroupLogger
 
 	private constructor (extCodeDir?: string) {
 		this.logLevel = LogLevel.Info
@@ -36,7 +48,9 @@ class Logger {
 			this.extensionCodeDir = extCodeDir
 		}
 		this.baseDir = Uri.joinPath(Uri.file(this.extensionCodeDir), '..').fsPath
+		this.group = new GroupLogger()
 	}
+
 	public static getInstance () {
 		Logger.instance = Logger.instance ?? new Logger()
 		return Logger.instance
diff --git a/test/parse/OpenedgeProjectParser.test.ts b/test/parse/OpenedgeProjectParser.test.ts
@@ -1,15 +1,15 @@
 import { getDLC, getOEVersion, getOpenEdgeProfileConfig } from 'parse/OpenedgeProjectParser'
-import { assert, getWorkspaceFolders, log } from '../testCommon'
+import { assert, getWorkspaceFolders, log, suiteSetupCommon_logGroup } from '../testCommon'
 import { toUri } from 'FileUtils'
 
 const workspaceFolder = getWorkspaceFolders()[0]
 
 suiteSetup('suiteSetup', () => {
-	log.info('----- suiteSetup -----')
+	suiteSetupCommon_logGroup()
 })
 
-setup('setup', () => {
-	log.info('----- setup -----')
+suiteTeardown('suiteTeardown', () => {
+	log.group.end()
 })
 
 test('OpenedgeProjectParser.test.1', () => {
diff --git a/test/parse/SourceMapRCodeParser.test.ts b/test/parse/SourceMapRCodeParser.test.ts
@@ -1,11 +1,12 @@
 import { commands, Uri } from 'vscode'
-import { assert, getRcodeCount, log, sleep, toUri } from '../testCommon'
+import { assert, getRcodeCount, log, sleep, suiteSetupCommon_logGroup, toUri } from '../testCommon'
 import { PropathParser } from 'ABLPropath'
 import { SourceMapItem } from 'parse/SourceMapParser'
 import { getSourceMapFromRCode } from 'parse/SourceMapRCodeParser'
 import * as FileUtils from 'FileUtils'
 
 suiteSetup('suiteSetup', () => {
+	suiteSetupCommon_logGroup()
 	log.info('delete xref files')
 	FileUtils.deleteFile(toUri('test_1/test.p.xref'))
 	FileUtils.deleteFile(toUri('test_2/test.p.xref'))
diff --git a/test/parse/SourceMapXrefParser.test.ts b/test/parse/SourceMapXrefParser.test.ts
@@ -1,11 +1,12 @@
 import { commands, Uri } from 'vscode'
-import { assert, getXrefCount, log, sleep, toUri } from '../testCommon'
+import { assert, getXrefCount, log, sleep, suiteSetupCommon_logGroup, toUri } from '../testCommon'
 import { PropathParser } from 'ABLPropath'
 import { SourceMapItem } from 'parse/SourceMapParser'
 import { getSourceMapFromXref } from 'parse/SourceMapXrefParser'
 import * as FileUtils from 'FileUtils'
 
 suiteSetup('suiteSetup', () => {
+	suiteSetupCommon_logGroup()
 	log.info('delete xref files')
 	FileUtils.deleteFile(toUri('test_1/test.p.xref'))
 	FileUtils.deleteFile(toUri('test_2/test.p.xref'))
diff --git a/test/suites/AtStart.test.ts b/test/suites/AtStart.test.ts
@@ -3,7 +3,7 @@ import { assert, log, runAllTests, suiteSetupCommon, extensions } from '../testC
 suite('projAtStart  - Extension Test Suite - bdd', () => {
 
 	suiteSetup('projAtStart - before', async () => {
-		await suiteSetupCommon(undefined, 0)
+		await suiteSetupCommon(undefined, undefined, 0)
 	})
 
 	test('projAtStart - ${workspaceFolder}/ablunit.json file exists - return promise', () => {
diff --git a/test/suites/DebugLines.test.ts b/test/suites/DebugLines.test.ts
@@ -8,7 +8,7 @@ import { ABLUnitConfig } from 'ABLUnitConfigWriter'
 const workspaceFolder = workspace.workspaceFolders![0]
 
 suiteSetup('debugLines - before', async () => {
-	await suiteSetupCommon(undefined, 10)
+	await suiteSetupCommon(undefined, undefined, 10)
 	const rcodeCount = getRcodeCount()
 	if (rcodeCount < 10) {
 		throw new Error('rcodeCount=' + rcodeCount + ' < 9')
diff --git a/test/suites/DebugListingPreview.test.ts b/test/suites/DebugListingPreview.test.ts
@@ -25,7 +25,7 @@ async function validateSelectionAfterChange (uri: Uri, expectedSelection: number
 }
 
 suiteSetup('debugLines - before', async () => {
-	await suiteSetupCommon(undefined, 1)
+	await suiteSetupCommon(undefined, undefined, 1)
 	const rcodeCount = getRcodeCount()
 	if (rcodeCount < 1) {
 		throw new Error('rcodeCount=' + rcodeCount + ' < 1')
diff --git a/test/suites/proj0.test.ts b/test/suites/proj0.test.ts
@@ -641,7 +641,7 @@ test('proj0.29 - database connection not valid', async () => {
 })
 
 test('proj0.30 - database connection not valid', async () => {
-	if (!process.env['CIRCLECI']) {
+	if (!process.env['CI']) {
 		log.info('skipping proj0.30 as it has a long execution time and is intended to run on CI only')
 		return
 	}
diff --git a/test/suites/proj7A.test.ts b/test/suites/proj7A.test.ts
@@ -3,7 +3,7 @@ import { assert, beforeProj7, Duration, log, runAllTests, suiteSetupCommon } fro
 suite('proj7A - Extension Test Suite', () => {
 
 	suiteSetup('proj7A - before', () => {
-		return suiteSetupCommon()
+		return suiteSetupCommon('proj7A')
 			.then(() => { return beforeProj7() })
 			.then(() => {
 				log.info('suiteSetup complete')
diff --git a/test/suites/proj7B.test.ts b/test/suites/proj7B.test.ts
@@ -1,8 +1,9 @@
 import { CancellationError, LogLevel, commands } from 'vscode'
-import { assert, RunStatus, beforeCommon, beforeProj7, cancelTestRun, getCurrentRunData, getTestControllerItemCount, isoDate, log, refreshTests, runAllTests, waitForTestRunStatus, sleep } from '../testCommon'
+import { assert, RunStatus, beforeCommon, beforeProj7, cancelTestRun, getCurrentRunData, getTestControllerItemCount, isoDate, log, refreshTests, runAllTests, waitForTestRunStatus, sleep, suiteSetupCommon } from '../testCommon'
 import { Duration } from 'ABLUnitCommon'
 
 suiteSetup('proj7B - before', async () => {
+	await suiteSetupCommon('proj7B')
 	beforeCommon()
 	await beforeProj7()
 })
diff --git a/test/suites/workspace0.test.ts b/test/suites/workspace0.test.ts
@@ -3,7 +3,7 @@ import { assert, getWorkspaceFolders, runAllTests, suiteSetupCommon, Uri } from
 suite('workspace0 - Extension Test Suite', () => {
 
 	suiteSetup('workspace0 - before', async () => {
-		await suiteSetupCommon()
+		await suiteSetupCommon('workspace0')
 	})
 
 	test('workspace0.1 - workspaceFolder/ablunit.json file exists', async () => {
diff --git a/test/suites/workspace1.test.ts b/test/suites/workspace1.test.ts
@@ -5,7 +5,7 @@ import * as FileUtils from 'FileUtils'
 suite('workspace1 - Extension Test Suite', () => {
 
 	suiteSetup('workspace1 - before', async () => {
-		await suiteSetupCommon()
+		await suiteSetupCommon('workspace1')
 
 		FileUtils.deleteFile([
 			Uri.joinPath(getWorkspaceUri(0), '.vscode', 'ablunit-test-profile.json'),
diff --git a/test/testCommon.ts b/test/testCommon.ts
