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

Add unit tests for <SaveCodeButton /> #354

Merged
merged 1 commit into from
Jan 31, 2023

Conversation

justinkambic
Copy link
Contributor

@justinkambic justinkambic commented Jan 26, 2023

Summary

NOTE TO REVIEWERS: This should not be reviewed until #353 is merged.

Related to #264. Adds test for <SaveCodeButton />.

Implementation details

Adds unit tests.

How to validate this change

If the test looks like it makes sense and it is passing CI no further testing needed.

@justinkambic justinkambic added the test:unit Pertains to unit testing label Jan 26, 2023
@justinkambic justinkambic self-assigned this Jan 26, 2023
@apmmachine
Copy link
Contributor

apmmachine commented Jan 26, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-01-31T15:53:29.014+0000

  • Duration: 3 min 48 sec

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

  • /test all : Re-trigger the build for all the stages.

@justinkambic justinkambic changed the title 264/save code button tests Add unit tests for <SaveCodeButton /> Jan 26, 2023
@justinkambic justinkambic force-pushed the 264/save-code-button-tests branch from 55673b6 to 8fb4ce5 Compare January 31, 2023 15:53
@justinkambic justinkambic merged commit 9fa6437 into elastic:main Jan 31, 2023
@justinkambic justinkambic deleted the 264/save-code-button-tests branch January 31, 2023 16:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test:unit Pertains to unit testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants