5 Commits

Author	SHA1	Message	Date
Damien Mathieu	5331939a74	Introduce logtest.AssertRecordEqual (#5499) ... This is a follow-up to the comments in https://github.com/open-telemetry/opentelemetry-go/pull/5468#discussion_r1624173196, introducing `logtest.AssertRecordEqual` so bridges can compare log records more easily. --------- Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>	2024-06-12 11:35:11 -07:00
Damien Mathieu	a2505aa68c	log/logtest: provide record with their context (#5468) ... So the instrumentation tests can check the provided context. See https://github.com/open-telemetry/opentelemetry-go-contrib/pull/5707 --------- Co-authored-by: Robert Pająk <pellared@hotmail.com> Co-authored-by: Sam Xie <sam@samxie.me>	2024-06-06 08:28:28 -07:00
Damien Mathieu	ebd0adee35	Split log/logtest into a recorder and a logger (#5365) ... The current logtest.Recorder implementation is wrong. We have a single `Recorder`, which acts as both a `LoggerProvider`, and a `Logger`, making it possible to emit a log entry with the root recorder, which shouldn't be possible with the API. This change introduces a new private struct, `logger` that acts as the recording logger, while `Recorder` becomes only a LoggerProvider and not a Logger anymore. Closes #5357. --------- Co-authored-by: Robert Pająk <pellared@hotmail.com>	2024-05-17 07:28:40 -07:00
Damien Mathieu	f33d40886d	log/logtest: add Record Factory (#5263)	2024-04-25 07:22:39 +02:00
Damien Mathieu	8d1d62b905	logtest: Add Recorder (#5134) ... * introduce in-memory log exporter * add changelog entry * move logtest into a recorder within the api * rename GetRecords to Result * rename InMemoryRecorder to Recorder * name the struct r * ensure Logger creates a struct copy * replace severity with enabledFn * Update CHANGELOG.md Co-authored-by: Robert Pająk <pellared@hotmail.com> * kUpdate log/logtest/config.go Co-authored-by: Robert Pająk <pellared@hotmail.com> * store all scope records, so we can retrieve everything with `Result()` * store child loggers instead of all scope records * no need to explicitly create a new slice * add concurrent safe test * handle default enabled function if the struct was manually created * rename WithEnabledFn to WithEnabledFunc * test result/reset with child loggers * add enabled to concurrent safe * fix lint missing period * rename defaultEnabledFn to defaultEnabledFunc * merge recorder.go and config.go * Update log/logtest/recorder_test.go Co-authored-by: Robert Pająk <pellared@hotmail.com> * create empty recorder in concurrent safe test * Update log/logtest/recorder_test.go Co-authored-by: Robert Pająk <pellared@hotmail.com> * fix lint * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * Update log/logtest/recorder.go Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> * make enabledFunc callable from outside the package * replace expected with want --------- Co-authored-by: Robert Pająk <pellared@hotmail.com> Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com> Co-authored-by: Sam Xie <sam@samxie.me>	2024-04-09 06:45:25 -07:00