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

Split out tests #37

Merged
merged 5 commits into from Feb 10, 2012
Merged

Conversation

oubiwann
Copy link
Contributor

Again, based on the other branches, this one finishes up with the split out (updating unit tests) and adds backwards compatible features so that old code shouldn't break if it uses code with the new client changes. If it does break, that's a bug!

Duncan McGreggor added 5 commits December 20, 2011 19:20
…uded

  updating all the references to those exceptions).
* Added a todo comment about fixing a bad variable reference in a future
  branch.
* Moved authors into __init__, since files are now split (and added myself as
  an author).
* Created a testing module for general, unit test customizations (testing
  infrastructure, as opposed to actual unit tests... which are in "tests").
* Renamed test_redis.py to test_client.py to match the module name it tests.
* Renamed test classes to be more specific (including "TestCase" as part of
  their names).
* Reformatted the hiredis test cases so that they didn't appear inside an
  import try/except and followed standard Twisted code formatting.
rlotun added a commit that referenced this pull request Feb 10, 2012
@rlotun rlotun merged commit bed9e2b into deldotdr:master Feb 10, 2012
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.

None yet

2 participants