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

Blui 5239 update test config & remove enzyme #326

Merged
merged 2 commits into from
Feb 1, 2024

Conversation

JeffGreiner-eaton
Copy link
Collaborator

@JeffGreiner-eaton JeffGreiner-eaton commented Jan 30, 2024

Fixes BLUI-5239.

Changes proposed in this Pull Request:

  • Remove Enzyme, types and adapters
  • Update test setup

Screenshots / Screen Recording (if applicable)

To Test:

  • checkout branch
  • install dependencies
  • yarn test

Any specific feedback you are looking for?

  • snapshot test options (can run either or)
    // snapshot test using react-test-renderer & jest
    test('renders correctly', () => {
        const tree = renderer.create(
            <WorkflowCardBody>
                <Text>This is workflow card body content.</Text>
            </WorkflowCardBody>
        ).toJSON();
        expect(tree).toMatchSnapshot();
      });

    //   snapshot test using @testing-library/react-native & jest
      test('renders correctly', () => {
        const tree = render(
            <WorkflowCardBody>
                <Text>This is workflow card body content.</Text>
            </WorkflowCardBody>
        )
        expect(screen.toJSON()).toMatchSnapshot()
      });

@github-actions github-actions bot added the brightlayer-ui Used to identify Brightlayer UI platform issues for easy filtering label Jan 30, 2024
@joebochill joebochill merged commit 927a725 into release/R31 Feb 1, 2024
3 checks passed
@joebochill joebochill deleted the feature/blui-5239-update-test-config branch February 1, 2024 18:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
brightlayer-ui Used to identify Brightlayer UI platform issues for easy filtering
Development

Successfully merging this pull request may close these issues.

2 participants