Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • erp5 erp5
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Merge requests 141
    • Merge requests 141
  • 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
  • nexedi
  • erp5erp5
  • Merge requests
  • !1563

Merged
Created Feb 23, 2022 by Vincent Pelletier@vpelletierOwner

testInvalidationBug: Tolerate multiple indexation activities being spawned.

  • Overview 5
  • Commits 2
  • 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
Reviewer
Request review from
Time tracking
Source branch: testInvalidationBug
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7