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: r930283 - in /continuum/trunk: continuum-commons/src/main/java/org/apache/continuum/profile/ continuum-commons/src/test/java/org/apache/continuum/profile/ continuum-webapp-test/src/test/testng/org/apache/continuum/web/test/
Date Tue, 06 Apr 2010 06:09:38 GMT
probably best to avoid confusion...

On 06/04/2010, at 3:44 PM, Jevica Arianne B. Zurbano wrote:

> 
> 
> Brett Porter wrote:
>> On 03/04/2010, at 1:41 AM, jzurbano@apache.org wrote:
>> 
>>  
>>> +        String BUILD_ENV_NAME = getProperty( "BUIL_ENV_NAME" );
>>>    
>> 
>> is this a typo in the properties file that needs to be fixed, or here? (see also
below).
>>  
> 
> I used the existing property in the properties file. Should I change it and update all
the tests that use it?
> 
>>  
>>> +        String newName = "NEW_BUILD_ENV";
>>> +        goToAddBuildEnvironment();
>>> +        addBuildEnvironment( newName, new String[] {}, true );
>>> +        goToEditBuildEnvironment( newName );
>>> +        editBuildEnvironment( BUILD_ENV_NAME, new String[] {}, false );
>>> +        assertTextPresent( "A Build Environment with the same name already exists"
);
>>> +    }
>>> +    +    @Test( dependsOnMethods = { "testEditDuplicatedBuildEnvironmentParallelBuilds"
} )
>>> +    public void testEditDuplicatedBuildEnvironmentDistributedBuilds()
>>> +    {
>>> +    try
>>> +        {
>>> +            enableDistributedBuilds();
>>> +
>>> +            String BUILD_ENV_NAME = getProperty( "BUIL_ENV_NAME" );
>>> +            String BUILD_AGENT_GROUPNAME = getProperty( "BUILD_AGENT_GROUPNAME"
);
>>> +            String newName = "NEW_BUILD_ENV";
>>> +            goToEditBuildEnvironment( newName );
>>> +            editBuildEnvironmentWithBuildAgentGroup( BUILD_ENV_NAME, new String[]
{}, BUILD_AGENT_GROUPNAME, false );
>>> +            assertTextPresent( "A Build Environment with the same name already
exists" );
>>> +        }
>>> +        finally
>>> +        {
>>> +            disableDistributedBuilds();
>>> +        }
>>> +    }
>>> }
>>> 
>>> 
>>>    
>> 
>> --
>> Brett Porter
>> brett@apache.org
>> http://brettporter.wordpress.com/
>> 
>> 
>> 
>> 
>> 
>>  
> 
> -- 
> 
> Thanks,
> 
> Jev
> 

--
Brett Porter
brett@apache.org
http://brettporter.wordpress.com/





Mime
View raw message