Skip to content

test(logger): remove redundant tests and parametrize duplicates in test_logger.py #830

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
Abhijeet Prasad (AbhiPrasad) merged 9 commits into from
Sep 29, 2026
Merged

test(logger): remove redundant tests and parametrize duplicates in test_logger.py #830

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
4e2adb0
test(logger): remove redundant and no-op tests
AbhiPrasad Sep 29, 2026
aabff5a
test(logger): parametrize traced generator limit and async function t…
AbhiPrasad Sep 29, 2026
d9f419c
test(logger): consolidate span export format and otel ID tests
AbhiPrasad Sep 29, 2026
1d47a89
test(logger): parametrize metadata, serialization, and masking tests
AbhiPrasad Sep 29, 2026
f1e137d
test(logger): dissolve TestLogger grab-bag and parametrize prompt tests
AbhiPrasad Sep 29, 2026
f5994e4
test(logger): parametrize init/state tests and use monkeypatch for env
AbhiPrasad Sep 29, 2026
48b04a5
test(logger): trim span name and log level overlap, tidy leftovers
AbhiPrasad Sep 29, 2026
26c5965
test(logger): consolidate emit_log template and t-string tests
AbhiPrasad Sep 29, 2026
cc1c498
test(logger): simplify consolidated tests
AbhiPrasad Sep 29, 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
5 changes: 5 additions & 0 deletions py/src/braintrust/test_gitutil.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,3 +63,8 @@ def test_repo_info_returns_none_when_git_cannot_be_resolved(monkeypatch: pytest.
assert gitutil._current_repo() is None
assert gitutil.repo_info() is None
assert list(gitutil.get_past_n_ancestors()) == []


def test_get_repo_info_without_settings_returns_none():
"""Direct call to get_repo_info with settings=None should return None."""
assert gitutil.get_repo_info(None) is None
Loading
Loading