Skip to content

NIFI-13164: Moving from jsdom to happy dom for test environment execution#8760

Merged
scottyaslan merged 1 commit intoapache:mainfrom
mcgilman:NIFI-13164
May 7, 2024
Merged

NIFI-13164: Moving from jsdom to happy dom for test environment execution#8760
scottyaslan merged 1 commit intoapache:mainfrom
mcgilman:NIFI-13164

Conversation

@mcgilman
Copy link
Contributor

@mcgilman mcgilman commented May 7, 2024

NIFI-13164:

  • Moving from jsdom to happy dom for test environment execution.
  • Removing test for the Documentation component that was failing because it was attempting to load an iframe with the nifi documentation that wasn't available.

- Moving from jsdom to happy dom for test environment execution.
- Removing test for the Documentation component that was failing because it was attempting to load an iframe with the nifi documentation that wasn't available.
@mcgilman mcgilman added the ui Pull requests for work relating to the user interface label May 7, 2024
Copy link
Contributor

@scottyaslan scottyaslan left a comment

Choose a reason for hiding this comment

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

LGTM. I confirmed that the unit test output no longer displays any errors.

@scottyaslan scottyaslan merged commit 4bbe08f into apache:main May 7, 2024
shubhluck pushed a commit to shubhluck/nifi that referenced this pull request Jun 1, 2024
- Moving from jsdom to happy dom for test environment execution.
- Removing test for the Documentation component that was failing because it was attempting to load an iframe with the nifi documentation that wasn't available.

This closes apache#8760
shubhluck pushed a commit to shubhluck/nifi that referenced this pull request Jun 1, 2024
- Moving from jsdom to happy dom for test environment execution.
- Removing test for the Documentation component that was failing because it was attempting to load an iframe with the nifi documentation that wasn't available.

This closes apache#8760
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui Pull requests for work relating to the user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants