Skip to content

test: run all tests on a single fact instance #1330

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

Draft
Molter73 wants to merge 6 commits into
base: mauro/fix/cleanup-lpm
Choose a base branch
from
Draft

test: run all tests on a single fact instance #1330

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
b8ae0e5
test: speed up integration tests with session-scoped fact container
Molter73 Jul 29, 2026
d87800c
test: add MetricsSnapshot for delta-based metric assertions
Molter73 Jul 29, 2026
91c63d2
test: decouple session-scoped fixtures from EventServer
Molter73 Jul 30, 2026
9afbb7a
chore: better log time slice for tests
Molter73 Jul 30, 2026
c04763e
fix: prevent reconnections between tests and start needed servers before
Molter73 Jul 31, 2026
4d03c81
fix: flush config file
Molter73 Jul 31, 2026
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
2 changes: 1 addition & 1 deletion fact/src/config/reloader/mod.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ impl Reloader {
let path = PathBuf::from(file);
if path.exists() {
let mtime = match path.metadata() {
Ok(m) => m.mtime(),
Ok(m) => m.mtime_nsec(),
Err(e) => {
warn!("Failed to stat {file}: {e}");
warn!("Configuration reloading may not work");
Expand Down
Loading
Loading