/brz/remove-bazaar

To get this branch, use:
bzr branch http://gegoxaren.bato24.eu/bzr/brz/remove-bazaar

« back to all changes in this revision

Viewing changes to bzrlib/tests/test_mail_client.py

  • Committer: John Arbash Meinel
  • Date: 2009-06-12 18:05:15 UTC
  • mto: (4371.4.5 vila-better-heads)
  • mto: This revision was merged to the branch mainline in revision 4449.
  • Revision ID: john@arbash-meinel.com-20090612180515-t0cwbjsnve094oik
Add a failing test for handling nodes that are in the same linear chain.

It fails because the ancestry skipping causes us to miss the fact that the two nodes
are actually directly related. We could check at the beginning, as the 
code used to do, but I think that will be incomplete for the more-than-two
heads cases.

Show diffs side-by-side

added added

removed removed

Lines of Context:
bzrlib/tests/test_mail_client.py
97
97
        # We won't be able to know the temporary file name at this stage
98
98
        # so we can't raise an assertion with assertEqual
99
99
        cmdline = eclient._get_compose_commandline(None, None, 'file%')
100
 
        if eclient.elisp_tmp_file is not None:
101
 
            self.addCleanup(osutils.delete_any, eclient.elisp_tmp_file)
102
100
        commandline = ' '.join(cmdline)
103
101
        self.assertContainsRe(commandline, '--eval')
104
102
        self.assertContainsRe(commandline, '(compose-mail nil nil)')
109
107
        eclient = mail_client.EmacsMail(None)
110
108
        commandline = eclient._get_compose_commandline(u'jrandom@example.org',
111
109
            u'Hi there!', u'file%')
112
 
        if eclient.elisp_tmp_file is not None:
113
 
            self.addCleanup(osutils.delete_any, eclient.elisp_tmp_file)
114
110
        for item in commandline:
115
111
            self.assertFalse(isinstance(item, unicode),
116
112
                'Command-line item %r is unicode!' % item)