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 media related components #919

Merged
merged 68 commits into from
May 20, 2019

Conversation

pinarol
Copy link
Contributor

@pinarol pinarol commented Apr 23, 2019

This PR adds unit-tests for media related components.

Gutenberg PR: WordPress/gutenberg#15119

To Test:

yarn test

@pinarol pinarol added the Testing Anything related to automated tests label Apr 23, 2019
@pinarol pinarol added this to the v1.4 milestone Apr 23, 2019
@pinarol pinarol self-assigned this Apr 23, 2019
@pinarol pinarol removed this from the v1.4 milestone Apr 23, 2019
# Conflicts:
#	react-native-gutenberg-bridge/android/src/main/java/org/wordpress/mobile/ReactNativeGutenbergBridge/GutenbergBridgeJS2Parent.java
#	react-native-gutenberg-bridge/android/src/main/java/org/wordpress/mobile/ReactNativeGutenbergBridge/RNReactNativeGutenbergBridgeModule.java
#	react-native-gutenberg-bridge/android/src/main/java/org/wordpress/mobile/WPAndroidGlue/WPAndroidGlueCode.java
#	yarn.lock
@pinarol pinarol changed the base branch from issue/add-video-block to develop May 17, 2019 16:12
@pinarol pinarol force-pushed the issue/add-video-block-tests branch from 4056f19 to 6173746 Compare May 17, 2019 16:44
@pinarol pinarol added this to the v1.6 milestone May 17, 2019
@pinarol pinarol requested a review from etoledom May 17, 2019 18:24
@pinarol
Copy link
Contributor Author

pinarol commented May 17, 2019

I expect that failing CI tests will succeed after #997 is merged

Copy link
Contributor

@etoledom etoledom left a comment

Choose a reason for hiding this comment

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

Nice new set of tests, thank you for adding this!


jest.mock( 'react-native-gutenberg-bridge', () => (
{
requestMediaPickFromMediaLibrary: jest.fn(),
Copy link
Contributor

Choose a reason for hiding this comment

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

Nice way to mock the bridge 👍

@pinarol pinarol merged commit 52feb26 into develop May 20, 2019
@pinarol pinarol deleted the issue/add-video-block-tests branch May 20, 2019 15:10
@hypest hypest mentioned this pull request May 21, 2019
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing Anything related to automated tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants