Skip to content

SNOW-2021007: disable prod tests#2140

Merged
sfc-gh-jszczerbinski merged 2 commits intomasterfrom
SNOW-2021007-disable-prod-tests
Apr 2, 2025
Merged

SNOW-2021007: disable prod tests#2140
sfc-gh-jszczerbinski merged 2 commits intomasterfrom
SNOW-2021007-disable-prod-tests

Conversation

@sfc-gh-jszczerbinski
Copy link
Copy Markdown
Contributor

@sfc-gh-jszczerbinski sfc-gh-jszczerbinski commented Apr 2, 2025

Overview

SNOW-2021007
Disable prod tests that fail due to changes in backend error responses, those tests will be changed in next quarter to be independent of backend.

Pre-review self checklist

  • PR branch is updated with all the changes from master branch
  • The code is correctly formatted (run mvn -P check-style validate)
  • New public API is not unnecessary exposed (run mvn verify and inspect target/japicmp/japicmp.html)
  • The pull request name is prefixed with SNOW-XXXX:
  • Code is in compliance with internal logging requirements

@sfc-gh-jszczerbinski sfc-gh-jszczerbinski requested a review from a team as a code owner April 2, 2025 09:26
@sfc-gh-jszczerbinski sfc-gh-jszczerbinski force-pushed the SNOW-2021007-disable-prod-tests branch from 7d4dc09 to ae5e690 Compare April 2, 2025 09:27
@sfc-gh-snowflakedb-snyk-sa
Copy link
Copy Markdown
Contributor

sfc-gh-snowflakedb-snyk-sa commented Apr 2, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

@sfc-gh-jszczerbinski sfc-gh-jszczerbinski force-pushed the SNOW-2021007-disable-prod-tests branch from ae5e690 to 2c58473 Compare April 2, 2025 09:35
@sfc-gh-jszczerbinski sfc-gh-jszczerbinski merged commit 0687a2e into master Apr 2, 2025
139 of 142 checks passed
@sfc-gh-jszczerbinski sfc-gh-jszczerbinski deleted the SNOW-2021007-disable-prod-tests branch April 2, 2025 12:33
@github-actions github-actions bot locked and limited conversation to collaborators Apr 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants