Skip to content

Test coverage for restricted issues search#1629

Merged
labkey-klum merged 1 commit intorelease25.7-SNAPSHOTfrom
25.7_fb_private_issue_search_fix
Feb 6, 2026
Merged

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

Conversation

@labkey-klum
Copy link
Contributor

@labkey-klum labkey-klum commented Feb 4, 2026

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

Related PR

LabKey/platform#7384

@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.

@brentlogan
Copy link
Collaborator

@labkey-martyp Let us know when it's merged in and we'll test it in the SNAPSHOT. Thanks!

@labkey-martyp
Copy link
Contributor

@labkey-martyp Let us know when it's merged in and we'll test it in the SNAPSHOT. Thanks!

Will do. Should be today sometime.

@labkey-klum labkey-klum merged commit 801333e into release25.7-SNAPSHOT Feb 6, 2026
9 checks passed
@labkey-klum labkey-klum deleted the 25.7_fb_private_issue_search_fix branch February 6, 2026 16:36
@labkey-martyp
Copy link
Contributor

@labkey-martyp Let us know when it's merged in and we'll test it in the SNAPSHOT. Thanks!

Will do. Should be today sometime.

@brentlogan The fix is merge in. It will be in build #927. Here is the PR with the fix.

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