fixed bug: imported properties should have higher priority than @sources

src/main/java/org/aeonbits/owner/PropertiesLoader.java

@@ -40,10 +40,10 @@ abstract class PropertiesLoader {
     static Properties load(Class<? extends Config> clazz, Map<?, ?>... imports) {
         try {
             Properties props = defaults(clazz);
-            merge(props, reverse(imports));
             ConfigURLStreamHandler handler = new ConfigURLStreamHandler(clazz.getClassLoader(), expander);
             Properties loadedFromFile = doLoad(clazz, handler);
             merge(props, loadedFromFile);
+            merge(props, reverse(imports));
             return props;
         } catch (IOException e) {
             throw new RuntimeException(e);

src/test/java/org/aeonbits/owner/ImportConfigTest.java

@@ -46,4 +46,27 @@ public void testImportOrder() {
         assertEquals("lime", cfg.bar()); // p1 prevails, so this is lime and not grapefruit
         assertEquals("blackberry", cfg.baz());
     }
+
+    interface ImportedPropertiesHaveHigherPriority extends Config {
+        Integer minAge();
+    }
+
+    @Test
+    public void testImportedPropertiesShouldOverrideSources() {
+        ImportedPropertiesHaveHigherPriority cfg = ConfigFactory.create(ImportedPropertiesHaveHigherPriority.class);
+        assertEquals(Integer.valueOf(18), cfg.minAge());
+
+        ImportedPropertiesHaveHigherPriority cfg2 = ConfigFactory.create(ImportedPropertiesHaveHigherPriority.class,
+                new Properties() {{
+                    setProperty("minAge", "21");
+                }},
+
+                new Properties() {{
+                    setProperty("minAge", "22");
+                }}
+
+        );
+
+        assertEquals(Integer.valueOf(21), cfg2.minAge());
+    }
 }

src/test/resources/org/aeonbits/owner/ImportConfigTest$ImportedPropertiesHaveHigherPriority.properties

@@ -0,0 +1 @@
+minAge=18