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
  • !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
None
Milestone
None
Assign milestone
Time tracking
Source branch: testInvalidationBug
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7