Skip to content

Review skipped tests #8184

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

Open
3 tasks done
mtrezza opened this issue Sep 20, 2022 · 1 comment
Open
3 tasks done

Review skipped tests #8184

mtrezza opened this issue Sep 20, 2022 · 1 comment
Labels
type:ci CI related issue

Comments

@mtrezza
Copy link
Member

mtrezza commented Sep 20, 2022

New Feature / Enhancement Checklist

Current Limitation

There are tests that are currently skipped. Tests have open been skipped in the past because they were flaky. We stopped this approach about 1 year ago because of the risks involved to introduce bugs through code changes.

Feature / Enhancement Description

Review all skipped tests and determine:

  • what is it skipped?
  • should be (a) enabled or (b) removed?
  • if (a) what's necessary to enable it?

Example Use Case

See skipped tests, starting with xit or xdescribe.

Alternatives / Workarounds

n/a

@parse-github-assistant
Copy link

Thanks for opening this issue!

  • 🎉 We are excited about your ideas for improvement!

@mtrezza mtrezza added the type:ci CI related issue label Sep 20, 2022
@mtrezza mtrezza pinned this issue Sep 20, 2022
@mtrezza mtrezza unpinned this issue Oct 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:ci CI related issue
Projects
None yet
Development

No branches or pull requests

1 participant