1. 10 Dec, 2019 4 commits
  2. 09 Dec, 2019 10 commits
  3. 06 Dec, 2019 5 commits
  4. 03 Dec, 2019 1 commit
    • Jérome Perrin's avatar
      monaco_editor_ui_test: repair test · 041b0614
      Jérome Perrin authored
      Now ERP5JS display list mode header only if the jio key does not contain
      /, so if we open a module with a hand-made URL with a trailing slash,
      it's no longer seen as a module.
      
      Repair this test by using a URL without that trailing / in the key.
      041b0614
  5. 02 Dec, 2019 10 commits
  6. 28 Nov, 2019 10 commits
    • Sebastien Robin's avatar
      69b88b4b
    • Jérome Perrin's avatar
      administration: fixes for pylint · 00e1736b
      Jérome Perrin authored
      00e1736b
    • Jérome Perrin's avatar
      ERP5Type: define ConstraintMixin._checkConsistency · 592e3eae
      Jérome Perrin authored
      Without this, subclassing defining this method will have arguments-differ
      pylint warning because the signature of Base._checkConsistency is
      different.
      592e3eae
    • Jérome Perrin's avatar
      forge/administration: move some scripts to administration · ca05b59a
      Jérome Perrin authored
      These scripts are used to check the site configuration and also in
      CodingStyleTest.
      
      This is just the minimal so that CodingStyleTest can run, but this could
      be extended to several scripts currently in erp5_toolbox skinfolder of
      erp5_forge which overlaps with erp5_administration. Also, regarding
      CodingStyleTest, it seems that erp5_administration contain a lot of
      things, so we may want to extract a smaller business template containing
      only the scripts used for static analysis of business template.
      ca05b59a
    • Jérome Perrin's avatar
      administration: include new component types in "check source code" · 783add94
      Jérome Perrin authored
      We want this check to run on all components.
      783add94
    • Jérome Perrin's avatar
      CodingStyle: ignore more skins and prefixs · a4e0fb2d
      Jérome Perrin authored
      Module is an acceptable prefix for modules
      
      Introduce a list of skins IDs that does not match our conventions but
      are acceptable because this is what external API are calling.
      a4e0fb2d
    • Jérome Perrin's avatar
      testXHTML: test portal types also from tools · 56e16adc
      Jérome Perrin authored
      Some portal types such as Business Template were not tested by testXHTML
      because this tests started by modules and recursively tests views based
      on allowed content types. Because of this approach, types that are not
      created in a module but in a tool were never tested.
      
      With unittest, the only way to dynamically add test methods to a class
      is to generate a test class in test_suite function. At this stage, the
      ERP5 site is not created yet, so the test had to introspects business
      templates XML.
      
      This now uses a slightly different approach, instead of finding modules
      and chain of allowed content types from business template XML, we only
      use business template to introspect the list of actions.
      The lookup of the appropriate containers is no longer done before setup
      by static analysis of business templates XML, but later once the site is
      created, by dynamic analysis of the modules and allowed content types on
      the running ERP5 site during the test method.
      If we don't find a chain of portal types, we create the test document in
      portal_trash, a tool without filter of content types.
      
      This way, we can test all views of all portal types. This revealed a few
      problems:
       - we need developer role to create components in portal_components, for
      this we add developer role to the current user.
       - Delivery Cell portal type looks not used, there are no container
      accepting it. We don't test delivery cell views for this reason.
       - VCS view on business template needs preferences and working copy
      setup. We just mark this test as expected failure for now.
       - Solver Decision has a form conditionnaly displayed when there's a
      relation to a solver, but this test does not evaluate action conditions
      and does not allow to call a script (that would made it possible to
      modify the document so that the condition is true). For now we also
      mark this as expected failure.
      56e16adc
    • Jérome Perrin's avatar
      immobilisation: remove views from Delivery Cell · 439745de
      Jérome Perrin authored
      Delivery Cell is not really used these days and cannot be created in
      testXHTML, so drop these useless views.
      
      For reference, the same views still exists on all other movements portal
      types, so we don't "loose anything important" by removing the views
      here.
      439745de
    • Jérome Perrin's avatar
      budget: mark Budget Transfer view not visible · 36a87f63
      Jérome Perrin authored
      The corresponding form, BudgetTransfer_viewBudgetTransferLine does not
      exist, so mark view as not visible for now so that it's not tested.
      36a87f63
    • Jérome Perrin's avatar
      base: make Geographical Location "View on map" a jump action · d095b939
      Jérome Perrin authored
      This redirects to another site, so it is not really a view action.
      
      This also confuses testXHTML.
      d095b939