Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Register
  • Sign in
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Merge requests 142
    • Merge requests 142
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Jobs
  • Commits
Collapse sidebar
  • nexedinexedi
  • erp5erp5
  • Merge requests
  • !1563

testInvalidationBug: Tolerate multiple indexation activities being spawned.

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Vincent Pelletier requested to merge vpelletier/erp5:testInvalidationBug into master Feb 23, 2022
  • Overview 5
  • Commits 2
  • Pipelines 0
  • Changes 1

@jm : Please check if these changes are compatible with the original intent of these tests.

In a future patchset I need tests to work with fulltext indexation enabled, which caused test failures where the test code was checking exactly how many activities were spawned. Fulltext indexation spawning an extra activity of course breaks such assumption, and I believe tests should not be sensitive to such extra activity. Hence these changes. I already pushed several more in the same idea, but in tests not as critical, and not going as deep into the details of how zope behaves, so I want to make sure I am not denaturing the intent of these tests.

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: testInvalidationBug
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7