Skip to content

Commit

Permalink
Update httpx implementation
Browse files Browse the repository at this point in the history
  • Loading branch information
macieyng committed Apr 19, 2023
1 parent d0b37c4 commit 836c2c7
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,7 @@ class ResponseInfo(typing.NamedTuple):


def _get_default_span_name(method: str) -> str:
return f"HTTP {method.strip()}"
return method.strip()


def _apply_status_code(span: Span, status_code: int) -> None:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -142,7 +142,7 @@ def test_basic(self):
span = self.assert_span()

self.assertIs(span.kind, trace.SpanKind.CLIENT)
self.assertEqual(span.name, "HTTP GET")
self.assertEqual(span.name, "GET")

self.assertEqual(
span.attributes,
Expand Down Expand Up @@ -258,7 +258,7 @@ def test_invalid_url(self):

span = self.assert_span()

self.assertEqual(span.name, "HTTP POST")
self.assertEqual(span.name, "POST")
self.assertEqual(
span.attributes[SpanAttributes.HTTP_METHOD], "POST"
)
Expand Down Expand Up @@ -350,7 +350,7 @@ def test_request_hook_no_span_change(self):

self.assertEqual(result.text, "Hello!")
span = self.assert_span()
self.assertEqual(span.name, "HTTP GET")
self.assertEqual(span.name, "GET")

def test_not_recording(self):
with mock.patch("opentelemetry.trace.INVALID_SPAN") as mock_span:
Expand Down Expand Up @@ -444,7 +444,7 @@ def test_request_hook_no_span_update(self):

self.assertEqual(result.text, "Hello!")
span = self.assert_span()
self.assertEqual(span.name, "HTTP GET")
self.assertEqual(span.name, "GET")
HTTPXClientInstrumentor().uninstrument()

def test_not_recording(self):
Expand Down

0 comments on commit 836c2c7

Please sign in to comment.