Skip to content

Conversation

@onetechnical
Copy link
Contributor

@onetechnical onetechnical commented May 29, 2025

Summary

This is a proof of concept of shifting unit and integration testing into GitHub Actions. The nightly full integration suite and the shorter PR workflow have been implemented.

These should not be marked as "Required" in GitHub yet - we should run these in parallel with existing tests to see how they compare. We should additionally see if we can reduce the timelines for the PR tests, as they are currently running for longer than half an hour.

In the nightly test I have commented out the upload section for now, as those are covered by existing workflows. If we deprecate those, we should uncomment this. We may need to add some additional secrets in that case, though.

Test Plan

These are tests, and were executed on a fork.

@codecov
Copy link

codecov bot commented May 29, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 50.42%. Comparing base (5ea77f2) to head (ee1afba).
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6346      +/-   ##
==========================================
- Coverage   51.90%   50.42%   -1.49%     
==========================================
  Files         653      653              
  Lines       87469   111205   +23736     
==========================================
+ Hits        45401    56070   +10669     
- Misses      39198    52272   +13074     
+ Partials     2870     2863       -7     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@onetechnical onetechnical requested a review from Copilot May 29, 2025 12:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

@onetechnical onetechnical merged commit e4aa2a8 into algorand:master May 30, 2025
48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants