Skip to content

Conversation

@yordis
Copy link
Contributor

@yordis yordis commented Jan 13, 2022

Before submitting a pull request, please take a look at our
Contributing guidelines and verify:

  • If you've added code that should be tested, please add tests.
  • Ensure your code lints and the test suite passes (yarn lint) & (yarn test).

Context

Avoided the need to spy on the internal method called and tested the behavior as thru the public API only.

@yordis
Copy link
Contributor Author

yordis commented Jan 13, 2022

@AbhiPrasad this is actually ready for code review ... learned about it messing around in the messy branch I am working on.

Copy link
Member

@AbhiPrasad AbhiPrasad left a comment

Choose a reason for hiding this comment

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

Thanks!

@AbhiPrasad AbhiPrasad enabled auto-merge (squash) January 13, 2022 14:05
@AbhiPrasad AbhiPrasad merged commit 197c0be into getsentry:master Jan 13, 2022
@yordis yordis deleted the improve-test-cases branch January 13, 2022 17:51
@yordis yordis restored the improve-test-cases branch May 13, 2022 04:03
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