Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(684)

Unified Diff: recipe_engine/unittests/loader_test.py

Issue 1755593003: Fix param_name for Properties. (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/recipes-py@master
Patch Set: Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « recipe_engine/loader.py ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: recipe_engine/unittests/loader_test.py
diff --git a/recipe_engine/unittests/loader_test.py b/recipe_engine/unittests/loader_test.py
index af2d96de1f52515a4ff9b1ff4419b99c08467783..781e72c816c41034e812921104693fdcd891c004 100755
--- a/recipe_engine/unittests/loader_test.py
+++ b/recipe_engine/unittests/loader_test.py
@@ -78,18 +78,18 @@ class TestInvoke(unittest.TestCase):
self.assertEqual(1, self.invoke(func, props, prop_defs, ['a', 'b']))
def testInvokeParamName(self):
- """Tests invoke with two different properties."""
- def func(a):
- return a
+ """Tests invoke with a param name."""
+ def func(c):
+ return c
prop_defs = {
- 'a': make_prop(name='b'),
+ 'b.a': make_prop(name="c", param_name="c"),
}
props = {
- 'b': 2,
+ 'b.a': 2,
}
- self.assertEqual(2, self.invoke(func, props, prop_defs, ['a']))
+ self.assertEqual(2, self.invoke(func, props, prop_defs, ['c']))
def testInvokeClass(self):
"""Tests invoking a class."""
« no previous file with comments | « recipe_engine/loader.py ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698