Skip to content
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

test: Remove disabled/skipped E2E tests #26980

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

rafaeelaudibert
Copy link
Member

I don't believe skipped/disabled tests have any actual value, and they add noise to our PR reviews. I'm proposing we remove them all. If there's anything here that needs to be fixed, then it should become a GH issue.

I don't believe skipped/disabled tests have any actual value, and they simply add noise to our PR reviews. I'm proposing we remove them all. If there's anything here that needs to actually be fixed, then it should become a GH issue.
@rafaeelaudibert
Copy link
Member Author

Does anyone disagree with my take? :)

Copy link
Member

@pauldambra pauldambra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd be tempted to uncomment/unskip and see if any work

but after that 👍

*
* Needs https://github.com/PostHog/posthog/issues/8250 before can query on timestamp
*/
it.skip('can filter after a date and can filter before it', () => {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that sounds super fixable but 🤷

Terrible idea, I know
@rafaeelaudibert rafaeelaudibert merged commit f4878a9 into master Dec 18, 2024
90 checks passed
@rafaeelaudibert rafaeelaudibert deleted the remove-skipped-commented-testcases branch December 18, 2024 20:57
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