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

WIP: fix(tests): A first attempt at a sane pattern to begin testing the handling of the protocol. #282

Merged
1 commit merged into from
Jan 9, 2015

Conversation

jirwin
Copy link
Collaborator

@jirwin jirwin commented Jan 9, 2015

This is by no means a finished product. Just a jumping off point to get a discussion started.

What

package.json: Change the test script so that we only test files in the test directory that start with test-
test/mockircd.js: A simple mock server that allows us to send and receive data, and makes that data retrievable later. This isn't supposed to be an implementation if an ircd, more a tool for unit testing.
test/test-irc.js: The beginnings of testing the irc part.
test/test-parse-message.js: Renamed so that it would be run by npm test. Removed an unneeded jshint line.

Todo

  • Figure out some fixture system for long 'transactions'
  • Prioritize parts to be tested first so we get tests out sooner than later

@jirwin jirwin force-pushed the jirwin/first-pass-at-tests branch from f539d01 to b0129fc Compare January 9, 2015 10:35
ghost pushed a commit that referenced this pull request Jan 9, 2015
WIP: fix(tests): A first attempt at a sane pattern to begin testing the handling of the protocol.
@ghost ghost merged commit a0a5b59 into master Jan 9, 2015
@jirwin jirwin deleted the jirwin/first-pass-at-tests branch January 9, 2015 20:18
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant