Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
8eee923
Added GTest result summary
YannikSpaeth Apr 15, 2026
918fb28
Update GitHub workflows to trigger on workflow_dispatch instead of pu…
YannikSpaeth Apr 15, 2026
c4d7fe7
T1
YannikSpaeth Apr 15, 2026
7c4932a
Enhance CI workflows to handle missing XML artifacts gracefully and u…
YannikSpaeth Apr 15, 2026
ff02192
Create test results directory before running tests and update JUnit o…
YannikSpaeth Apr 15, 2026
e482097
Aded Tag from ecal repo
YannikSpaeth Apr 17, 2026
666555a
Changed xml build directory
YannikSpaeth Apr 17, 2026
d3deba8
Added intentinal GTest Fails
YannikSpaeth Apr 17, 2026
0920c1c
Wire fork CI to reusable summary workflow and upload Windows JUnit ar…
YannikSpaeth Apr 21, 2026
9549d50
removed old scripts
YannikSpaeth Apr 21, 2026
9e7a617
Merge pull request #1 from YannikSpaeth/test/sum
YannikSpaeth Apr 21, 2026
894c672
Merge branch 'master' into test/reusable-ci-summary
YannikSpaeth Apr 21, 2026
7a2a7ed
Merge pull request #2 from YannikSpaeth/test/reusable-ci-summary
YannikSpaeth Apr 21, 2026
3db1a9f
removed hardcoded collect and report
YannikSpaeth Apr 21, 2026
e0b52a8
Fixed Readme
YannikSpaeth Apr 21, 2026
81e8d43
Use per-matrix Ubuntu JUnit filenames to avoid artifact overwrite
YannikSpaeth Apr 21, 2026
3dea1d2
Fix Ubuntu JUnit path: use absolute workspace path for ctest output/u…
YannikSpaeth Apr 21, 2026
7d5efc8
ci: trigger ubuntu summary with updated parser
YannikSpaeth Apr 21, 2026
3677234
ci: trigger summary with log-line links
YannikSpaeth Apr 21, 2026
729e585
Update reusable workflow repo name after rename
YannikSpaeth Apr 21, 2026
43c5779
Merge pull request #2612 from YannikSpaeth/master
FlorianReimold May 12, 2026
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 2 additions & 5 deletions .github/workflows/build-macos.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,7 @@
name: Build macOS

on:
push:
pull_request:
branches:
- master
on:
workflow_dispatch:

jobs:
build-macos:
Expand Down
27 changes: 26 additions & 1 deletion .github/workflows/build-ubuntu.yml
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,17 @@ jobs:
if: ${{ always() && steps.cmake-configure.outcome == 'success' }}

- name: Run Tests
run: ctest -V --test-dir _build
run: |
mkdir -p "${{ github.workspace }}/_build/test-results"
ctest -V --test-dir _build --output-junit "${{ github.workspace }}/_build/test-results/test-results-${{ matrix.os }}.xml"

- name: Upload JUnit XML
if: always()
uses: actions/upload-artifact@v7
with:
name: 'test-results-${{ matrix.os }}'
path: ${{ github.workspace }}/_build/test-results/test-results-${{ matrix.os }}.xml
if-no-files-found: warn

- name: CPack
run: cpack -G DEB -R '${{ env.DEB_VERSION }}' --config _build/CPackConfig.cmake
Expand All @@ -194,6 +204,21 @@ jobs:
path: ${{ github.workspace }}/_build/lib/*

#---------------------------------------------------------------------------------------------------

report-ubuntu-tests:
needs: [build-ubuntu]
if: always()
permissions:
contents: read
pull-requests: write
uses: YannikSpaeth/ci-failure-summary-workflow/.github/workflows/ci-failure-summary.yml@main
with:
artifact_name: 'test-results-*'
pr_number: ${{ github.event.pull_request.number }}
sha: ${{ github.sha }}
secrets: inherit

#---------------------------------------------------------------------------------------------------

run-benchmarks:
if: github.repository == 'eclipse-ecal/ecal' && github.ref_name == 'master'
Expand Down
38 changes: 31 additions & 7 deletions .github/workflows/build-windows.yml
Original file line number Diff line number Diff line change
@@ -1,10 +1,7 @@
name: Build Windows

on:
push:
pull_request:
branches:
- master
workflow_dispatch:
env:
IS_JENKINS_SIGNING_ALLOWED: true

Expand Down Expand Up @@ -200,13 +197,25 @@ jobs:
run: cmake --build "${{ runner.workspace }}/_build/csharp" --config Release
shell: cmd

- name: Ensure JUnit output directory exists
run: mkdir "${{ runner.workspace }}\_build\test-results"
shell: cmd

- name: Run C# Tests
run: ctest -C Release -V --test-dir "${{ runner.workspace }}/_build/csharp"
run: ctest -C Release -V --test-dir "${{ runner.workspace }}/_build/csharp" --output-junit "${{ runner.workspace }}/_build/test-results/test-results-csharp.xml"

- name: Run Tests
run: ctest -C Release -V
run: ctest -C Release -V --output-junit "${{ runner.workspace }}/_build/test-results/test-results-windows.xml"
working-directory: ${{ runner.workspace }}/_build/complete

- name: Upload JUnit XML
if: always()
uses: actions/upload-artifact@v7
with:
name: test-results-windows
path: ${{ runner.workspace }}/_build/test-results/*.xml
if-no-files-found: warn

- name: Pack SDK
run: cpack -C Debug
working-directory: ${{ runner.workspace }}/_build/sdk
Expand Down Expand Up @@ -263,7 +272,22 @@ jobs:
path: ${{ runner.workspace }}/_build/complete/bin/*.dll

# --------------------------------------------------------------------------------------------------


report-windows-tests:
needs: [build-windows]
if: always()
permissions:
contents: read
pull-requests: write
uses: YannikSpaeth/ecal-ci-pipeline-improvements/.github/workflows/ci-failure-summary.yml@main
with:
artifact_name: test-results-*
pr_number: ${{ github.event.pull_request.number }}
sha: ${{ github.sha }}
secrets: inherit

# --------------------------------------------------------------------------------------------------

run-benchmarks:
if: github.repository == 'eclipse-ecal/ecal' && github.ref_name == 'master'
runs-on: windows-2022
Expand Down
2 changes: 0 additions & 2 deletions .github/workflows/build_wheels.yml
Original file line number Diff line number Diff line change
@@ -1,8 +1,6 @@
name: Build Wheels

on:
push:
pull_request:
workflow_dispatch:

jobs:
Expand Down
15 changes: 15 additions & 0 deletions ecal/tests/cpp/core_test/src/core_test.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -112,6 +112,21 @@ TEST(core_cpp_core, eCAL_Ok)
EXPECT_EQ(false, eCAL::Ok());
}

TEST(core_cpp_core, IntentionalFailure_One)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: all parameters should be named in a function [readability-named-parameter]

Suggested change
TEST(core_cpp_core, IntentionalFailure_One)
TEST(core_cpp_core /*unused*/, IntentionalFailure_One /*unused*/)

{
EXPECT_EQ(1, 2) << "Intentional CI failure #1 for summary validation";
}

TEST(core_cpp_core, IntentionalFailure_Two)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: all parameters should be named in a function [readability-named-parameter]

Suggested change
TEST(core_cpp_core, IntentionalFailure_Two)
TEST(core_cpp_core /*unused*/, IntentionalFailure_Two /*unused*/)

{
EXPECT_TRUE(false) << "Intentional CI failure #2 for summary validation";
}

TEST(core_cpp_core, IntentionalFailure_Three)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

warning: all parameters should be named in a function [readability-named-parameter]

Suggested change
TEST(core_cpp_core, IntentionalFailure_Three)
TEST(core_cpp_core /*unused*/, IntentionalFailure_Three /*unused*/)

{
EXPECT_STREQ("expected", "actual") << "Intentional CI failure #3 for summary validation";
}

/* excluded for now, system timer jitter too high */
#if 0
namespace
Expand Down
Loading