Conversation
I was sleepy and dumb wen I wrote the first ARG_REF_ALL_ARGS implementation. It's a lot simpler to just treat all the logic on flex and bison. Also, this implementation is buggy, if there is no `WORD` with the `ARG_REF_ALL_ARGS` it will leave a `TEXT` node with not children behind.
artP2
reviewed
Jan 27, 2025
Contributor
|
Add a section for the test suite in docs/CONTRIBUTING.md or README.md |
artP2
reviewed
Jan 27, 2025
artP2
reviewed
Jan 27, 2025
artP2
reviewed
Jan 27, 2025
artP2
reviewed
Jan 28, 2025
Our project is getting larger so we need some tests to be certain that everything is working as intended. To solve this, create new testing framework called _Vunit_. It tries to achieve the following points: - TAP[1] output. It's a protocol for test output. It's moves the complexity of formatting the output to other utilities. For now we're going to use the autoconf support[2] for it. - Use of macros for simplifying writing tests. With the use of `VUNIT_TEST_SUITE` and the `ASSERT` macros it tries to remove a lot of the boilerplate code. [1]: https://testanything.org/ [2]: https://www.gnu.org/savannah-checkouts/gnu/automake/manual/html_node/Introduction-to-TAP.html Fixes: #23
We're going to write tests that treat the vinumc as black box. Giving input to the program and expecting an specific output. To facilitate those tests, create `vunit_run_vinumc*` functions.
Test the basic vinumc operation. This certainly does not cover all the system, another test are realy needed. These are just the first ones.
Now that we have test addthem to the CI.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The first commit is here because it fixes a bug the tests caught.