Hi Brett,
I am new to review via the commit list. If I am doing things that I can
improve upon, don't be shy to critique. I will consider the correction a
high compliment.
@@ -160,9 +159,7 @@ public abstract class AbstractPurgeTest
> protected void goToAddDirectoryPurge()
> {
> goToGeneralPurgePage();
> - assertGeneralPurgePage();
>
Is there a reason that the assert was removed here and not in the other
method?
> - clickLinkWithXPath(
> - "//preceding::input[@value='directory' and
> @type='hidden']//following::input[@type='submit']" );
> + clickLinkWithXPath(
> "//form[@name='addDirPurgeConfig']/input[@type='submit']" );
> assertAddEditDirectoryPurgePage();
> }
>
