snowflakedb / snowpark-java-scala

Snowflake Snowpark Java & Scala API
Apache License 2.0
18 stars 21 forks source link

SNOW-1338064 Make tests not rely on ordering of results #99

Closed sfc-gh-llorimer closed 6 months ago

sfc-gh-llorimer commented 6 months ago

Please answer these questions before submitting your pull requests. Thanks!

  1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    Fixes SNOW-1338064

  2. Fill out the following pre-review checklist:

    • [ ] I am adding a new automated test(s) to verify correctness of my new code
    • [ ] I am adding new logging messages
    • [ ] I am adding a new telemetry message
    • [ ] I am adding new credentials
    • [ ] I am adding a new dependency
  3. Please describe how your code solves the related issue.

    This test changed in this PR rely on implicit ordering. This PR adds explicit ordering to this test by sorting the output.

Pre-review checklist

(For Snowflake employees)