Vincent Massol wrote:
> On Apr 16, 2009, at 12:56 PM, sdumitriu (SVN) wrote:
> 
>> Author: sdumitriu
>> Date: 2009-04-16 12:56:05 +0200 (Thu, 16 Apr 2009)
>> New Revision: 18727
>>
>> Added:
>>   enterprise/trunk/distribution-test/selenium-tests/src/test/it/com/ 
>> xpn/xwiki/it/selenium/SimpleAdvancedTest.java
>> Modified:
>>   enterprise/trunk/distribution-test/selenium-tests/src/test/it/com/ 
>> xpn/xwiki/it/selenium/AllTests.java
>> Log:
>> XAADMINISTRATION-35: Administrative users get their own setting on  
>> usertype, and not the setting for the user whose profile they're  
>> viewing
>> Added test.
>> Patch submitted by Arun Reddy Nelakurthi, applied with codestyle  
>> changes
>>
>>
>> Modified: enterprise/trunk/distribution-test/selenium-tests/src/test/ 
>> it/com/xpn/xwiki/it/selenium/AllTests.java
>> ===================================================================
>> --- enterprise/trunk/distribution-test/selenium-tests/src/test/it/ 
>> com/xpn/xwiki/it/selenium/AllTests.java      2009-04-16 10:54:34 UTC (rev  
>> 18726)
>> +++ enterprise/trunk/distribution-test/selenium-tests/src/test/it/ 
>> com/xpn/xwiki/it/selenium/AllTests.java      2009-04-16 10:56:05 UTC (rev  
>> 18727)
>> @@ -79,6 +79,7 @@
>>         addTestCase(suite, DocExtraTest.class);
>>         addTestCase(suite, PanelsTest.class);
>>         addTestCase(suite, SectionTest.class);
>> +        addTestCase(suite, SimpleAdvancedTest.class);
> 
> Shouldn't this go in the existing UserProfileTest.class instead?
> 
> There seems to be some duplication too to create a user.

Yes, it should.

-- 
Sergiu Dumitriu
http://purl.org/net/sergiu/
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to