Skip to content

Commit

Permalink
Remove SetPatch::Put from conformance test
Browse files Browse the repository at this point in the history
Summary: SetPatch::Put is deprecated operation that isn't used in production.

Differential Revision: D68188105

fbshipit-source-id: c6f082a29374fba41fbbf813a13b0f8e70a31d3d
  • Loading branch information
TJ Yin authored and facebook-github-bot committed Jan 16, 2025
1 parent 82eb816 commit 47a553a
Showing 1 changed file with 8 additions and 8 deletions.
16 changes: 8 additions & 8 deletions thrift/conformance/data/PatchGenerator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -520,15 +520,15 @@ Test createListSetPatchTest(
prependSetCase.name() = makeTestName(value, "insert");
prependSetCase.test().emplace().objectPatch_ref() =
makeContainerInsertTC<ContainerTag, TT>(registry, protocol, value);
} else {
auto patchValue = value.value;
op::clear<TT>(patchValue);
auto& appendCase = test.testCases()->emplace_back();
appendCase.name() = makeTestName(value, "append");
appendCase.test().emplace().objectPatch_ref() =
makeContainerAppendTC<ContainerTag>(
registry, protocol, value.value, patchValue);
}

auto patchValue = value.value;
op::clear<TT>(patchValue);
auto& appendCase = test.testCases()->emplace_back();
appendCase.name() = makeTestName(value, "append");
appendCase.test().emplace().objectPatch_ref() =
makeContainerAppendTC<ContainerTag>(
registry, protocol, value.value, patchValue);
}
});

Expand Down

0 comments on commit 47a553a

Please sign in to comment.