Skip to content
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

Add test based on the Jaeger spec #645

Merged
merged 5 commits into from
Mar 31, 2022
Merged

Add test based on the Jaeger spec #645

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
f9f66c1
Add test based on the spec
Grunet Mar 31, 2022
5db1912
Fix the other tests which appear flaky
Grunet Mar 31, 2022
d819a2d
Clarify the randomness in these tests
Grunet Mar 31, 2022
a8895e2
Add concrete example from Go for the trace id splitting
Grunet Mar 31, 2022
9088731
Fix style
Grunet Mar 31, 2022
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
8 changes: 8 additions & 0 deletions tests/Unit/Contrib/IdConverterTest.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,14 @@
*/
class IdConverterTest extends TestCase
{
//Based on this section of the Jaeger spec https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/trace/sdk_exporters/jaeger.md#ids
public function test_correctly_converts_example_from_spec()
{
$hex = 'FF00000000000000';

$this->assertEquals(-72057594037927936, IdConverter::convertOtelToJaegerSpanId($hex));
brettmc marked this conversation as resolved.
Show resolved Hide resolved
}

public function test_correctly_converted_span_id()
{
// 16 char hex string
Expand Down