/b-gtk/fix-viz

# Copyright (C) 2007 John Arbash Meinel <john@arbash-meinel.com>

#

# This program is free software; you can redistribute it and/or modify

# it under the terms of the GNU General Public License as published by

# the Free Software Foundation; either version 2 of the License, or

# (at your option) any later version.

#

# This program is distributed in the hope that it will be useful,

# but WITHOUT ANY WARRANTY; without even the implied warranty of

# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the

# GNU General Public License for more details.

#

# You should have received a copy of the GNU General Public License

# along with this program; if not, write to the Free Software

# Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA

"""Test the Commit functionality."""

import os

import gtk

from bzrlib import (

    tests,

    revision,

    )

from bzrlib.plugins.gtk import commit

# TODO: All we need is basic ancestry code to test this, we shouldn't need a

# TestCaseWithTransport, just a TestCaseWithMemoryTransport or somesuch.

class TestPendingRevisions(tests.TestCaseWithTransport):

    def test_pending_revisions_none(self):

        tree = self.make_branch_and_tree('.')

        tree.commit('one')

        self.assertIs(None, commit.pending_revisions(tree))

    def test_pending_revisions_simple(self):

        tree = self.make_branch_and_tree('tree')

        rev_id1 = tree.commit('one')

        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()

        rev_id2 = tree2.commit('two')

        tree.merge_from_branch(tree2.branch)

        self.assertEqual([rev_id1, rev_id2], tree.get_parent_ids())

        pending_revisions = commit.pending_revisions(tree)

        # One primary merge

        self.assertEqual(1, len(pending_revisions))

        # Revision == rev_id2

        self.assertEqual(rev_id2, pending_revisions[0][0].revision_id)

        # No children of this revision.

        self.assertEqual([], pending_revisions[0][1])

    def test_pending_revisions_with_children(self):

        tree = self.make_branch_and_tree('tree')

        rev_id1 = tree.commit('one')

        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()

        rev_id2 = tree2.commit('two')

        rev_id3 = tree2.commit('three')

        rev_id4 = tree2.commit('four')

        tree.merge_from_branch(tree2.branch)

        self.assertEqual([rev_id1, rev_id4], tree.get_parent_ids())

        pending_revisions = commit.pending_revisions(tree)

        # One primary merge

        self.assertEqual(1, len(pending_revisions))

        # Revision == rev_id2

        self.assertEqual(rev_id4, pending_revisions[0][0].revision_id)

        # Two children for this revision

        self.assertEqual(2, len(pending_revisions[0][1]))

        self.assertEqual(rev_id3, pending_revisions[0][1][0].revision_id)

        self.assertEqual(rev_id2, pending_revisions[0][1][1].revision_id)

    def test_pending_revisions_multi_merge(self):

        tree = self.make_branch_and_tree('tree')

        rev_id1 = tree.commit('one')

        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()

        rev_id2 = tree2.commit('two')

        rev_id3 = tree2.commit('three')

        tree3 = tree2.bzrdir.sprout('tree3').open_workingtree()

        rev_id4 = tree3.commit('four')

        rev_id5 = tree3.commit('five')

        tree.merge_from_branch(tree2.branch)

        tree.merge_from_branch(tree3.branch)

        self.assertEqual([rev_id1, rev_id3, rev_id5], tree.get_parent_ids())

        pending_revisions = commit.pending_revisions(tree)

        # Two primary merges

        self.assertEqual(2, len(pending_revisions))

        # Revision == rev_id2

        self.assertEqual(rev_id3, pending_revisions[0][0].revision_id)

        self.assertEqual(rev_id5, pending_revisions[1][0].revision_id)

        # One child for the first merge

        self.assertEqual(1, len(pending_revisions[0][1]))

        self.assertEqual(rev_id2, pending_revisions[0][1][0].revision_id)

        # One child for the second merge

        self.assertEqual(1, len(pending_revisions[1][1]))

        self.assertEqual(rev_id4, pending_revisions[1][1][0].revision_id)

class Test_RevToPendingInfo(tests.TestCaseWithTransport):

    def test_basic_info(self):

        tree = self.make_branch_and_tree('tree')

        rev_id = tree.commit('Multiline\ncommit\nmessage',

                             committer='Joe Foo <joe@foo.com>',

                             timestamp=1191012408.674,

                             timezone=-18000

                             )

        rev = tree.branch.repository.get_revision(rev_id)

        rev_dict = commit.CommitDialog._rev_to_pending_info(rev)

        self.assertEqual({'committer':'Joe Foo',

                          'summary':'Multiline',

                          'date':'2007-09-28',

                          'revision_id':rev_id,

                         }, rev_dict)

class CommitDialogNoWidgets(commit.CommitDialog):

    def construct(self):

        pass # Don't create any widgets here

    def fill_in_data(self):

        pass # With no widgets, there are no widgets to fill out

class TestCommitDialogSimple(tests.TestCaseWithTransport):

    def test_setup_parameters_no_pending(self):

        tree = self.make_branch_and_tree('tree')

        rev_id = tree.commit('first')

        dlg = CommitDialogNoWidgets(tree)

        self.assertEqual(rev_id, dlg._basis_tree.get_revision_id())

        self.assertIs(None, dlg._pending)

        self.assertFalse(dlg._is_checkout)

    def test_setup_parameters_checkout(self):

        tree = self.make_branch_and_tree('tree')

        rev_id = tree.commit('first')

        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()

        tree2.branch.bind(tree.branch)

        dlg = CommitDialogNoWidgets(tree2)

        self.assertEqual(rev_id, dlg._basis_tree.get_revision_id())

        self.assertIs(None, dlg._pending)

        self.assertTrue(dlg._is_checkout)

    def test_setup_parameters_pending(self):

        tree = self.make_branch_and_tree('tree')

        rev_id1 = tree.commit('one')

        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()

        rev_id2 = tree2.commit('two')

        tree.merge_from_branch(tree2.branch)

        dlg = CommitDialogNoWidgets(tree)

        self.assertEqual(rev_id1, dlg._basis_tree.get_revision_id())

        self.assertIsNot(None, dlg._pending)

        self.assertEqual(1, len(dlg._pending))

        self.assertEqual(rev_id2, dlg._pending[0][0].revision_id)

    def test_setup_parameters_delta(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a'])

        tree.add(['a'], ['a-id'])

        dlg = CommitDialogNoWidgets(tree)

        self.assertIs(None, dlg._delta)

        dlg._compute_delta()

        delta = dlg._delta

        self.assertEqual([], delta.modified)

        self.assertEqual([], delta.renamed)

        self.assertEqual([], delta.removed)

        self.assertEqual([(u'a', 'a-id', 'file')], delta.added)

class TestCommitDialog(tests.TestCaseWithTransport):

    def test_no_pending(self):

        tree = self.make_branch_and_tree('tree')

        rev_id1 = tree.commit('one')

        dlg = commit.CommitDialog(tree)

        # TODO: assert that the pending box is hidden

        commit_col = dlg._treeview_files.get_column(0)

        self.assertEqual('Commit', commit_col.get_title())

        renderer = commit_col.get_cell_renderers()[0]

        self.assertTrue(renderer.get_active())

    def test_pending(self):

        tree = self.make_branch_and_tree('tree')

        rev_id1 = tree.commit('one')

        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()

        rev_id2 = tree2.commit('two',

                               committer='Joe Foo <joe@foo.com>',

                               timestamp=1191264271.05,

                               timezone=+7200)

        tree.merge_from_branch(tree2.branch)

        dlg = commit.CommitDialog(tree)

        # TODO: assert that the pending box is set to show

        commit_col = dlg._treeview_files.get_column(0)

        self.assertEqual('Commit*', commit_col.get_title())

        renderer = commit_col.get_cell_renderers()[0]

        self.assertFalse(renderer.get_active())

        values = [(r[0], r[1], r[2], r[3]) for r in dlg._pending_store]

        self.assertEqual([(rev_id2, '2007-10-01', 'Joe Foo', 'two')], values)

    def test_pending_multiple(self):

        tree = self.make_branch_and_tree('tree')

        rev_id1 = tree.commit('one')

        tree2 = tree.bzrdir.sprout('tree2').open_workingtree()

        rev_id2 = tree2.commit('two',

                               committer='Joe Foo <joe@foo.com>',

                               timestamp=1191264271.05,

                               timezone=+7200)

        rev_id3 = tree2.commit('three',

                               committer='Jerry Foo <jerry@foo.com>',

                               timestamp=1191264278.05,

                               timezone=+7200)

        tree.merge_from_branch(tree2.branch)

        tree3 = tree.bzrdir.sprout('tree3').open_workingtree()

        rev_id4 = tree3.commit('four',

                               committer='Joe Foo <joe@foo.com>',

                               timestamp=1191264279.05,

                               timezone=+7200)

        rev_id5 = tree3.commit('five',

                               committer='Jerry Foo <jerry@foo.com>',

                               timestamp=1191372278.05,

                               timezone=+7200)

        tree.merge_from_branch(tree3.branch)

        dlg = commit.CommitDialog(tree)

        # TODO: assert that the pending box is set to show

        values = [(r[0], r[1], r[2], r[3]) for r in dlg._pending_store]

        self.assertEqual([(rev_id3, '2007-10-01', 'Jerry Foo', 'three'),

                          (rev_id2, '2007-10-01', 'Joe Foo', 'two'),

                          (rev_id5, '2007-10-03', 'Jerry Foo', 'five'),

                          (rev_id4, '2007-10-01', 'Joe Foo', 'four'),

                         ], values)

    def test_filelist_added(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b/', 'tree/b/c'])

        tree.add(['a', 'b', 'b/c'], ['a-id', 'b-id', 'c-id'])

        dlg = commit.CommitDialog(tree)

        values = [(r[0], r[1], r[2], r[3], r[4]) for r in dlg._files_store]

        self.assertEqual([(None, None, True, 'All Files', ''),

                          ('a-id', 'a', True, 'a', 'added'),

                          ('b-id', 'b', True, 'b/', 'added'),

                          ('c-id', 'b/c', True, 'b/c', 'added'),

                         ], values)

    def test_filelist_renamed(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b/', 'tree/b/c'])

        tree.add(['a', 'b', 'b/c'], ['a-id', 'b-id', 'c-id'])

        rev_id1 = tree.commit('one')

        tree.rename_one('b', 'd')

        tree.rename_one('a', 'd/a')

        dlg = commit.CommitDialog(tree)

        values = [(r[0], r[1], r[2], r[3], r[4]) for r in dlg._files_store]

        self.assertEqual([(None, None, True, 'All Files', ''),

                          ('b-id', 'd', True, 'b/ => d/', 'renamed'),

                          ('a-id', 'd/a', True, 'a => d/a', 'renamed'),

                         ], values)

    def test_filelist_modified(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a'])

        tree.add(['a'], ['a-id'])

        rev_id1 = tree.commit('one')

        self.build_tree_contents([('tree/a', 'new contents for a\n')])

        dlg = commit.CommitDialog(tree)

        values = [(r[0], r[1], r[2], r[3], r[4]) for r in dlg._files_store]

        self.assertEqual([(None, None, True, 'All Files', ''),

                          ('a-id', 'a', True, 'a', 'modified'),

                         ], values)

    def test_filelist_renamed_and_modified(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b/', 'tree/b/c'])

        tree.add(['a', 'b', 'b/c'], ['a-id', 'b-id', 'c-id'])

        rev_id1 = tree.commit('one')

        tree.rename_one('b', 'd')

        tree.rename_one('a', 'd/a')

        self.build_tree_contents([('tree/d/a', 'new contents for a\n'),

                                  ('tree/d/c', 'new contents for c\n'),

                                 ])

        # 'c' is not considered renamed, because only its parent was moved, it

        # stayed in the same directory

        dlg = commit.CommitDialog(tree)

        values = [(r[0], r[1], r[2], r[3], r[4]) for r in dlg._files_store]

        self.assertEqual([(None, None, True, 'All Files', ''),

                          ('b-id', 'd', True, 'b/ => d/', 'renamed'),

                          ('a-id', 'd/a', True, 'a => d/a', 'renamed and modified'),

                          ('c-id', 'd/c', True, 'd/c', 'modified'),

                         ], values)

    def test_filelist_kind_changed(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b'])

        tree.add(['a', 'b'], ['a-id', 'b-id'])

        tree.commit('one')

        os.remove('tree/a')

        self.build_tree(['tree/a/'])

        # XXX:  This is technically valid, and the file list handles it fine,

        #       but 'show_diff_trees()' does not, so we skip this part of the

        #       test for now.

        # tree.rename_one('b', 'c')

        # os.remove('tree/c')

        # self.build_tree(['tree/c/'])

        dlg = commit.CommitDialog(tree)

        values = [(r[0], r[1], r[2], r[3], r[4]) for r in dlg._files_store]

        self.assertEqual([(None, None, True, 'All Files', ''),

                          ('a-id', 'a', True, 'a => a/', 'kind changed'),

                          # ('b-id', 'c', True, 'b => c/', 'renamed and modified'),

                         ], values)

    def test_filelist_removed(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b/'])

        tree.add(['a', 'b'], ['a-id', 'b-id'])

        tree.commit('one')

        os.remove('tree/a')

        tree.remove('b', force=True)

        dlg = commit.CommitDialog(tree)

        values = [(r[0], r[1], r[2], r[3], r[4]) for r in dlg._files_store]

        self.assertEqual([(None, None, True, 'All Files', ''),

                          ('a-id', 'a', True, 'a', 'removed'),

                          ('b-id', 'b', True, 'b/', 'removed'),

                         ], values)

    def test_diff_view(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b'])

        tree.add(['a', 'b'], ['a-id', 'b-id'])

        tree.commit('one')

        self.build_tree_contents([('tree/a', 'new contents for a\n')])

        tree.remove('b')

        dlg = commit.CommitDialog(tree)

        diff_buffer = dlg._diff_view.buffer

        text = diff_buffer.get_text(diff_buffer.get_start_iter(),

                                    diff_buffer.get_end_iter()).splitlines(True)

        self.assertEqual("=== removed file 'b'\n", text[0])

        self.assertContainsRe(text[1],

            r"--- b\t\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d [+-]\d\d\d\d")

        self.assertEqual('+++ b\t1970-01-01 00:00:00 +0000\n', text[2])

        self.assertEqual('@@ -1,1 +0,0 @@\n', text[3])

        self.assertEqual('-contents of tree/b\n', text[4])

        self.assertEqual('\n', text[5])

        self.assertEqual("=== modified file 'a'\n", text[6])

        self.assertContainsRe(text[7],

            r"--- a\t\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d [+-]\d\d\d\d")

        self.assertContainsRe(text[8],

            r"\+\+\+ a\t\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d [+-]\d\d\d\d")

        self.assertEqual('@@ -1,1 +1,1 @@\n', text[9])

        self.assertEqual('-contents of tree/a\n', text[10])

        self.assertEqual('+new contents for a\n', text[11])

        self.assertEqual('\n', text[12])

        self.assertEqual('Diff for All Files', dlg._diff_label.get_text())

    def test_file_selection(self):

        """Several things should happen when a file has been selected."""

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b'])

        tree.add(['a', 'b'], ['a-id', 'b-id'])

        dlg = commit.CommitDialog(tree)

        diff_buffer = dlg._diff_view.buffer

        self.assertEqual('Diff for All Files', dlg._diff_label.get_text())

        self.assertEqual('File commit message',

                         dlg._file_message_expander.get_label())

        self.assertFalse(dlg._file_message_expander.get_expanded())

        self.assertFalse(dlg._file_message_expander.get_property('sensitive'))

        dlg._treeview_files.set_cursor((1,))

        self.assertEqual('Diff for a', dlg._diff_label.get_text())

        text = diff_buffer.get_text(diff_buffer.get_start_iter(),

                                    diff_buffer.get_end_iter()).splitlines(True)

        self.assertEqual("=== added file 'a'\n", text[0])

        self.assertContainsRe(text[1],

            r"--- a\t\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d [+-]\d\d\d\d")

        self.assertContainsRe(text[2],

            r"\+\+\+ a\t\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d [+-]\d\d\d\d")

        self.assertEqual('@@ -0,0 +1,1 @@\n', text[3])

        self.assertEqual('+contents of tree/a\n', text[4])

        self.assertEqual('\n', text[5])

        self.assertEqual('Commit message for a',

                         dlg._file_message_expander.get_label())

        self.assertTrue(dlg._file_message_expander.get_expanded())

        self.assertTrue(dlg._file_message_expander.get_property('sensitive'))

        dlg._treeview_files.set_cursor((2,))

        self.assertEqual('Diff for b', dlg._diff_label.get_text())

        text = diff_buffer.get_text(diff_buffer.get_start_iter(),

                                    diff_buffer.get_end_iter()).splitlines(True)

        self.assertEqual("=== added file 'b'\n", text[0])

        self.assertContainsRe(text[1],

            r"--- b\t\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d [+-]\d\d\d\d")

        self.assertContainsRe(text[2],

            r"\+\+\+ b\t\d\d\d\d-\d\d-\d\d \d\d:\d\d:\d\d [+-]\d\d\d\d")

        self.assertEqual('@@ -0,0 +1,1 @@\n', text[3])

        self.assertEqual('+contents of tree/b\n', text[4])

        self.assertEqual('\n', text[5])

        self.assertEqual('Commit message for b',

                         dlg._file_message_expander.get_label())

        self.assertTrue(dlg._file_message_expander.get_expanded())

        self.assertTrue(dlg._file_message_expander.get_property('sensitive'))

        dlg._treeview_files.set_cursor((0,))

        self.assertEqual('Diff for All Files', dlg._diff_label.get_text())

        self.assertEqual('File commit message',

                         dlg._file_message_expander.get_label())

        self.assertFalse(dlg._file_message_expander.get_expanded())

        self.assertFalse(dlg._file_message_expander.get_property('sensitive'))

    def test_file_selection_message(self):

        """Selecting a file should bring up its commit message."""

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b/'])

        tree.add(['a', 'b'], ['a-id', 'b-id'])

        def get_file_text():

            buf = dlg._file_message_text_view.get_buffer()

            return buf.get_text(buf.get_start_iter(), buf.get_end_iter())

        def get_saved_text(path):

            """Get the saved text for a given record."""

            return dlg._files_store.get_value(dlg._files_store.get_iter(path), 5)

        dlg = commit.CommitDialog(tree)

        self.assertEqual('File commit message',

                         dlg._file_message_expander.get_label())

        self.assertFalse(dlg._file_message_expander.get_expanded())

        self.assertFalse(dlg._file_message_expander.get_property('sensitive'))

        self.assertEqual('', get_file_text())

        dlg._treeview_files.set_cursor((1,))

        self.assertEqual('Commit message for a',

                         dlg._file_message_expander.get_label())

        self.assertTrue(dlg._file_message_expander.get_expanded())

        self.assertTrue(dlg._file_message_expander.get_property('sensitive'))

        self.assertEqual('', get_file_text())

        self.assertEqual('', get_saved_text(1))

        dlg._file_message_text_view.get_buffer().set_text('Some text\nfor a\n')

        dlg._save_current_file_message()

        # We should have updated the ListStore with the new file commit info

        self.assertEqual('Some text\nfor a\n', get_saved_text(1))

        dlg._treeview_files.set_cursor((2,))

        self.assertEqual('Commit message for b/',

                         dlg._file_message_expander.get_label())

        self.assertTrue(dlg._file_message_expander.get_expanded())

        self.assertTrue(dlg._file_message_expander.get_property('sensitive'))

        self.assertEqual('', get_file_text())

        self.assertEqual('', get_saved_text(2))

        dlg._file_message_text_view.get_buffer().set_text('More text\nfor b\n')

        # Now switch back to 'a'. The message should be saved, and the buffer

        # should be updated with the other text

        dlg._treeview_files.set_cursor((1,))

        self.assertEqual('More text\nfor b\n', get_saved_text(2))

        self.assertEqual('Commit message for a',

                         dlg._file_message_expander.get_label())

        self.assertTrue(dlg._file_message_expander.get_expanded())

        self.assertTrue(dlg._file_message_expander.get_property('sensitive'))

        self.assertEqual('Some text\nfor a\n', get_file_text())

    def test_toggle_all_files(self):

        """When checking the All Files entry, it should toggle all fields"""

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b/'])

        tree.add(['a', 'b'], ['a-id', 'b-id'])

        dlg = commit.CommitDialog(tree)

        self.assertEqual([(None, None, True),

                          ('a-id', 'a', True),

                          ('b-id', 'b', True),

                         ], [(r[0], r[1], r[2]) for r in dlg._files_store])

        # TODO: jam 20071002 I'm not sure how to exactly trigger a toggle, it

        #       looks like we need to call renderer.activate() and pass an

        #       event and widget, and lots of other stuff I'm not sure what to

        #       do with. So instead, we just call toggle directly, and assume

        #       that toggle is hooked in correctly

        # column = dlg._treeview_files.get_column(0)

        # renderer = column.get_cell_renderers()[0]

        # Toggle a single entry should set just that entry to False

        dlg._toggle_commit(None, 1, dlg._files_store)

        self.assertEqual([(None, None, True),

                          ('a-id', 'a', False),

                          ('b-id', 'b', True),

                         ], [(r[0], r[1], r[2]) for r in dlg._files_store])

        # Toggling the main entry should set all entries

        dlg._toggle_commit(None, 0, dlg._files_store)

        self.assertEqual([(None, None, False),

                          ('a-id', 'a', False),

                          ('b-id', 'b', False),

                         ], [(r[0], r[1], r[2]) for r in dlg._files_store])

        dlg._toggle_commit(None, 2, dlg._files_store)

        self.assertEqual([(None, None, False),

                          ('a-id', 'a', False),

                          ('b-id', 'b', True),

                         ], [(r[0], r[1], r[2]) for r in dlg._files_store])

        dlg._toggle_commit(None, 0, dlg._files_store)

        self.assertEqual([(None, None, True),

                          ('a-id', 'a', True),

                          ('b-id', 'b', True),

                         ], [(r[0], r[1], r[2]) for r in dlg._files_store])

class TestCommitDialog_Commit(tests.TestCaseWithTransport):

    """Tests on the actual 'commit' button being pushed."""

    def test_commit_no_message(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a', 'tree/b'])

        tree.add(['a'], ['a-id'])

        rev_id = tree.commit('one')

        tree.add(['b'], ['b-id'])

        dlg = commit.CommitDialog(tree)

        questions = []

        def _question_cancel(*args):

            questions.append(args)

            questions.append('NO')

            return gtk.RESPONSE_NO

        def _question_ok(*args):

            questions.append(args)

            questions.append('OK')

            return gtk.RESPONSE_OK

        dlg._question_dialog = _question_cancel

        dlg._do_commit()

        self.assertEqual(

            [('Commit with an empty message?',

              'You can describe your commit intent in the message.'),

              'NO',

            ], questions)

        # By saying NO, nothing should be committed.

        self.assertEqual(rev_id, tree.last_revision())

        self.assertIs(None, dlg.committed_revision_id)

        dlg._question_dialog = _question_ok

        del questions[:]

        dlg._do_commit()

        self.assertEqual(

            [('Commit with an empty message?',

              'You can describe your commit intent in the message.'),

              'OK',

            ], questions)

        committed = tree.last_revision()

        self.assertNotEqual(rev_id, committed)

        self.assertEqual(committed, dlg.committed_revision_id)

    def test_initial_commit(self):

        tree = self.make_branch_and_tree('tree')

        self.build_tree(['tree/a'])

        tree.add(['a'], ['a-id'])

        dlg = commit.CommitDialog(tree)

        dlg._global_message_text_view.get_buffer().set_text('Some text\n')

        dlg._do_commit()

        last_rev = tree.last_revision()

        self.assertEqual(last_rev, dlg.committed_revision_id)

        rev = tree.branch.repository.get_revision(last_rev)

        self.assertEqual(last_rev, rev.revision_id)

        self.assertEqual('Some text\n', rev.message)