Commit 24cbbcb5 authored by Greg Ward's avatar Greg Ward

Added test_initial_whitespace() to ensure that SF bug #622849 is fixed.

Change LongWordTestCase.setUp() -- remove leading whitespace from
  text string.
Comment fix.
parent ab73d46e
...@@ -224,11 +224,21 @@ What a mess! ...@@ -224,11 +224,21 @@ What a mess!
self.check_split("what the--.", ["what", " ", "the--."]) self.check_split("what the--.", ["what", " ", "the--."])
self.check_split("--text--.", ["--text--."]) self.check_split("--text--.", ["--text--."])
# I think David got this wrong in the bug report, but it can't # My initial mis-interpretation of part of the bug report --
# hurt to make sure it stays right! # These were always handled correctly, but it can't hurt to make
# sure that they *stay* correct!
self.check_split("--option", ["--option"]) self.check_split("--option", ["--option"])
self.check_split("--option-opt", ["--option-", "opt"]) self.check_split("--option-opt", ["--option-", "opt"])
def test_initial_whitespace(self):
# SF bug #622849 reported inconsistent handling of leading
# whitespace; let's test that a bit, shall we?
text = " This is a sentence with leading whitespace."
self.check_wrap(text, 50,
[" This is a sentence with leading whitespace."])
self.check_wrap(text, 30,
[" This is a sentence with", "leading whitespace."])
def test_split(self): def test_split(self):
# Ensure that the standard _split() method works as advertised # Ensure that the standard _split() method works as advertised
# in the comments # in the comments
...@@ -244,7 +254,7 @@ What a mess! ...@@ -244,7 +254,7 @@ What a mess!
class LongWordTestCase (BaseTestCase): class LongWordTestCase (BaseTestCase):
def setUp(self): def setUp(self):
self.wrapper = TextWrapper() self.wrapper = TextWrapper()
self.text = ''' self.text = '''\
Did you say "supercalifragilisticexpialidocious?" Did you say "supercalifragilisticexpialidocious?"
How *do* you spell that odd word, anyways? How *do* you spell that odd word, anyways?
''' '''
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment