Skip to content
This repository has been archived by the owner on May 22, 2024. It is now read-only.

[terra-functional-testing] update logger output for the --disable-server flag #852

Merged
merged 1 commit into from
Mar 8, 2024

Conversation

sdadn
Copy link
Contributor

@sdadn sdadn commented Mar 8, 2024

Summary

This PR corrects the logger output for the target ip address when the --disable-server flag is used.

Before:

CleanShot 2024-03-08 at 14 51 10

After:
CleanShot 2024-03-08 at 14 56 26

This will now display the correct ip address for debugging purposes.

Testing

This change was tested using:

  • WDIO
  • Jest
  • Visual testing (please attach a screenshot or recording)
  • Other (please describe below)
  • No tests are needed

This was tested by installing a local copy in terra-framework and running the wdio test, resulting in the screenshots above.

Reviews

In addition to engineering reviews, this PR needs:

  • UX review
  • Accessibility review
  • Functional review

Additional Details


Thank you for contributing to Terra.
@cerner/terra

@sdadn sdadn self-assigned this Mar 8, 2024
@sdadn sdadn requested a review from a team as a code owner March 8, 2024 21:00
@github-actions github-actions bot temporarily deployed to preview-pr-852 March 8, 2024 21:00 Destroyed
Copy link
Contributor

github-actions bot commented Mar 8, 2024

Fails
🚫 Please include a CHANGELOG entry for each changed package on this PR. Looks like a CHANGELOG entry is missing for:
  • terra-functional-testing

Generated by 🚫 dangerJS against b628982

@sdadn
Copy link
Contributor Author

sdadn commented Mar 8, 2024

Fails
🚫 Please include a CHANGELOG entry for each changed package on this PR. Looks like a CHANGELOG entry is missing for:

  • terra-functional-testing

Generated by 🚫 dangerJS against b628982

This is a niche fix for an unreleased change, I don't think it warrants a CHANGELOG entry.

@sdadn sdadn merged commit a0eac08 into main Mar 8, 2024
3 of 5 checks passed
@sdadn sdadn deleted the update-terra-functional-testing-logger branch March 8, 2024 21:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants