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

fix(VideoPlayer): fix for mocking VideoPlayerAPI in storyshots #2692

Merged
merged 5 commits into from
Jun 8, 2020

Conversation

jeffchew
Copy link
Member

@jeffchew jeffchew commented Jun 8, 2020

Related Ticket(s)

Refs #2688

Description

The CI checks were running with obfuscated Kaltura IDs since they are
passed in as environment variables, so this is setting up a mock for
the VideoPlayerAPI service to return back mocked data instead.

Changelog

Changed

  • Created jest mock for @carbon/ibmdotcom-services in React package

The CI checks were running with obfuscated Kaltura IDs since they are
passed in as environment variables, so this is setting up a mock for
the VideoPlayerAPI service to return back mocked data instead.
@jeffchew jeffchew added the package: react Work necessary for the Carbon for IBM.com react components package label Jun 8, 2020
annawen1 and others added 4 commits June 8, 2020 16:13
…arbon-design-system#2691)

When using the video card in <LinkList />, it's pulling in the generating the kaltura thumbnail by default. Need to pass in a prop, disableImage to prevent the thumbnail from being generated.
The CI checks were running with obfuscated Kaltura IDs since they are
passed in as environment variables, so this is setting up a mock for
the VideoPlayerAPI service to return back mocked data instead.
…ideoplayer-mock

# Conflicts:
#	packages/react/src/__tests__/__snapshots__/storyshots.test.js.snap
Copy link
Contributor

@asudoh asudoh left a comment

Choose a reason for hiding this comment

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

LGTM 👍 - Thanks @jeffchew!

@ibmdotcom-bot
Copy link
Contributor

@jeffchew jeffchew merged commit 3699699 into carbon-design-system:master Jun 8, 2020
@jeffchew jeffchew deleted the fix/videoplayer-mock branch June 8, 2020 20:45
@ibmdotcom-bot
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: react Work necessary for the Carbon for IBM.com react components package
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants