Merge lp:~jelmer/brz/remaining-test into lp:brz

Proposed by Jelmer Vernooij
Status: Merged
Approved by: Jelmer Vernooij
Approved revision: no longer in the source branch.
Merge reported by: The Breezy Bot
Merged at revision: not available
Proposed branch: lp:~jelmer/brz/remaining-test
Merge into: lp:brz
Diff against target: 51 lines (+10/-13)
2 files modified
breezy/symbol_versioning.py (+9/-12)
python3.passing (+1/-1)
To merge this branch: bzr merge lp:~jelmer/brz/remaining-test
Reviewer Review Type Date Requested Status
Martin Packman Approve
Review via email: mp+354803@code.launchpad.net

Commit message

Fix remaining deprecated function name test.

Description of the change

Fix remaining deprecated function name test.

To post a comment you must log in.
Revision history for this message
Martin Packman (gz) wrote :

This is still more complex that it needs to be for how it's used, but fixed counts as fixed for now. :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'breezy/symbol_versioning.py'
--- breezy/symbol_versioning.py 2018-09-02 21:15:23 +0000
+++ breezy/symbol_versioning.py 2018-09-12 20:34:43 +0000
@@ -76,19 +76,16 @@
76 have a single %s operator in it. a_callable will be turned into a nice76 have a single %s operator in it. a_callable will be turned into a nice
77 python symbol and then substituted into deprecation_version.77 python symbol and then substituted into deprecation_version.
78 """78 """
79 if PY3:79 if getattr(a_callable, '__self__', None) is not None:
80 func = getattr(a_callable, '__func__', a_callable)80 symbol = "%s.%s.%s" % (a_callable.__self__.__class__.__module__,
81 symbol ='%s.%s' % (a_callable.__module__,81 a_callable.__self__.__class__.__name__,
82 a_callable.__qualname__)82 a_callable.__name__)
83 elif getattr(a_callable, '__qualname__', None) is not None and not '<' in a_callable.__qualname__:
84 symbol = "%s.%s" % (a_callable.__module__,
85 a_callable.__qualname__)
83 else:86 else:
84 if getattr(a_callable, '__self__', None) is None:87 symbol = "%s.%s" % (a_callable.__module__,
85 symbol = "%s.%s" % (a_callable.__module__,88 a_callable.__name__)
86 a_callable.__name__)
87 else:
88 symbol = "%s.%s.%s" % (a_callable.__self__.__class__.__module__,
89 a_callable.__self__.__class__.__name__,
90 a_callable.__name__
91 )
92 return deprecation_version % symbol89 return deprecation_version % symbol
9390
9491
9592
=== modified file 'python3.passing'
--- python3.passing 2018-09-11 07:31:44 +0000
+++ python3.passing 2018-09-12 20:34:43 +0000
@@ -1,6 +1,5 @@
1# This is the list of tests that are known to pass with Python3.1# This is the list of tests that are known to pass with Python3.
2# "make check-nodocs3" verifies that these pass.2# "make check-nodocs3" verifies that these pass.
3breezy.git.tests.test_workingtree.GitWorkingTreeTests.test_is_ignored_directory
4breezy.git.tests.test_blackbox.ShallowTests.test_log_shallow3breezy.git.tests.test_blackbox.ShallowTests.test_log_shallow
5breezy.git.tests.test_blackbox.ShallowTests.test_version_info_custom_without_revno4breezy.git.tests.test_blackbox.ShallowTests.test_version_info_custom_without_revno
6breezy.git.tests.test_blackbox.ShallowTests.test_version_info_custom_with_revno5breezy.git.tests.test_blackbox.ShallowTests.test_version_info_custom_with_revno
@@ -29012,6 +29011,7 @@
29012breezy.tests.test_selftest.TestConvenienceMakers.test_make_tree_for_local_vfs_backed_transport29011breezy.tests.test_selftest.TestConvenienceMakers.test_make_tree_for_local_vfs_backed_transport
29013breezy.tests.test_selftest.TestCounterHooks.test_no_hook29012breezy.tests.test_selftest.TestCounterHooks.test_no_hook
29014breezy.tests.test_selftest.TestCounterHooks.test_run_hook_once29013breezy.tests.test_selftest.TestCounterHooks.test_run_hook_once
29014breezy.tests.test_selftest.TestDeprecations.test_applyDeprecated_not_deprecated
29015breezy.tests.test_selftest.TestDeprecations.test_callDeprecated29015breezy.tests.test_selftest.TestDeprecations.test_callDeprecated
29016breezy.tests.test_selftest.TestDocTestSuiteIsolation.test_deleted_variable29016breezy.tests.test_selftest.TestDocTestSuiteIsolation.test_deleted_variable
29017breezy.tests.test_selftest.TestDocTestSuiteIsolation.test_injected_variable29017breezy.tests.test_selftest.TestDocTestSuiteIsolation.test_injected_variable

Subscribers

People subscribed via source and target branches