Rev 5422: Fix style issues, including vertical spaces, lines too long and multi lines imports. in file:///home/vila/src/bzr/reviews/mergetools/
Vincent Ladeuil
v.ladeuil+lp at free.fr
Mon Dec 6 10:01:50 GMT 2010
At file:///home/vila/src/bzr/reviews/mergetools/
------------------------------------------------------------
revno: 5422
revision-id: v.ladeuil+lp at free.fr-20101206100150-jakhpj9ud2exxplq
parent: gordon at doxxx.net-20101204055739-703a0zhdpp4hjhov
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: mergetools
timestamp: Mon 2010-12-06 11:01:50 +0100
message:
Fix style issues, including vertical spaces, lines too long and multi lines imports.
-------------- next part --------------
=== modified file 'bzrlib/mergetools.py'
--- a/bzrlib/mergetools.py 2010-12-02 05:08:52 +0000
+++ b/bzrlib/mergetools.py 2010-12-06 10:01:50 +0000
@@ -59,6 +59,7 @@
class MergeTool(object):
+
def __init__(self, name, commandline):
"""Initializes the merge tool with a name and a command-line (a string
or sequence of strings).
=== modified file 'bzrlib/tests/features.py'
--- a/bzrlib/tests/features.py 2010-11-26 14:55:58 +0000
+++ b/bzrlib/tests/features.py 2010-12-06 10:01:50 +0000
@@ -19,7 +19,10 @@
import os
import stat
-from bzrlib import osutils, tests
+from bzrlib import (
+ osutils,
+ tests,
+ )
from bzrlib.symbol_versioning import deprecated_in
=== modified file 'bzrlib/tests/test_cmdline.py'
--- a/bzrlib/tests/test_cmdline.py 2010-11-27 05:03:48 +0000
+++ b/bzrlib/tests/test_cmdline.py 2010-12-06 10:01:50 +0000
@@ -116,6 +116,7 @@
class TestUnSplit(tests.TestCase):
+
def test_unsplit(self):
self.assertEquals('foo bar', cmdline.unsplit(['foo', 'bar']))
=== modified file 'bzrlib/tests/test_mergetools.py'
--- a/bzrlib/tests/test_mergetools.py 2010-12-02 04:57:27 +0000
+++ b/bzrlib/tests/test_mergetools.py 2010-12-06 10:01:50 +0000
@@ -27,6 +27,7 @@
class TestBasics(tests.TestCase):
+
def setUp(self):
super(TestBasics, self).setUp()
self.tool = mergetools.MergeTool('sometool',
@@ -103,14 +104,18 @@
class TestUnicodeBasics(tests.TestCase):
+
def setUp(self):
super(TestUnicodeBasics, self).setUp()
- self.tool = mergetools.MergeTool(u'someb\u0414r',
- u'/path/to/b\u0414r --opt {base} -x {this} {other} --stuff {result}')
+ self.tool = mergetools.MergeTool(
+ u'someb\u0414r',
+ u'/path/to/b\u0414r --opt {base} -x {this} {other}'
+ ' --stuff {result}')
def test_get_commandline(self):
self.assertEqual(
- u'/path/to/b\u0414r --opt {base} -x {this} {other} --stuff {result}',
+ u'/path/to/b\u0414r --opt {base} -x {this} {other}'
+ ' --stuff {result}',
self.tool.get_commandline())
def test_get_commandline_as_list(self):
@@ -179,6 +184,7 @@
class TestMergeToolOperations(tests.TestCaseInTempDir):
+
def test_filename_substitution(self):
def dummy_invoker(executable, args, cleanup):
self._commandline = [executable] + args
@@ -247,6 +253,7 @@
class TestModuleFunctions(tests.TestCaseInTempDir):
+
def test_detect(self):
# only way to reliably test detection is to add a known existing
# executable to the list used for detection
=== modified file 'bzrlib/tests/test_osutils.py'
--- a/bzrlib/tests/test_osutils.py 2010-12-04 05:40:58 +0000
+++ b/bzrlib/tests/test_osutils.py 2010-12-06 10:01:50 +0000
@@ -2119,6 +2119,7 @@
class TestFindExecutableInPath(tests.TestCase):
+
def test_windows(self):
if sys.platform != 'win32':
raise tests.TestSkipped('test requires win32')
More information about the bazaar-commits
mailing list