Skip to content

Commit 1cba948

Browse files
Fix the UpdateIndex functional test. (#822)
Previously: add and remove index in one request, now - two requests. Resolves: OLPEDGE-1899 Signed-off-by: Mykhailo Kuchma <[email protected]>
1 parent 001ba13 commit 1cba948

File tree

1 file changed

+31
-15
lines changed

1 file changed

+31
-15
lines changed

tests/functional/olp-cpp-sdk-dataservice-write/DataserviceWriteIndexLayerClientTest.cpp

Lines changed: 31 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -209,21 +209,37 @@ TEST_F(DataserviceWriteIndexLayerClientTest, PublishDataAsync) {
209209
}
210210

211211
TEST_F(DataserviceWriteIndexLayerClientTest, UpdateIndex) {
212-
Index index = GetTestIndex();
213-
index.SetId("2f269191-5ef7-42a4-a445-fdfe53f95d92");
214-
215-
auto response =
216-
client_
217-
->UpdateIndex(
218-
UpdateIndexRequest()
219-
.WithIndexAdditions({index})
220-
.WithIndexRemovals({"2f269191-5ef7-42a4-a445-fdfe53f95d92"})
221-
.WithLayerId(GetTestLayer()))
222-
.GetFuture()
223-
.get();
224-
225-
EXPECT_SUCCESS(response);
226-
EXPECT_EQ("", response.GetError().GetMessage());
212+
const auto id = "2f269191-5ef7-42a4-a445-fdfe53f95d92";
213+
214+
{
215+
SCOPED_TRACE("Add index");
216+
Index index = GetTestIndex();
217+
index.SetId(id);
218+
219+
auto response_addition =
220+
client_
221+
->UpdateIndex(
222+
UpdateIndexRequest().WithIndexAdditions({index}).WithLayerId(
223+
GetTestLayer()))
224+
.GetFuture()
225+
.get();
226+
227+
EXPECT_SUCCESS(response_addition);
228+
EXPECT_EQ("", response_addition.GetError().GetMessage());
229+
}
230+
{
231+
SCOPED_TRACE("Remove index");
232+
auto response_removal =
233+
client_
234+
->UpdateIndex(
235+
UpdateIndexRequest().WithIndexRemovals({id}).WithLayerId(
236+
GetTestLayer()))
237+
.GetFuture()
238+
.get();
239+
240+
EXPECT_SUCCESS(response_removal);
241+
EXPECT_EQ("", response_removal.GetError().GetMessage());
242+
}
227243
}
228244

229245
TEST_F(DataserviceWriteIndexLayerClientTest, PublishNoData) {

0 commit comments

Comments
 (0)