----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15175/#review28089 -----------------------------------------------------------
Also your patch failed to apply for me on latest master: ~/workspace/cloudstack/incubator-cloudstack(branch:master) ยป git am patch/element.patch Applying: Added a verifyElementInList utility function /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:27: trailing whitespace. @Description: /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:33: trailing whitespace. at a given pos /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:34: trailing whitespace. @Input: /Users/tsp/workspace/cloudstack/incubator-cloudstack/.git/rebase-apply/patch:36: trailing whitespace. II : Element to verify whether it exists in the list error: patch failed: tools/marvin/marvin/codes.py:39 error: tools/marvin/marvin/codes.py: patch does not apply error: patch failed: tools/marvin/marvin/integration/lib/utils.py:353 error: tools/marvin/marvin/integration/lib/utils.py: patch does not apply Patch failed at 0001 Added a verifyElementInList utility function When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort". - Prasanna Santhanam On Nov. 1, 2013, 7:44 a.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15175/ > ----------------------------------------------------------- > > (Updated Nov. 1, 2013, 7:44 a.m.) > > > Review request for cloudstack, Girish Shilamkar and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > The purpose is to verify a given element in list at a given position with few > error checks like list type,empty list and position > Returns appropriate codes based upon the inputs. Can be used under tests > instead of multiple asserts > > Earlier, there was a list validity function, this is on top of that. > > Need to go to both master and 4.2 > > > Diffs > ----- > > tools/marvin/marvin/codes.py bd01ad3 > tools/marvin/marvin/integration/lib/utils.py b6e38ec > > Diff: https://reviews.apache.org/r/15175/diff/ > > > Testing > ------- > > Tested. > > >>> from utils import verifyElementInList > >>> a=[1,2,3] > >>> verifyElementInList(a,2) > [0, 'ELEMENT NOT FOUND IN THE INPUT'] > >>> verifyElementInList(a,1) > [1, None] > >>> verifyElementInList(a,3) > [0, 'ELEMENT NOT FOUND IN THE INPUT'] > > > Thanks, > > Santhosh Edukulla > >