← Back to branch summary

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

« back to all changes in this revision

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

  • Committer: Robert Collins
  • Date: 2011-10-24 05:21:01 UTC
  • Revision ID: robertc@robertcollins.net-20111024052101-m1er1qzem8v9wibg
https://launchpad.net/bugs/880641
Do not use url for pageid when pageid is missing.

Show diffs side-by-side

added added

removed removed

Lines of Context:
src/oopstools/oops/test/test_dboopsloader.py
147
147
        unicode_url = u'http://example.com/foo\u2019s asset'
148
148
        report = { 'url': unicode_url, 'id': 'testurlhandling'}
149
149
        expected_url = urllib.quote(unicode_url.encode('utf8'))
150
 
        oops = parsed_oops_to_model_oops(report, '/dev/null')
 
150
        oops = parsed_oops_to_model_oops(report, 'test_url_handling')
151
151
        self.assertEqual(expected_url, oops.url)
152
152
 
 
153
    def test_no_topic_pageid_empty_bug_880641(self):
 
154
        report = { 'url': 'foo', 'id': 'testnotopichandling'}
 
155
        oops = parsed_oops_to_model_oops(report, 'bug_880641')
 
156
        self.assertEqual('', oops.pageid)