AMBARI-20823. Remove user input from invalid renderer error message (Attila Magyar via adoroszlai)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/3c844cad Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/3c844cad Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/3c844cad Branch: refs/heads/branch-2.5 Commit: 3c844cad8dafbb5b68a7d0c96ee64896ae492840 Parents: 2daa20e Author: Attila Magyar <amag...@hortonworks.com> Authored: Tue Apr 25 21:36:13 2017 +0200 Committer: Attila Doroszlai <adorosz...@hortonworks.com> Committed: Tue Apr 25 21:36:45 2017 +0200 ---------------------------------------------------------------------- .../ambari/server/api/resources/BaseResourceDefinition.java | 3 +-- .../ambari/server/api/resources/BaseResourceDefinitionTest.java | 2 +- 2 files changed, 2 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/3c844cad/ambari-server/src/main/java/org/apache/ambari/server/api/resources/BaseResourceDefinition.java ---------------------------------------------------------------------- diff --git a/ambari-server/src/main/java/org/apache/ambari/server/api/resources/BaseResourceDefinition.java b/ambari-server/src/main/java/org/apache/ambari/server/api/resources/BaseResourceDefinition.java index aaeed43..a11b837 100644 --- a/ambari-server/src/main/java/org/apache/ambari/server/api/resources/BaseResourceDefinition.java +++ b/ambari-server/src/main/java/org/apache/ambari/server/api/resources/BaseResourceDefinition.java @@ -132,8 +132,7 @@ public abstract class BaseResourceDefinition implements ResourceDefinition { || name.contains("zero_padding")) { return new MetricsPaddingRenderer(name); } else { - throw new IllegalArgumentException("Invalid renderer name: " + name + - " for resource of type: " + m_type); + throw new IllegalArgumentException("Invalid renderer name for resource of type " + m_type); } } http://git-wip-us.apache.org/repos/asf/ambari/blob/3c844cad/ambari-server/src/test/java/org/apache/ambari/server/api/resources/BaseResourceDefinitionTest.java ---------------------------------------------------------------------- diff --git a/ambari-server/src/test/java/org/apache/ambari/server/api/resources/BaseResourceDefinitionTest.java b/ambari-server/src/test/java/org/apache/ambari/server/api/resources/BaseResourceDefinitionTest.java index 46f3ef7..7342411 100644 --- a/ambari-server/src/test/java/org/apache/ambari/server/api/resources/BaseResourceDefinitionTest.java +++ b/ambari-server/src/test/java/org/apache/ambari/server/api/resources/BaseResourceDefinitionTest.java @@ -140,7 +140,7 @@ public class BaseResourceDefinitionTest { resource.getRenderer("foo"); fail("Should have thrown an exception due to invalid renderer type"); } catch (IllegalArgumentException e) { - // expected + assertEquals("Invalid renderer name for resource of type Service", e.getMessage()); } }