Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Automated test plan #338

Draft
wants to merge 5 commits into
base: master
Choose a base branch
from
Draft

Automated test plan #338

wants to merge 5 commits into from

Conversation

edipascale
Copy link
Contributor

No description provided.

@edipascale edipascale force-pushed the ema/peering-test branch 2 times, most recently from dd013f7 to 6b55803 Compare January 23, 2025 14:29
@Frostman
Copy link
Member

@edipascale could you please make a separate PR with the things refactoring in the testing.go so we can get it merged quickly and you'll not have to rebase

@edipascale edipascale force-pushed the ema/peering-test branch 5 times, most recently from beb5602 to 0cc35cd Compare January 29, 2025 11:53
Copy link

github-actions bot commented Jan 29, 2025

Test Results

0 tests   0 ✅  0s ⏱️
0 suites  0 💤
1 files    0 ❌
1 errors

For more details on these parsing errors, see this check.

Results for commit 08c5385.

♻️ This comment has been updated with latest results.

Signed-off-by: Emanuele Di Pascale <[email protected]>
add skip boolean to test functions, although right now we hardly use it

shuffle functions around so all tests are grouped together

fix a couple of small issues

Signed-off-by: Emanuele Di Pascale <[email protected]>
Signed-off-by: Emanuele Di Pascale <[email protected]>
...so that a github action can pick it up in the CI, once we add it.
we might want to avoid the conversion and directly store the test
results in a junit compatible format, but I did not want to commit to
refactoring the code until it was clear that this worked as intended

Signed-off-by: Emanuele Di Pascale <[email protected]>
Signed-off-by: Emanuele Di Pascale <[email protected]>
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