continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@apache.org>
Subject Re: svn commit: r1542934 - /continuum/branches/CONTINUUM-2713/continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/parent/AbstractPurgeTest.java
Date Tue, 03 Dec 2013 06:30:47 GMT


Brent Atkinson wrote:
> 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.

Thanks for taking a look!

>
>
> @@ -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?

Seems I missed that - I'll remove them now.

Cheers,
Brett


Mime
View raw message