Skip to content

WIP tests: Render ui output to HTML#4488

Closed
wisp3rwind wants to merge 1 commit intobeetbox:masterfrom
wisp3rwind:pr_render_ui
Closed

WIP tests: Render ui output to HTML#4488
wisp3rwind wants to merge 1 commit intobeetbox:masterfrom
wisp3rwind:pr_render_ui

Conversation

@wisp3rwind
Copy link
Copy Markdown
Member

This is a bare-bones implementation of an idea I've had in mind for a while: Record unit test output and render it to HTML to ease development and review regarding our UI.

Does that sound like a useful tool (@davidswarbrick)? Any other thoughts?

Commit message:

experiment to help with development of the commandline user interface. The idea is to record representative output from tests, instead of needing to manually run some beet commands to get feedback. This works by

  • recording pytest output in a new tox run (tox -e py310-render_ui)
  • parsing stdout in the tox commands_post, and turning it into a basic html document with ansi2html

To be useful, a bunch of new tests should be written that showcase most of the ui and enable color. (It might also be useful to only run this script for those specific tests, instead of the entire test suite.)

experiment to help with development of the commandline user interface.
The idea is to record representative output from tests, instead of
needing to manually run some beet commands to get feedback.
This works by

- recording pytest output in a new tox run (tox -e py310-render_ui)
- parsing stdout in the tox commands_post, and turning it into a basic html
  document with ansi2html

To be useful, a bunch of new tests should be written that showcase most
of the ui and enable color. (It might also be useful to only run this
script for those specific tests, instead of the entire test suite.)
@sampsyo
Copy link
Copy Markdown
Member

sampsyo commented Sep 17, 2022

Wow; this is incredibly cool!! I really like this idea—it could be useful not only for tests but also for producing nice screenshots and the like.

@stale
Copy link
Copy Markdown

stale bot commented Jan 16, 2023

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jan 16, 2023
@sampsyo
Copy link
Copy Markdown
Member

sampsyo commented Jan 16, 2023

I think it would be great to keep this in play!

@stale stale bot removed the stale label Jan 16, 2023
@stale
Copy link
Copy Markdown

stale bot commented May 20, 2023

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label May 20, 2023
@stale stale bot closed this Jun 10, 2023
@wisp3rwind
Copy link
Copy Markdown
Member Author

I'm probably not going to continue working on this anytime soon; but I came across two related projects that might serve as inspiration:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants