-
Notifications
You must be signed in to change notification settings - Fork 17
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
tests: make integration testing faster #726
Conversation
982805a
to
acf5f8b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! Thank you for the patchset.
Generally I'm OK.
Please add to the commit body the reason and what profit we have.
Am I correct in assuming that further ticket work is planned or more ideas to accelerate testing? |
Separated integration tests from linter, unit, e2e and examples checks, because integration tests run longer then all other. Result: 15% faster testing. Part of #690
58ac602
to
f53d0c9
Compare
magefile.go
Outdated
@@ -161,9 +161,19 @@ func Unit() error { | |||
// Run integration tests | |||
func Integration() error { | |||
mg.Deps(GenerateGoCode) | |||
|
|||
build_projects_path := os.TempDir() + "/cartridge_cli_test_temp" | |||
err := os.Mkdir(build_projects_path, 0777) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
0777 - looks like overkill (write description in comments).
How about this question? |
a29d950
to
750a49b
Compare
Added pre-building of project_with_cartridge and default_project before integration tests. Result: ~25-50% faster testing. Closes #390
750a49b
to
714d564
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Separated integration tests from linter, unit, e2e and examples checks.
Result: ~15% faster testing.
Added pre-building of slow projects.
Total result: ~40-65% faster.
Closes #690