Skip to content

Commit 855fffc

Browse files
Nathan Mauefacebook-github-bot
authored andcommitted
fix ThriftContextPropHandlerTest
Summary: This test was consistently failing when running on my devserver because it was filling in the user ID in the baggage property automatically. Updating this test to explicitly clear baggage makes it pass again Reviewed By: shuyang790 Differential Revision: D73464679 fbshipit-source-id: be0515db37624b650fc982ebf526ce5b398407af
1 parent fd9b133 commit 855fffc

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

third-party/thrift/src/thrift/lib/hack/src/__tests__/ThriftContextPropHandlerTest.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@ public function testValueNull(): void {
6565
ThriftContextPropState::get()->setRequestId(null);
6666
ThriftContextPropState::get()->setOriginId(null);
6767
ThriftContextPropState::get()->setPrivacyUniverse(null);
68+
ThriftContextPropState::get()->setBaggage(null);
6869
$v = ThriftContextPropHandler::makeV();
6970
expect($v)->toBeNull();
7071

0 commit comments

Comments
 (0)