Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14142 )
Change subject: [test] move filter tests from ksck-test to ksck_remote-test ...................................................................... [test] move filter tests from ksck-test to ksck_remote-test Some filter tests in ksck-test just tested filter functions in MockKsckCluster class, but RemoteKsckCluster class which is used by CLI tool hasn't been tested. It seems unreasonable. This patch implemented same functional tests in ksck_remote-test, and removed original tests in ksck-test. Change-Id: I1139bf9b3582ac9fa45bf341ec5b97f8048c92b0 Reviewed-on: http://gerrit.cloudera.org:8080/14142 Tested-by: Kudu Jenkins Reviewed-by: Adar Dembo <[email protected]> --- M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck_remote-test.cc 2 files changed, 230 insertions(+), 171 deletions(-) Approvals: Kudu Jenkins: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/14142 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1139bf9b3582ac9fa45bf341ec5b97f8048c92b0 Gerrit-Change-Number: 14142 Gerrit-PatchSet: 5 Gerrit-Owner: Yifan Zhang <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Yifan Zhang <[email protected]>
