Bug 974501 - Its possible to create a dynaGroup definition without expression via CLI
Summary: Its possible to create a dynaGroup definition without expression via CLI
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: JBoss Operations Network
Classification: JBoss
Component: CLI
Version: JON 3.2
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: DR04
: JON 3.2.0
Assignee: Jirka Kremser
QA Contact: Mike Foley
URL:
Whiteboard:
Depends On:
Blocks: 880819
TreeView+ depends on / blocked
 
Reported: 2013-06-14 09:50 UTC by Filip Brychta
Modified: 2014-01-02 20:38 UTC (History)
3 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed:
Type: Bug
Embargoed:


Attachments (Terms of Use)

Description Filip Brychta 2013-06-14 09:50:56 UTC
Description of problem:
Its possible to pass null or empty string as a expression during creation of dynaGroup definition. Dynagroup definition is created without expression. This is not possible via GUI.

Version-Release number of selected component (if applicable):
Version: 4.8.0-SNAPSHOT
Build Number: 7b31b01

How reproducible:
Always

Steps to Reproduce:
1. ./rhq-cli.sh -p rhqadmin -u rhqadmin
2. dynaGroupDef.setExpression(null);  // or empty string ("") or don't use this setter at all
3. GroupDefinitionManager.createGroupDefinition(dynaGroupDef)

Actual results:
no exception and definition is created without expression

Expected results:
Exception with informative message (i.e. expression is required) is thrown and definition is not created.

Comment 1 Mike Foley 2013-07-02 15:19:56 UTC
moved to JON 3.3 per 7/2/2013 meeting with crouch, larry, heiko, foley

Comment 2 Jirka Kremser 2013-07-10 15:11:18 UTC
branch:  master
link:    http://git.fedorahosted.org/cgit/rhq/rhq.git/commit/?id=627c48a7a
time:    2013-07-10 17:10:25 +0200
commit:  627c48a7a18a03d9a91032a32c1833d4fcbae86a
author:  Jirka Kremser - jkremser
message: [BZ 974501] - Its possible to create a dynaGroup definition without
         expression via CLI - check whether expression is null of an
         empty string

Comment 3 Simeon Pinder 2013-08-26 15:36:22 UTC
Moving to ON_QA as available for testing in latest 3.2.x ALPHA 58 build:

https://brewweb.devel.redhat.com//buildinfo?buildID=291947

Comment 4 Filip Brychta 2013-08-27 15:24:53 UTC
Verified on 
Version: 3.2.0.ALPHA_QA
Build Number: 10b0467:adad71f


Note You need to log in before you can comment on or make changes to this bug.