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

Add test suite? #168

Open
mandarg opened this issue Dec 8, 2016 · 1 comment
Open

Add test suite? #168

mandarg opened this issue Dec 8, 2016 · 1 comment

Comments

@mandarg
Copy link
Contributor

@mandarg mandarg commented Dec 8, 2016

I introduced a change in #166 that broke a use case that I knew existed in the back of my mind, but didn't test correctly. A test suite could have helped me catch it sooner.

From looking through the codebase, I don't think one currently exists but I'd be interested in writing some up (for mausezahn at least). Any suggestions for frameworks, or pointers towards existing similar projects that have a good test suite? bats and Check are some things I have seen around that test at different levels.

@vkochan
Copy link
Contributor

@vkochan vkochan commented Dec 9, 2016

Right, there is no any testing code. For mausezahn/trafgen might be used scapy for sniffing and packets
comparing.

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

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.