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

Avoid hitting real API in test #1047

Merged
merged 2 commits into from
Oct 23, 2023
Merged

Conversation

keithbro-imx
Copy link
Contributor

Summary

Sandbox is currently unavailable and this is causing all PRs to fail due a failing test which is trying to get a 200 response from our sandbox API.

Instead, use an Axios adapter to intercept the request and check the relevant info that's of interest.

Customer Impact

No impact.

Added

Changed

Deprecated

Removed

Fixed

Security

Things worth calling out

Before submitting the PR, please consider the following:

  • Prefix your PR title with feat: , fix: , chore: , docs:, or refactor:.

@keithbro-imx keithbro-imx requested a review from a team as a code owner October 23, 2023 02:42
import { ImxConfiguration, ImxModuleConfiguration, createImmutableXConfiguration } from './config';

describe('ImmutableXClient', () => {
it('should instantiate a SANDBOX ImmutableXClient', async () => {
it.skip('should instantiate a SANDBOX ImmutableXClient', async () => {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Couldn't get this one to work

@alex-connolly alex-connolly merged commit 5f70919 into main Oct 23, 2023
6 checks passed
@alex-connolly alex-connolly deleted the task/avoid-hitting-real-api branch October 23, 2023 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants