Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Sep 15, 2025

The BDD test workflow summary was showing the full Docker build output alongside the actual test results, making it difficult to quickly see the test status. This change filters the output to show only the relevant test execution results in the GitHub workflow summary while preserving the complete output in the job logs.

Changes Made

Modified .github/workflows/on_call_staging_test.yaml to add AWK filtering that:

  • Captures output starting from the npm script execution (> orionrobots.github.io@1.0.0 test:bdd)
  • Excludes all Docker build output from the workflow summary
  • Preserves the complete output (including Docker build logs) in the job logs for debugging

Before

The workflow summary showed verbose Docker build output mixed with test results:

[+] Building 45.2s (10/10) FINISHED
 => [internal] load build definition from Dockerfile...
 => [tests 5/8] RUN apt-get update && apt-get install...
 => [tests 6/8] RUN npx playwright install chromium...
> orionrobots.github.io@1.0.0 test:bdd
> cucumber-js tests/staging/features...
2 scenarios (2 passed)
7 steps (7 passed)

After

The workflow summary now shows only the clean test results:

> orionrobots.github.io@1.0.0 test:bdd
> cucumber-js tests/staging/features --require tests/staging/step_definitions/**/*.js

.......

2 scenarios (2 passed)
7 steps (7 passed)
0m03.387s (executing steps: 0m03.198s)

The Docker build output remains available in the full job logs for debugging purposes.

Fixes #286.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: dannystaple <426859+dannystaple@users.noreply.github.com>
@Copilot Copilot AI changed the title [WIP] BDD test output should be abridged Filter Docker build output from BDD test workflow summary Sep 15, 2025
@Copilot Copilot AI requested a review from dannystaple September 15, 2025 09:47
Copilot finished work on behalf of dannystaple September 15, 2025 09:47
Copy link
Contributor

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Warnings Elapsed time
❌ ACTION actionlint 5 6 0 0.08s
✅ BASH bash-exec 1 0 0 0.0s
✅ BASH shellcheck 1 0 0 0.06s
⚠️ BASH shfmt 1 1 0 0.0s
❌ C cppcheck 8 8 0 0.29s
❌ CPP cppcheck 8 8 0 0.27s
✅ JSON npm-package-json-lint yes no no 0.41s
✅ JSON v8r 12 0 0 8.92s
❌ MARKDOWN markdown-link-check 741 214 0 586.88s
✅ REPOSITORY gitleaks yes no no 13.42s
✅ REPOSITORY git_diff yes no no 0.16s
❌ REPOSITORY grype yes 2 no 33.74s
✅ REPOSITORY secretlint yes no no 42.01s
✅ REPOSITORY syft yes no no 1.53s
✅ REPOSITORY trivy-sbom yes no no 3.17s
✅ REPOSITORY trufflehog yes no no 4.4s
✅ XML xmllint 1 0 0 197.48s
✅ YAML v8r 13 0 0 5.84s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@dannystaple dannystaple marked this pull request as ready for review September 15, 2025 13:09
@dannystaple dannystaple merged commit 787b693 into master Sep 15, 2025
6 checks passed
@dannystaple dannystaple deleted the copilot/fix-286 branch September 15, 2025 13:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BDD test output should be abridged
2 participants