fix tests assuming index of default build configs for comparision, now iterate the list to ensure a compatible config is found
authortimkelly
Fri, 27 Aug 2010 11:41:08 -0500
changeset 1903 8aca9dfa10ce
parent 1902 0dee9df9d770
child 1904 e1859c9e4688
fix tests assuming index of default build configs for comparision, now iterate the list to ensure a compatible config is found
builder/com.nokia.carbide.cdt.builder.test/src/com/nokia/carbide/cdt/builder/test/ProjectPropertiesTest.java
--- a/builder/com.nokia.carbide.cdt.builder.test/src/com/nokia/carbide/cdt/builder/test/ProjectPropertiesTest.java	Fri Aug 27 09:56:34 2010 -0500
+++ b/builder/com.nokia.carbide.cdt.builder.test/src/com/nokia/carbide/cdt/builder/test/ProjectPropertiesTest.java	Fri Aug 27 11:41:08 2010 -0500
@@ -188,7 +188,17 @@
 		assertEquals(3, cpi.getBuildConfigurations().size());
 		
 		ICarbideBuildConfiguration defConfig = cpi.getDefaultConfiguration();
-		assertEquals(stockBuildConfigs.get(1).getDisplayString(), defConfig.getDisplayString());
+		
+		boolean foundConfigFromStock = false;
+		for (ISymbianBuildContext context : stockBuildConfigs){
+			if (defConfig.getDisplayString().equals(context.getDisplayString())){
+				foundConfigFromStock = true;
+				break;
+			}
+		}
+		
+		assertTrue(foundConfigFromStock);
+		
 	}
 	
 	// Test the reading and writing of the default configuration
@@ -197,10 +207,15 @@
 		assertEquals(3, cpm.getBuildConfigurations().size());
 		
 		ICarbideBuildConfiguration defConfig = cpm.getDefaultConfiguration();
-		assertEquals(stockBuildConfigs.get(1).getDisplayString(), defConfig.getDisplayString());
 		
 		// make another configuration the default one...
-		ICarbideBuildConfiguration newDefaultConfig = cpm.getNamedConfiguration(stockBuildConfigs.get(2).getDisplayString());
+		ICarbideBuildConfiguration newDefaultConfig = null;
+		for (ISymbianBuildContext context : stockBuildConfigs){
+			if (context.getTargetString().equalsIgnoreCase("udeb") && context.getPlatformString().equalsIgnoreCase("armv5")){
+				newDefaultConfig = cpm.getNamedConfiguration(context.getDisplayString());
+				break;
+			}
+		}
 		assertNotNull(newDefaultConfig);
 		cpm.setDefaultConfiguration(newDefaultConfig);
 		cpm.saveChanges();
@@ -210,7 +225,17 @@
 
 		// Check to see that we got the new default config
 		defConfig = cpi.getDefaultConfiguration();
-		assertEquals(stockBuildConfigs.get(2).getDisplayString(), defConfig.getDisplayString());
+
+		boolean foundConfigFromStock = false;
+		for (ISymbianBuildContext context : stockBuildConfigs){
+			if (defConfig.getDisplayString().equals(context.getDisplayString())){
+				foundConfigFromStock = true;
+				break;
+			}
+		}
+		
+		assertTrue(foundConfigFromStock);
+		
 	}
 	
 	public void testWritePKGData(){