Minor fix + more verbose assert in test to debug issue.
Xavier Ducrohet [Thu, 7 Feb 2013 00:23:43 +0000 (16:23 -0800)]
Change-Id: If201822e568f670f41866f490651b7060a8416e1

builder/src/main/java/com/android/builder/DefaultSdkParser.java
builder/src/test/java/com/android/builder/resources/BaseTestCase.java
builder/src/test/java/com/android/builder/resources/ResourceMergerTest.java

index bb07854..ac3041f 100644 (file)
@@ -23,12 +23,14 @@ import com.android.sdklib.SdkManager;
 import com.android.sdklib.internal.repository.packages.FullRevision;
 import com.android.sdklib.repository.PkgProps;
 import com.android.utils.ILogger;
+import com.google.common.base.Charsets;
 import com.google.common.io.Closeables;
 
 import java.io.File;
+import java.io.FileInputStream;
 import java.io.FileNotFoundException;
-import java.io.FileReader;
 import java.io.IOException;
+import java.io.InputStreamReader;
 import java.io.Reader;
 import java.util.Properties;
 
@@ -82,7 +84,9 @@ public class DefaultSdkParser implements SdkParser {
 
         Reader reader = null;
         try {
-            reader = new FileReader(new File(platformTools, FN_SOURCE_PROP));
+            reader = new InputStreamReader(
+                    new FileInputStream(new File(platformTools, FN_SOURCE_PROP)),
+                    Charsets.UTF_8);
             Properties props = new Properties();
             props.load(reader);
 
index c82c587..3d1b68a 100644 (file)
@@ -122,11 +122,11 @@ public abstract class BaseTestCase extends TestCase {
             DataMerger<? extends DataItem, ? extends DataFile, ? extends DataSet> dataMerger) {
 
         // Loop on all the data sets.
-        for (DataSet set : dataMerger.getDataSets()) {
+        for (DataSet<? extends DataItem, ? extends DataFile> set : dataMerger.getDataSets()) {
             // get the source files and verify they exists.
             List<File> files = set.getSourceFiles();
             for (File file : files) {
-                assertTrue(file.isDirectory());
+                assertTrue("Not a folder: " + file.getAbsolutePath(), file.isDirectory());
             }
 
             // for each source file, also check that the files inside are in fact inside
@@ -138,7 +138,8 @@ public abstract class BaseTestCase extends TestCase {
                 DataFile dataFile = item.getSource();
                 File file = dataFile.getFile();
 
-                assertNotNull(set.findMatchingSourceFile(file));
+                assertNotNull("Not in source file: " + file.getAbsolutePath(),
+                        set.findMatchingSourceFile(file));
             }
         }
     }
index 844506b..0e8c07c 100755 (executable)
@@ -403,9 +403,6 @@ public class ResourceMergerTest extends BaseTestCase {
 
         // ----------------
         // first set is the main one, no change here
-        ResourceSet mainSet = sets.get(0);
-        File mainBase = new File(root, "main");
-        File mainValues = new File(mainBase, "values");
 
         // ----------------
         // second set is the overlay one