commit 32add5d67a9ba2f003f60ac2e1af4a12cb43725d
author Brian Norris <briannorris@chromium.org> Mon Jun 19 11:09:54 2017 -0700
committer chrome-bot <chrome-bot@chromium.org> Tue Jun 20 11:30:26 2017 -0700
tree 514ad13427f793d003124ef63a6003ab28414aa4
parent c790e144d2f4950dedc9c33737b2b5eb92c91e31

dynamic_suite: make deprecation errors more useful

This:

 "UserWarning: Calling method "create_fs_getter" from Suite is deprecated"

is much more useful than:

 "UserWarning: Calling this method from Suite is deprecated"

BUG=chromium:734662
TEST=test_that, while using deprecated Suite-level functions

Change-Id: I2d234bda6bce2cddf3754a7c3b216ea51c641bfc
Signed-off-by: Brian Norris <briannorris@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/540117
Reviewed-by: Allen Li <ayatane@chromium.org>

server/cros/dynamic_suite/suite.py

diff --git a/server/cros/dynamic_suite/suite.py b/server/cros/dynamic_suite/suite.py
index dfa30fb..d366cfd 100644
--- a/server/cros/dynamic_suite/suite.py
+++ b/server/cros/dynamic_suite/suite.py
@@ -918,7 +918,8 @@
     @functools.wraps(func)
     def wrapper(*args, **kwargs):
         """Wraps |func| for warning."""
-        warnings.warn('Calling this method from Suite is deprecated')
+        warnings.warn('Calling method "%s" from Suite is deprecated' %
+                      func.__name__)
         return func(*args, **kwargs)
     return staticmethod(wrapper)