Use warning instead of value error when tests are not being run in check_query

[scott-routledge2]

Changes included in this PR

As titled. This is to enable a green nightly, we can later go back and look at the tests that failed on an individual basis to see if they need to be removed/reworked.

Testing strategy

Locally/on CI

User facing changes

No

Checklist

• Pipelines passed before requesting review. To run CI you must include [run CI] in your commit message.
• I am familiar with the Contributing Guide
• I have installed + ran pre-commit hooks.

[IsaacWarren]

How many tests was this an issue for? Thanks Scott

[scott-routledge2]

@IsaacWarren It's a lot, maybe around 300? Not sure how many separate tests there are though.

[IsaacWarren]

@IsaacWarren It's a lot, maybe around 300? Not sure how many separate tests there are though.

Do we know when this started? Seems concerning, should we try to fix them instead of merging this? That seems like too many broken tests to do a release IMO

[ehsantn]

Looks like this is reverting a change from a recent PR: https://github.com/bodo-ai/Bodo/pull/83/files#diff-371bc62229377d93fed7d1bb28fbb123437b4f9d6bf9c1675de84f2ef9d14da7L325

[IsaacWarren]

Looks like this is reverting a change from a recent PR: https://github.com/bodo-ai/Bodo/pull/83/files#diff-371bc62229377d93fed7d1bb28fbb123437b4f9d6bf9c1675de84f2ef9d14da7L325

Ah ok, thanks for the context, approving

[scott-routledge2]

@IsaacWarren It is sort of by design that the tests aren't being run in certain situations on azure to save time since we know another runner will run the test. So for example if check_query is passed as only_1D_jit=True it is skipped on one rank but the test still gets executed in the multi rank case.

The warning was added here because we had a situation where we were testing something with only_1D_jit and inspecting the logs which failed because the test wasn't being run on one rank.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (main@a28fb89). Learn more about missing BASE report.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #118   +/-   ##
=======================================
  Coverage        ?   77.84%           
=======================================
  Files           ?      160           
  Lines           ?    62064           
  Branches        ?     8769           
=======================================
  Hits            ?    48312           
  Misses          ?    11627           
  Partials        ?     2125