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

Overhaul pretty-printing in Testing #234

Open
florentc opened this issue Jan 30, 2023 · 2 comments
Open

Overhaul pretty-printing in Testing #234

florentc opened this issue Jan 30, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@florentc
Copy link
Member

florentc commented Jan 30, 2023

The Testing module invokes the revamped and parameterizable printers for skeletons, mockchain errors, and UTxO state (in the soon to be merged PR #228) but the highest level testing messages are still legacy basic string concatenation from Cooked V1.

It would be interesting to:

  • Write better printers that rely on Prettyprinter and PrettyCooked to replace legacy printing such as in testBinaryRelatedBy or testOneEquivClass and convert to string only at the very end using renderString from Cooked.Pretty
  • Move actual pretty printing to the Cooked.Pretty module or a Cooked.Pretty submodule
  • Investigate how to tweak Tasty's default output to include additional information (e.g. number of evaluated cooked traces when a test succeeds and nothing would be printed otherwise)
  • Investigate if it is possible to render pretty-printed Doc elements in Tasty instead of converting to String (which is now done with renderString) to benefit from a rendering that adapts to the terminal size in which the test suite is ran
@florentc florentc added the enhancement New feature or request label Jan 30, 2023
@mmontin
Copy link
Collaborator

mmontin commented Jul 15, 2024

@florentc is this issue outdated? what would be the actions to take if we were to handle this now?

@florentc
Copy link
Member Author

I think we can close it and reopen items in individual issues if needed. Especially the items regarding the testing module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants