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

Improving test facilities to new format #99

Open
phorward opened this issue Jan 2, 2023 · 1 comment
Open

Improving test facilities to new format #99

phorward opened this issue Jan 2, 2023 · 1 comment
Assignees
Labels
feature New feature or request tests

Comments

@phorward
Copy link
Member

phorward commented Jan 2, 2023

It might be worth to change the current implementation of testmode:repl tests into a different notation, which can also be used for examples or explanations.

Current notation (Tokay v0.6)

#testmode:repl
print("a")
print("b" Int)
#---
#23
#---
#a
#b23

Improved notation (proposal by this issue):

#testmode:repl
print("a")  # a
print("b" Int)  # b23
#---
#23

The reason for this proposal:

  1. Better readability, especially for huger testcases. See dict.tok, for example. It isn't obvious to which input the output lines refer to.
  2. Testcases can also be used for documentation, to quickly show which result a given command produces when typed into the REPL
@phorward phorward added feature New feature or request tokay tests labels Jan 2, 2023
@phorward phorward self-assigned this Jan 2, 2023
@phorward phorward removed the tokay label Oct 8, 2023
@phorward
Copy link
Member Author

After some further development on Logics and its test suite, it became clearer and more convenient to provide a test-format as described here: https://github.com/viur-framework/logics#shared-test-cases

@phorward phorward pinned this issue Dec 22, 2023
@phorward phorward unpinned this issue Dec 22, 2023
@phorward phorward pinned this issue Dec 22, 2023
@phorward phorward changed the title Improving testmode:repl facilities Improving test facilities to new format Dec 22, 2023
@phorward phorward unpinned this issue Feb 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request tests
Projects
None yet
Development

No branches or pull requests

1 participant