Skip to content

DX-377 Refactor Flakeguard: Separate runner, executor, and parser concerns #1776

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.

Sign up for GitHub

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

Merged
lukaszcl merged 16 commits into from
Apr 16, 2025
Merged

DX-377 Refactor Flakeguard: Separate runner, executor, and parser concerns #1776

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
61c3e9a
wip refactoring
lukaszcl Apr 11, 2025
ddd1064
fix
lukaszcl Apr 11, 2025
2768edb
Enhance command execution in Flakeguard: capture stdout and stderr se…
lukaszcl Apr 11, 2025
9b91339
Use configs
lukaszcl Apr 11, 2025
ba16dd0
Refactor panic and race attribution functions in Flakeguard: update r…
lukaszcl Apr 11, 2025
074843c
Add more tests
lukaszcl Apr 11, 2025
e0487f8
Add integration tests for the runner package in Flakeguard
lukaszcl Apr 11, 2025
0f64baf
fix go test
lukaszcl Apr 11, 2025
e3eff35
simplify makefile
lukaszcl Apr 11, 2025
1e66985
Add unit tests for the runner package in Flakeguard: implement mock e…
lukaszcl Apr 11, 2025
7b09bb6
more refactoring
lukaszcl Apr 11, 2025
30df99c
refactor cmd/run.go
lukaszcl Apr 11, 2025
8ab4790
Handle command-line-arguments when go test <file> is used
lukaszcl Apr 14, 2025
0f2667f
cleanup comments
lukaszcl Apr 14, 2025
df64008
Add unit test for parser to verify handling of parent failures in sub…
lukaszcl Apr 14, 2025
82f0d55
Merge branch 'main' into DX-377-refactor-flakeguard
lukaszcl Apr 16, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions tools/flakeguard/Makefile
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,12 +1,12 @@
.PHONY: test_unit
test_unit:
go list ./... | grep -v 'example_test_package' | xargs go test -timeout 5m -json -cover -covermode=count -coverprofile=unit-test-coverage.out 2>&1 | tee /tmp/gotest.log | gotestloghelper -ci
go test ./... -timeout 5m -json -cover -covermode=count -coverprofile=unit-test-coverage.out 2>&1 | tee /tmp/gotest.log | gotestloghelper -ci

.PHONY: test
test:
go install github.com/gotesttools/gotestfmt/v2/cmd/gotestfmt@latest
set -euo pipefail
go list ./... | grep -v 'example_test_package' | xargs go test -json -cover -coverprofile unit-test-coverage.out -v 2>&1 | tee /tmp/gotest.log | gotestfmt
go test ./... -json -cover -coverprofile unit-test-coverage.out -v 2>&1 | tee /tmp/gotest.log | gotestfmt

.PHONY: test-package
test-package:
Expand All @@ -16,7 +16,7 @@ test-package:

.PHONY: test-race
test-race:
go list ./... | grep -v 'example_test_package' | xargs go test -count=1 -race
go test ./... -count=1 -race

.PHONY: bench
bench:
Expand Down
Loading
Loading