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

Test for lexer/parser #14

Open
alienscience opened this issue Nov 5, 2014 · 2 comments
Open

Test for lexer/parser #14

alienscience opened this issue Nov 5, 2014 · 2 comments

Comments

@alienscience
Copy link
Owner

The lexer and parser are likely to have more bugs in them. It would be good to have a test that throws valid and invalid commands at the lexer/parser and checks the results.

@EtienneBruines
Copy link
Collaborator

There have been several commits regarding lexer_test.go. What are the criteria for this being finished?

Perhaps referencing in commit messages, like Added tests for lexer, see #14 would make this easier to find.

@alienscience
Copy link
Owner Author

There are few tests at the moment because the Lexer and Parser are unfinished. There are some IMAP commands that will require big changes to both the lexer and parser. This issue will likely be open for a long time.

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

No branches or pull requests

2 participants