Bug 1294010

Created attachment 1109124 [details]
reproducer

Description of problem:
The query argument does not work with MBeanServerConnection#queryNames( https://docs.oracle.com/javase/7/docs/api/javax/management/MBeanServerConnection.html#queryNames%28javax.management.ObjectName,%20javax.management.QueryExp%29).

Version-Release number of selected component (if applicable):
EAP 6.4.5.

How reproducible:
Using MBeanServerConnection on EAP 6.

Steps to Reproduce:
1. Deploye test.war.
2. Execute "http://localhost:8080/test/test.jsp".

Actual results:
You can get all object by MBeanServerConnection#queryNames(null, query) even if it is set a QueryExp object as query argument.

- test.jsp.
~~
      QueryExp exp = Query.eq(Query.attr("test-attr"), Query.value("1234"));
      Set<ObjectName> names = mbsc.queryNames(null, exp);
~~

Expected results:
The query argument should have to be available in MBeanServerConnection.

Additional info:
As long as I checked source code of org.jboss.as.jmx.model.ModelControllerMBeanHelper.queryNames(ObjectName, QueryExp), there was no implementation for the query argument(It was TODO on source code).

- ModelControllerMBeanHelper#queryNames
~~~
    Set<ObjectName> queryNames(final ObjectName name, final QueryExp query) {
        return new RootResourceIterator<Set<ObjectName>>(accessControlUtil, getRootResourceAndRegistration().getResource(),
                new ObjectNameMatchResourceAction<Set<ObjectName>>(name) {

            Set<ObjectName> set = new HashSet<ObjectName>();

            @Override
            public boolean onResource(ObjectName resourceName) {
                if (name == null || name.apply(resourceName)) {
                    //TODO check query
                    set.add(resourceName);
                }
                return true;
            }

            @Override
            public Set<ObjectName> getResult() {
                if (set.size() == 1 && set.contains(ModelControllerMBeanHelper.createRootObjectName(domain))) {
                  return Collections.emptySet();
                }
                return set;
            }
        }).iterate();
~~~