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, [email protected] 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 >> [email protected] >> http://brettporter.wordpress.com/ >> >> >> >> >> >> > > -- > > Thanks, > > Jev > -- Brett Porter [email protected] http://brettporter.wordpress.com/
