Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
erp5 erp5
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Labels
    • Labels
  • Merge requests 139
    • Merge requests 139
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedi
  • erp5erp5
  • Merge requests
  • !958

Closed
Created Oct 16, 2019 by Jérome Perrin@jeromeOwner
  • Report abuse
Report abuse

Fixes for test results for issues found when deploying

  • Overview 7
  • Commits 5
  • Pipelines 2
  • Changes 8

Fixes for the following problems found after deploying !924 (merged) :

  1. list of test result lines needs to show failed tests first (it was apparently a local change we lost)
  2. TestResult_getTestSuiteData was failing on some production data. Note that it still fail for test suites where a test suite repository does not have a buildout section id, but this property is supposed to be mandatory. If that cause error, I think we should just fix the test suites instead.
  3. gitlab API does not support repository names nexedi%2Fslapos.core . It would support nexedi%2Fslapos%2Ecore but requests does not allow that.
  4. Test results URL are the old UI, we want ERP5JS
Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: fix/test_result_after_mep
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7