Khazar Mammadli has posted comments on this change. ( http://gerrit.cloudera.org:8080/17577 )
Change subject: [rest] add integration test ...................................................................... Patch Set 44: (1 comment) http://gerrit.cloudera.org:8080/#/c/17577/44/src/kudu/integration-tests/rest_server-itest.cc File src/kudu/integration-tests/rest_server-itest.cc: http://gerrit.cloudera.org:8080/#/c/17577/44/src/kudu/integration-tests/rest_server-itest.cc@192 PS44, Line 192: ASSERT_EQ("message: Table was not deleted",fail_response_json[2]); Parser that've written isn't perfect, it splits the strings by the comma and in this case it doesn't necessarily put all of the response into one vector element but rather splits them up. Still this I think is a good enough indication that the test has been completed without issues (The exact line should be :Table was not deleted, either it doesn't exists or client error) -- To view, visit http://gerrit.cloudera.org:8080/17577 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic1b2900c67d601fa5a3a747d2475e1e86a6ad2e3 Gerrit-Change-Number: 17577 Gerrit-PatchSet: 44 Gerrit-Owner: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Wed, 27 Apr 2022 15:18:20 +0000 Gerrit-HasComments: Yes
