Skip to content

Test coverage for restricted issues search#1629

Open
labkey-klum wants to merge 1 commit intorelease25.7-SNAPSHOTfrom
25.7_fb_private_issue_search_fix
Open

Test coverage for restricted issues search#1629
labkey-klum wants to merge 1 commit intorelease25.7-SNAPSHOTfrom
25.7_fb_private_issue_search_fix

Conversation

@labkey-klum
Copy link
Contributor

Rationale

This adds test automation to ensure expected behavior when searching for restricted issues:

  • Issues that a user doesn't necessarily have access to should still appear in the search listing
  • Search details link should display an error page if the user doesn't have access

@labkey-klum labkey-klum self-assigned this Feb 4, 2026
@labkey-klum labkey-klum requested review from a team, labkey-martyp and labkey-tchad and removed request for a team February 4, 2026 23:45
@brentlogan
Copy link
Collaborator

Please let us know if you'd like us to test this on one of our test VMs before you squash and merge.

@labkey-martyp
Copy link
Contributor

Please let us know if you'd like us to test this on one of our test VMs before you squash and merge.

Thanks for asking @brentlogan. The fix this PR is testing is a pretty straight forward fix in platform that we've tested and believe is low risk of any issues. It's also a little more challenging to get you a platform feature branch installer. Can you test on one of your test servers after we merge it in? We can quickly backout the change if there's something that isn't matching your expectations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants