Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Add more tests for serialization of field syntax #350

Merged
merged 1 commit into from
Jan 20, 2022

Conversation

djaglowski
Copy link
Member

Field syntax can be used to refer directly to the body of a log,
but the serialization of this case was not well tested. This
change adds additional tests to cover valid cases.

Field syntax can be used to refer directly to the body of a log,
but the serialization of this case was not well tested. This
change adds additional tests to cover valid cases.
@codecov
Copy link

codecov bot commented Jan 20, 2022

Codecov Report

Merging #350 (235d7f9) into main (d1c9d7a) will decrease coverage by 0.0%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            main    #350     +/-   ##
=======================================
- Coverage   77.2%   77.1%   -0.1%     
=======================================
  Files         94      94             
  Lines       4471    4471             
=======================================
- Hits        3452    3451      -1     
- Misses       699     701      +2     
+ Partials     320     319      -1     
Impacted Files Coverage Δ
...perator/builtin/transformer/recombine/recombine.go 74.3% <0.0%> (-2.1%) ⬇️
operator/builtin/input/tcp/tcp.go 79.6% <0.0%> (+1.6%) ⬆️

@djaglowski djaglowski marked this pull request as ready for review January 20, 2022 20:08
@djaglowski djaglowski requested a review from a team January 20, 2022 20:08
@djaglowski djaglowski merged commit 39eda63 into open-telemetry:main Jan 20, 2022
@djaglowski djaglowski deleted the test-field-syntax-body branch January 20, 2022 20:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants