Issue #38: Test for features overrides. (#841)

acquia · Dec 22, 2016 · 51ee2c5 · 51ee2c5
1 parent 549171f
commit 51ee2c5
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/phing/tasks/setup.xml b/phing/tasks/setup.xml
@@ -293,6 +293,18 @@
     <drush command="fra" assume="yes" alias="${drush.alias}">
       <option name="bundle">${bundle}</option>
     </drush>
+    <if>
+      <equals arg1="${cm.features.no-overrides}" arg2="true"/>
+      <then>
+        <exec command="${drush.bin} @${drush.alias} fl --bundle=${bundle} | grep -Ei '(changed|conflicts|added)( *)$'" outputProperty="features.overrides"/>
+        <if>
+          <istrue value="${features.overrides}"/>
+          <then>
+            <fail>A feature in the ${bundle} bundle is overridden. You must re-export this feature to incorporate the changes.</fail>
+          </then>
+        </if>
+      </then>
+    </if>
   </target>
 
   <target name="setup:toggle-modules" description="Enables and uninstalls specified modules.">

diff --git a/template/blt/project.yml b/template/blt/project.yml
@@ -61,6 +61,7 @@ target-hooks:
 # cm:
 #   features:
 #     bundle: blted8
+#     no-overrides: 'true'
 
 # Define any custom Phing files that you'd like to import. E.g., ${repo.root}/blt/phing/build.xml
 import: ~