continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jevica Arianne B. Zurbano" <jevica.aria...@gmail.com>
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 05:44:11 GMT


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


Mime
View raw message