Closed MSevey closed 4 months ago
This update introduces several improvements and modifications across the codebase. Key changes include updates to the GitHub Actions workflow for testing, enhancements to the Makefile
for dynamic timeout settings, and various package renamings in the end-to-end tests. Additionally, functions have been refactored to use shared assertion utilities, and some test paths and environment variable handling have been adjusted.
File(s) | Summary |
---|---|
.github/workflows/knuu_testing.yml |
Added merge_group trigger, introduced a strategy block with a matrix for test packages and timeouts, and modified make test to use dynamic values. |
Makefile |
Added a timeout variable set to 120m for dynamic timeout in go test . |
e2e/basic/probe_test.go |
Updated file path in TestProbe function. |
e2e/bittwister/bittwister_test.go , e2e/bittwister/main_test.go |
Changed package declaration to bittwister and added test initialization. |
e2e/system/... |
Renamed package from basic to system in multiple test files; updated function calls to use e2e package utilities. |
pkg/builder/kaniko/utils_test.go |
Replaced testFilePerms with os.ModePerm for file permissions in TestCreateTarGz . |
In the code, a rabbit hops with glee,
Through workflows and tests, changes we see.
Packages renamed, functions refined,
Dynamic timeouts, all well-aligned.
With each commit, the system grows,
A garden of code, where innovation flows. πΈπ
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Overview
Closes #403
This PR does the following:
We could also now enable the tests that are passing as required, which I highly recommend. So that we have better protection on future changes.
Summary by CodeRabbit
New Features
Bug Fixes
Refactor
basic
tosystem
across multiple test files for better organization.e2e
package.Chores
timeout
variable in theMakefile
for dynamic test timeout configuration.