details:   https://code.openbravo.com/erp/devel/pi/rev/d45f2b8d352e
changeset: 35179:d45f2b8d352e
user:      Asier Lostalé <asier.lostale <at> openbravo.com>
date:      Fri Nov 30 15:14:27 2018 +0100
summary:   fixed bug 39725: ViewGeneration fails in retail CI

  Retail now has a new grid configuration which caused:
   1. SortingFilteringGridConfiguration to be skipped as it is detected as 
custom
      grid config.
   2. SortingFilteringGridConfiguration to leave core module in an inconsistent
      state having isIndevelopment = null

  This fix:
   * Ensures even skipped SortingFilteringGridConfiguration core is kept in a
     correct state
   * Registers the new grid config so that SortingFilteringGridConfiguration is
     executed
   * Recovers ViewGeneration execution

diffstat:

 src-test/src/org/openbravo/test/views/GridConfigurationTest.java             | 
 3 ++-
 src-test/src/org/openbravo/test/views/SortingFilteringGridConfiguration.java | 
 2 +-
 src-test/src/org/openbravo/test/views/ViewGeneration.java                    | 
 2 --
 3 files changed, 3 insertions(+), 4 deletions(-)

diffs (44 lines):

diff -r 6bb6bc73bc75 -r d45f2b8d352e 
src-test/src/org/openbravo/test/views/GridConfigurationTest.java
--- a/src-test/src/org/openbravo/test/views/GridConfigurationTest.java  Fri Nov 
30 14:06:35 2018 +0100
+++ b/src-test/src/org/openbravo/test/views/GridConfigurationTest.java  Fri Nov 
30 15:14:27 2018 +0100
@@ -41,7 +41,8 @@
    * Standard Grid Configuration IDs
    */
   private static final List<String> CORE_DEFAULT_GRID_CONFIGS = Arrays.asList(
-      "4701BC23719C41FAA422305FCDBBAF85", "FDA9AFD8D7504E18A220EFC01F5D28D3");
+      "4701BC23719C41FAA422305FCDBBAF85", "FDA9AFD8D7504E18A220EFC01F5D28D3",
+      "1AD989605ACA4F5FB6C11B2E7AC88867");
 
   /**
    * @return the current number of grid configurations defined in the system.
diff -r 6bb6bc73bc75 -r d45f2b8d352e 
src-test/src/org/openbravo/test/views/SortingFilteringGridConfiguration.java
--- 
a/src-test/src/org/openbravo/test/views/SortingFilteringGridConfiguration.java  
    Fri Nov 30 14:06:35 2018 +0100
+++ 
b/src-test/src/org/openbravo/test/views/SortingFilteringGridConfiguration.java  
    Fri Nov 30 15:14:27 2018 +0100
@@ -81,7 +81,7 @@
 
   @AfterClass
   public static void cleanUp() {
-    if (Boolean.TRUE.equals(coreWasInDevelopment)) {
+    if (getNumberOfGridConfigurations() > 0 || 
Boolean.TRUE.equals(coreWasInDevelopment)) {
       return;
     }
     OBContext.setAdminMode(true);
diff -r 6bb6bc73bc75 -r d45f2b8d352e 
src-test/src/org/openbravo/test/views/ViewGeneration.java
--- a/src-test/src/org/openbravo/test/views/ViewGeneration.java Fri Nov 30 
14:06:35 2018 +0100
+++ b/src-test/src/org/openbravo/test/views/ViewGeneration.java Fri Nov 30 
15:14:27 2018 +0100
@@ -34,7 +34,6 @@
 
 import org.apache.logging.log4j.LogManager;
 import org.apache.logging.log4j.Logger;
-import org.junit.Ignore;
 import org.junit.Test;
 import org.openbravo.base.weld.test.WeldBaseTest;
 import org.openbravo.client.application.ViewComponent;
@@ -55,7 +54,6 @@
   private ViewComponent vc;
 
   @Test
-  @Ignore("Failing with retail mods. Temporaryly ignoring. See #39725")
   public void viewsShouldBeGeneratedAfterADCSCaching() {
     setSystemAdministratorContext();
 


_______________________________________________
Openbravo-commits mailing list
Openbravo-commits@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to