← Back to branch summary

~lifeless/python-oops-tools/bug-881400

« back to all changes in this revision

Viewing changes to src/oopstools/oops/test/test_dboopsloader.py

  • Committer: Robert Collins
  • Date: 2011-10-31 02:07:24 UTC
  • mfrom: (6.2.2 python-oops-tools)
  • Revision ID: robertc@robertcollins.net-20111031020724-1itg2mas5r16bk21
Updated version of Roman's bad-url fix.

Show diffs side-by-side

added added

removed removed

Lines of Context:
src/oopstools/oops/test/test_dboopsloader.py
154
154
        report = { 'url': 'foo', 'id': 'testnotopichandling'}
155
155
        oops = parsed_oops_to_model_oops(report, 'bug_880641')
156
156
        self.assertEqual('', oops.pageid)
 
157
 
 
158
    def test_broken_url_handling(self):
 
159
        # This URL is not a valid URL - URL's are a subset of ASCII.
 
160
        broken_url = '/somep\xe1th'
 
161
        report = { 'url': broken_url, 'id': 'testbrokenurl'}
 
162
        # We handle such URL's by url quoting them, failing to do so being a
 
163
        # (not uncommon) producer mistake.
 
164
        expected_url = urllib.quote(broken_url)
 
165
        oops = parsed_oops_to_model_oops(report, 'test_broken_url_handling')
 
166
        self.assertEqual(expected_url, oops.url)
 
167