Skip to content

Run as many tests as possible in CI - #1127

Open
papeh wants to merge 1 commit into
mainfrom
feature/more-ci-tests
Open

Run as many tests as possible in CI#1127
papeh wants to merge 1 commit into
mainfrom
feature/more-ci-tests

Conversation

@papeh

@papeh papeh commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

to catch test failures before merging PRs. Previously, these tests had been saved for CD builds to save time, but those failures are too easy to ignore.

https://github.com/sillsdev/FieldWorks/actions/workflows/CI.yml completed in 14:09 for this PR, which is the third-shortest successful time in the last page runs (range 12:33-16:14), so we are saving next to no time by excluding "long-running" tests from CI builds.


This change is Reviewable

to catch test failures before merging PRs. Previously, these tests had been
saved for CD builds to save time, but those failures are too easy to ignore.
@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown

NUnit Tests

    1 files  ± 0      1 suites  ±0   12m 11s ⏱️ + 2m 3s
6 049 tests +25  5 964 ✅ +21  85 💤 +4  0 ❌ ±0 
6 058 runs  +25  5 973 ✅ +21  85 💤 +4  0 ❌ ±0 

Results for commit c797fab. ± Comparison against base commit 67c88a7.

@codecov-commenter

codecov-commenter commented Sep 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 38.62%. Comparing base (193b2a9) to head (c797fab).
⚠️ Report is 14 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1127      +/-   ##
==========================================
+ Coverage   38.44%   38.62%   +0.18%     
==========================================
  Files        1507     1514       +7     
  Lines      350698   351047     +349     
  Branches    40314    40360      +46     
==========================================
+ Hits       134813   135603     +790     
+ Misses     186652   186246     -406     
+ Partials    29233    29198      -35     

see 63 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@papeh
papeh marked this pull request as ready for review September 8, 2026 19:45
@papeh
papeh enabled auto-merge (squash) September 8, 2026 19:59
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.

2 participants