Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test components for biologics cross-folder input form actions tests #1821

Merged
merged 4 commits into from
Feb 15, 2024

Conversation

labkey-chrisj
Copy link
Contributor

@labkey-chrisj labkey-chrisj commented Feb 8, 2024

Rationale

This adds components to support test coverage of cross-folder form-based data inputs in Biologics.

It's possible both of these components belong in biologics but I'm starting them out here because they might be shared at some point
Also, they probably belong in different locations- they're in the best places I could figure out, given the context. @labkey-tchad , probably this is your bailiwick

SelectInputOption is useful for scraping row data out of reactSelect options that put lots of metadata into them- this format is definitely used in component selectors in biologics registry, not sure if this data format is universal or if these select-items are formatted on an ad-hoc basis @labkey-nicka , @labkey-alan, thoughts?

The Modal dialog, when submitted, calls the 'doAndWaitForUpdate' method implemented on the component getting it

Related Pull Requests

https://github.com/LabKey/biologics/pull/2704

Changes

  • Dialog to prompt changing target project, clearing form
  • wrapper for select-input-option

@labkey-chrisj labkey-chrisj self-assigned this Feb 8, 2024
@labkey-chrisj labkey-chrisj marked this pull request as draft February 8, 2024 01:41
@labkey-chrisj labkey-chrisj removed their assignment Feb 8, 2024
@labkey-chrisj labkey-chrisj changed the title Test coverage for biologics cross-folder input form actions Test components for biologics cross-folder input form actions tests Feb 8, 2024
@labkey-chrisj labkey-chrisj self-assigned this Feb 9, 2024
@labkey-chrisj labkey-chrisj marked this pull request as ready for review February 9, 2024 05:18
@labkey-chrisj labkey-chrisj merged commit 6eb1d8a into develop Feb 15, 2024
3 checks passed
@labkey-chrisj labkey-chrisj deleted the fb_crossFolderFormActionTest branch February 15, 2024 01:16
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.

2 participants