--- a/review/tests/test_comment.py Sat Jun 19 00:17:31 2010 -0400
+++ b/review/tests/test_comment.py Sat Jun 19 01:55:46 2010 -0400
@@ -6,6 +6,8 @@
from mercurial import util as hgutil
from mercurial.node import hex
+# TODO: Figure out how to handle external editors nicely with nose.
+
a1, a2 = (messages.REVIEW_LOG_COMMENT_AUTHOR % '|').split('|')
@with_setup(setup_reviewed_sandbox, teardown_sandbox)
@@ -14,25 +16,6 @@
assert messages.REVIEW_LOG_COMMENTS % (0, 0) in output
-# TODO: Figure out how to handle external editors nicely with nose.
-#@with_setup(setup_reviewed_sandbox, teardown_sandbox)
-#def test_blank_comment():
- #try:
- #review(comment=True, message=' \t\n')
- #except hgutil.Abort, e:
- #error = str(e)
- #assert messages.COMMENT_REQUIRES_MESSAGE in error
- #else:
- #assert False, 'The correct error message was not printed.'
-
- #try:
- #review(comment=True, message=messages.COMMENT_EDITOR_MESSAGE)
- #except hgutil.Abort, e:
- #error = str(e)
- #assert messages.COMMENT_REQUIRES_MESSAGE in error
- #else:
- #assert False, 'The correct error message was not printed.'
-
@with_setup(setup_reviewed_sandbox, teardown_sandbox)
def test_comment_formatting():
review(comment=True, message=' \tTest comment one.\t ')
--- a/review/tests/test_signoff.py Sat Jun 19 00:17:31 2010 -0400
+++ b/review/tests/test_signoff.py Sat Jun 19 01:55:46 2010 -0400
@@ -4,6 +4,8 @@
from mercurial.node import hex
from .. import messages
+# TODO: Figure out how to handle external editors nicely with nose.
+
s1, s2 = (messages.REVIEW_LOG_SIGNOFF_AUTHOR % ('|', 'neutral')).split('|')
sy1, sy2 = (messages.REVIEW_LOG_SIGNOFF_AUTHOR % ('|', 'yes')).split('|')
sn1, sn2 = (messages.REVIEW_LOG_SIGNOFF_AUTHOR % ('|', 'no')).split('|')
@@ -13,25 +15,6 @@
output = review()
assert messages.REVIEW_LOG_SIGNOFFS % (0, 0, 0, 0) in output
-# TODO: Figure out how to handle external editors nicely with nose.
-#@with_setup(setup_reviewed_sandbox, teardown_sandbox)
-#def test_blank_signoff():
- #try:
- #review(signoff=True, message=' \t\n')
- #except hgutil.Abort, e:
- #error = str(e)
- #assert messages.SIGNOFF_REQUIRES_MESSAGE in error
- #else:
- #assert False, 'The correct error message was not printed.'
-
- #try:
- #review(signoff=True, message=messages.SIGNOFF_EDITOR_MESSAGE)
- #except hgutil.Abort, e:
- #error = str(e)
- #assert messages.SIGNOFF_REQUIRES_MESSAGE in error
- #else:
- #assert False, 'The correct error message was not printed.'
-
@with_setup(setup_reviewed_sandbox, teardown_sandbox)
def test_signoff_formatting():
review(signoff=True, message=' \tTest signoff one.\t ')