Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/17041 )

Change subject: [test] Fix PasswdWebserverTest.TestCrashInFIPSMode test
......................................................................

[test] Fix PasswdWebserverTest.TestCrashInFIPSMode test

The order of parameters to the ASSERT_STR_CONTAINS was reversed
causing the test to fail as seen below. We don’t see this failure in
standard CI test runs given none of them run in FIPS mode.

Example Failure:
   
/data0/somelongdirectorytoavoidrpathissues/src/kudu/src/kudu/server/webserver-test.cc:181:
 Failure
   Value of: "Digest authentication in FIPS approved mode"
   Expected: has substring "Illegal state: Webserver cannot be started with 
Digest authentication in FIPS approved mode"
   Actual: "Digest authentication in FIPS approved mode" (of type char [44])

Change-Id: I6bc32cd115bbf1d2b0aa00cc9b9fd80de5433c14
Reviewed-on: http://gerrit.cloudera.org:8080/17041
Tested-by: Kudu Jenkins
Reviewed-by: Bankim Bhavsar <[email protected]>
Reviewed-by: Alexey Serbin <[email protected]>
---
M src/kudu/server/webserver-test.cc
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Kudu Jenkins: Verified
  Bankim Bhavsar: Looks good to me, approved
  Alexey Serbin: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/17041
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I6bc32cd115bbf1d2b0aa00cc9b9fd80de5433c14
Gerrit-Change-Number: 17041
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to