----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6614/#review10863 -----------------------------------------------------------
I might be in the wrong here but none of the test methods seem to be verifying the operation worked or not. - Alex Huang On Aug. 14, 2012, 9:27 p.m., Krishna Sadasivam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/6614/ > ----------------------------------------------------------- > > (Updated Aug. 14, 2012, 9:27 p.m.) > > > Review request for cloudstack. > > > Description > ------- > > Patch to use Base64 class from org.apache.commons.codec.binary.Base64 > > > Diffs > ----- > > > plugins/network-elements/brocade/test/com/cloud/network/element/BrocadePluginTest.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/6614/diff/ > > > Testing > ------- > > Unit test > > > Thanks, > > Krishna Sadasivam > >
