Skip to content

Fix get_queries_test#91

Merged
labkey-alan merged 1 commit intodevelopfrom
fb_fix_get_queries_test
Mar 3, 2026
Merged

Fix get_queries_test#91
labkey-alan merged 1 commit intodevelopfrom
fb_fix_get_queries_test

Conversation

@labkey-alan
Copy link
Contributor

Rationale

This PR fixes the test failure for get_queries

Related Pull Requests

Changes

  • Remove test steps that pass include_columns, include_view_data_url flags
    • These test steps don't really add much to the get_queries test, and they result in behavior that is inconsistent between environments. This test passes 100% of the time for me locally, but fails consistently on TC. We have unit tests that verify we're properly translating the snake_case kwargs to camelCase, so I'm not particularly worried about losing coverage.

@labkey-alan labkey-alan self-assigned this Mar 2, 2026
@labkey-alan labkey-alan merged commit 7147482 into develop Mar 3, 2026
4 checks passed
@labkey-alan labkey-alan deleted the fb_fix_get_queries_test branch March 3, 2026 15:55
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