🚑 Fix odoo core addon taking priority over extra version

Pull #84 introduced a regression that violated [the addon priority table](https://github.com/Tecnativa/docker-odoo-base#optodoocustomsrcaddonsyaml). This fixes the regression and tests it.

.travis.yml

@@ -19,13 +19,11 @@ env:
   global:
     # Variables found by default in Docker Hub builder
     - DOCKER_REPO=tecnativa/odoo-base
-    - DOCKER_TAG=latest
-    - IMAGE_NAME=$DOCKER_REPO:$DOCKER_TAG
   matrix:
-    - DOCKER_TAG=8.0 ODOO_VERSIONS=8.0
-    - DOCKER_TAG=9.0 ODOO_VERSIONS=9.0
-    - DOCKER_TAG=10.0 ODOO_VERSIONS=10.0
-    - DOCKER_TAG=11.0 ODOO_VERSIONS=11.0
+    - DOCKER_TAG=8.0
+    - DOCKER_TAG=9.0
+    - DOCKER_TAG=10.0
+    - DOCKER_TAG=11.0
 
 before_install:
   - sudo apt-get update

hooks/build

@@ -7,6 +7,10 @@ else
     dockerfile="$DOCKER_TAG.Dockerfile"
 fi
 
+if [ -z "$IMAGE_NAME" ]; then
+    IMAGE_NAME="$DOCKER_REPO:$DOCKER_TAG"
+fi
+
 # See http://label-schema.org/rc1/#build-time-labels
 time docker image build \
     --build-arg VCS_REF="$GIT_SHA1" \

lib/odoobaselib/__init__.py

@@ -41,8 +41,7 @@
 def addons_config():
     """Yield addon name and path from ``ADDONS_YAML``."""
     config = dict()
-    private_done = False
-    core_done = False
+    special_missing = {PRIVATE, CORE}
     try:
         with open(ADDONS_YAML) as addons_file:
             for doc in yaml.load_all(addons_file):
@@ -53,11 +52,8 @@ def addons_config():
                     continue
                 # Flatten all sections in a single dict
                 for repo, addons in doc.items():
-                    if repo == PRIVATE:
-                        private_done = True
-                    elif repo == CORE:
-                        core_done = True
                     logging.debug("Processing %s repo", repo)
+                    special_missing.discard(repo)
                     for glob in addons:
                         logging.debug("Expanding glob %s", glob)
                         for addon in iglob(os.path.join(SRC_DIR, repo, glob)):
@@ -66,31 +62,27 @@ def addons_config():
                             config.setdefault(addon, set())
                             config[addon].add(repo)
     except IOError:
-        logging.debug('Could not find addons configuration yml.')
+        logging.debug('Could not find addons configuration yaml.')
     # By default, all private and core addons are enabled
-    if not private_done:
-        logging.debug("Auto-adding all private repo addons")
-        config.update({
-            os.path.basename(addon): {PRIVATE}
-            for addon in iglob(os.path.join(SRC_DIR, PRIVATE, "*"))
-        })
-    if not core_done:
-        logging.debug("Auto-adding all core repo addons")
-        config.update({
-            os.path.basename(addon): {CORE}
-            for addon in iglob(os.path.join(SRC_DIR, CORE, "*"))
-        })
+    for repo in special_missing:
+        logging.debug("Auto-adding all addons from %s", repo)
+        for addon in iglob(os.path.join(SRC_DIR, repo, "*")):
+            addon = os.path.basename(addon)
+            config.setdefault(addon, set())
+            config[addon].add(repo)
     logging.debug("Resulting configuration: %r", config)
     for addon, repos in config.items():
         # Private addons are most important
         if PRIVATE in repos:
             yield addon, PRIVATE
+            continue
         # Odoo core addons are least important
-        elif repos == {CORE}:
+        if repos == {CORE}:
             yield addon, CORE
+            continue
+        repos.discard(CORE)
         # Other addons fall in between
-        elif len(repos) != 1:
+        if len(repos) != 1:
             logging.error("Addon %s defined in several repos %s", addon, repos)
             raise Exception
-        else:
-            yield addon, repos.pop()
+        yield addon, repos.pop()

tests/__init__.py

@@ -20,7 +20,7 @@
 DIR = dirname(__file__)
 ODOO_PREFIX = ("odoo", "--stop-after-init", "--workers=0")
 ODOO_VERSIONS = frozenset(environ.get(
-    "ODOO_VERSIONS", "8.0 9.0 10.0 11.0").split())
+    "DOCKER_TAG", "8.0 9.0 10.0 11.0").split())
 PG_VERSIONS = frozenset(environ.get(
     "PG_VERSIONS", "9.6").split())
 SCAFFOLDINGS_DIR = join(DIR, "scaffoldings")
@@ -179,6 +179,10 @@ def test_dotd(self):
                 ("test", "-d", "custom/src/private/private_addon"),
                 ("test", "-f", "custom/src/private/private_addon/__init__.py"),
                 ("test", "-e", "auto/addons/private_addon"),
+                # Addon from extra repo takes higher priority than core version
+                ("realpath", "auto/addons/product"),
+                ("bash", "-c", 'test "$(realpath auto/addons/product)" == '
+                               '/opt/odoo/custom/src/dummy_repo/product'),
                 # ``odoo`` command works
                 ("odoo", "--version"),
                 # Implicit ``odoo`` command also works

tests/scaffoldings/dotd/custom/src/addons.yaml

@@ -1,5 +1,6 @@
 dummy_repo:
   - dummy_addon
+  - product
 
 ---
 ONLY: