This is why we now have tests, right?

It's really nice to have tests to catch this stuff now.
open-telemetry · Dec 7, 2020 · f312ad0 · f312ad0
1 parent bc08844
commit f312ad0
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/instrumentation/opentelemetry-instrumentation-grpc/tests/test_server_interceptor.py b/instrumentation/opentelemetry-instrumentation-grpc/tests/test_server_interceptor.py
@@ -385,7 +385,12 @@ def handler(request, context):
 
         # make sure this span errored, with the right status and detail
         self.assertEqual(span.status.status_code, StatusCode.ERROR)
-        self.assertEqual(span.status.description, failure_message)
+        self.assertEqual(
+            span.status.description,
+            "{}:{}".format(
+                grpc.StatusCode.FAILED_PRECONDITION, failure_message
+            ),
+        )
 
         # Check attributes
         self.assert_span_has_attributes(