No need to check if no activities left as any left innocently failing activity...

No need to check if no activities left as any left innocently failing activity can mark site as not installed. Fix skin layer and remove Guards so ERP5Site_isReady script can be use from anonymous user (this is fine as long as we use this bt5 ONLY for automated testing).

bt5/erp5_wendelin_auto_configuration/SkinTemplateItem/portal_skins/erp5_wendelin_auto_configuration.xml

@@ -6,12 +6,33 @@
     </pickle>
     <pickle>
       <dictionary>
+        <item>
+            <key> <string>_local_properties</string> </key>
+            <value>
+              <tuple>
+                <dictionary>
+                  <item>
+                      <key> <string>id</string> </key>
+                      <value> <string>business_template_skin_layer_priority</string> </value>
+                  </item>
+                  <item>
+                      <key> <string>type</string> </key>
+                      <value> <string>float</string> </value>
+                  </item>
+                </dictionary>
+              </tuple>
+            </value>
+        </item>
         <item>
             <key> <string>_objects</string> </key>
             <value>
               <tuple/>
             </value>
         </item>
+        <item>
+            <key> <string>business_template_skin_layer_priority</string> </key>
+            <value> <float>110.0</float> </value>
+        </item>
         <item>
             <key> <string>id</string> </key>
             <value> <string>erp5_wendelin_auto_configuration</string> </value>

bt5/erp5_wendelin_auto_configuration/SkinTemplateItem/portal_skins/erp5_wendelin_auto_configuration/ERP5Site_isReady.py

@@ -2,10 +2,8 @@
   The purpose of the script is to tell if configuration of a site was finished or not.
   It is used from Deployment Tests.
 """
-number_of_activities = len(context.portal_activities.getMessageList())
-
 installed_bt5_list = [x.getTitle() for x in context.portal_templates.getInstalledBusinessTemplateList()]
-if "erp5_wendelin" in installed_bt5_list and number_of_activities == 0:
+if "erp5_wendelin" in installed_bt5_list:
   return 1
 else:
   return 0

bt5/erp5_wendelin_auto_configuration/SkinTemplateItem/portal_skins/erp5_wendelin_auto_configuration/ERP5Site_isReady.xml

@@ -53,9 +53,11 @@
             <value> <string></string> </value>
         </item>
         <item>
-            <key> <string>guard</string> </key>
+            <key> <string>_proxy_roles</string> </key>
             <value>
-              <persistent> <string encoding="base64">AAAAAAAAAAI=</string> </persistent>
+              <tuple>
+                <string>Manager</string>
+              </tuple>
             </value>
         </item>
         <item>
@@ -65,21 +67,4 @@
       </dictionary>
     </pickle>
   </record>
-  <record id="2" aka="AAAAAAAAAAI=">
-    <pickle>
-      <global name="Guard" module="Products.DCWorkflow.Guard"/>
-    </pickle>
-    <pickle>
-      <dictionary>
-        <item>
-            <key> <string>roles</string> </key>
-            <value>
-              <tuple>
-                <string>Manager</string>
-              </tuple>
-            </value>
-        </item>
-      </dictionary>
-    </pickle>
-  </record>
 </ZopeData>