Updated wildfly cli version to 3.1.0.Final

Author: Amit Chaudhary

pom.xml

@@ -7,7 +7,7 @@
     <modelVersion>4.0.0</modelVersion>
     <groupId>uk.co.techblue</groupId>
     <artifactId>jboss-controller-operation-executor</artifactId>
-    <version>2.1-SNAPSHOT</version>
+    <version>3.0</version>
     <name>JBoss AS 7 controller operation executor</name>
     <description>Client API  used to execute operations on JBoss AS 7 controller interface.</description>
 
@@ -62,7 +62,7 @@
         <dependency>
             <groupId>org.wildfly.core</groupId>
             <artifactId>wildfly-cli</artifactId>
-            <version>2.1.0.Final</version>
+            <version>3.1.0.Final</version>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>

src/main/java/uk/co/techblue/jboss/controller/ControllerOperationExecutor.java

@@ -23,7 +23,6 @@
 import uk.co.techblue.jboss.controller.vo.ControllerClientConfig;
 import uk.co.techblue.jboss.controller.vo.JndiDataSource;
 
-// TODO: Auto-generated Javadoc
 /**
  * A service that executes operations on an application server management model controller.
  * 

src/main/java/uk/co/techblue/jboss/controller/as7/JBoss7ControllerOpeartionExecutor.java

@@ -84,44 +84,15 @@
 public class JBoss7ControllerOpeartionExecutor implements ControllerOperationExecutor {
 
     /** The logger instance. */
-    private final Logger logger = LoggerFactory.getLogger(getClass());
-
-    /**
-     * The main method. ONLY FOR TESTING!
-     * 
-     * @param args the arguments
-     */
-    public static void main(String[] args) {
-        try {
-            final JndiDataSource dataSource = new JndiDataSource("java:/mysql-testjboss7", "jdbc:mysql://localhost:3306/test",
-                "com.mysql", "root", "root");
-            dataSource.setMaxPoolSize(10);
-            dataSource.setPoolPrefill(true);
-            final ControllerClientConfig clientConfig = new ControllerClientConfig("127.0.0.1");
-            // clientConfig.setUserName("ajay");
-            // clientConfig.setPassword("ajay");
-            // List<ModelNode> dataSources = new JBoss7ControllerOpeartionExecutor().getDatasources(clientConfig, "",
-            // DatasourceStatus.ALL);
-            // for (ModelNode dataSource : dataSources) {
-            // System.out.println(dataSource.asProperty().getName());
-            // }
-            // new JBoss7ControllerOpeartionExecutor().disableDataSource(clientConfig, "java:/mysql-testjboss7","");
-            new JBoss7ControllerOpeartionExecutor().removeDatasource(clientConfig, "java:/mysql-testjboss7", "");
-            // new JBoss7ControllerOpeartionExecutor().createDatasource(clientConfig, dataSource, true);
-            // new JBoss7ControllerOpeartionExecutor().isDatasourceExists(clientConfig, dataSource.getJndiName());
-            // new JBoss7ControllerOpeartionExecutor().createDatasource(clientConfig, dataSource, true, "ha");
-            // new JBoss7ControllerOpeartionExecutor().isDatasourceExists(clientConfig, dataSource.getJndiName(), "full-ha");
-        } catch (Exception e) {
-            e.printStackTrace();
-        }
-    }
+    private final Logger logger = LoggerFactory.getLogger(JBoss7ControllerOpeartionExecutor.class);
 
     /*
      * (non-Javadoc)
      * 
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#createDatasource(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, uk.co.techblue.jboss.controller.vo.JndiDataSource, boolean, java.lang.String[])
      */
+    @Override
     public void createDatasource(final ControllerClientConfig controllerClientConfig, final JndiDataSource dataSource,
         final boolean enable, final String... serverProfileNames) throws ControllerOperationException {
         if (serverProfileNames != null && serverProfileNames.length > 0) {
@@ -202,6 +173,7 @@ private void createDatasource(final ControllerClientConfig controllerClientConfi
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#removeDatasource(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, uk.co.techblue.jboss.controller.vo.JndiDataSource, java.lang.String[])
      */
+    @Override
     public void removeDatasource(final ControllerClientConfig controllerClientConfig, final String datasourceName,
         final String... serverProfileNames) throws ControllerOperationException {
         if (serverProfileNames != null && serverProfileNames.length > 0) {
@@ -378,6 +350,7 @@ private static boolean isLocalIpAddress(final InetAddress address) {
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#isDatasourceExists(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, java.lang.String)
      */
+    @Override
     public boolean isDatasourceExists(final ControllerClientConfig controllerClientConfig, final String dataSourceName)
         throws ControllerOperationException {
         return isDatasourceExists(controllerClientConfig, dataSourceName, null);
@@ -389,6 +362,7 @@ public boolean isDatasourceExists(final ControllerClientConfig controllerClientC
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#isDatasourceExists(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, java.lang.String, java.lang.String)
      */
+    @Override
     public boolean isDatasourceExists(final ControllerClientConfig controllerClientConfig, final String dataSourceName,
         final String serverProfileName) throws ControllerOperationException {
         logger.info("Checking if datasource '{}' exists...", dataSourceName);
@@ -416,6 +390,7 @@ public boolean isDatasourceExists(final ControllerClientConfig controllerClientC
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#enableDataSource(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, java.lang.String, java.lang.String[])
      */
+    @Override
     public void enableDataSource(final ControllerClientConfig controllerClientConfig, final String datasourceName,
         final String... serverProfileNames) throws ControllerOperationException {
         if (serverProfileNames != null && serverProfileNames.length > 0) {
@@ -591,6 +566,7 @@ private List<ModelNode> getFilteredDataSources(final ControllerClientConfig cont
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#isDatasourceEnabled(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, java.lang.String, java.lang.String)
      */
+    @Override
     public boolean isDatasourceEnabled(final ControllerClientConfig controllerClientConfig, final String serverProfileName,
         final String datasource) throws ControllerOperationException {
         final ModelNode request = new ModelNode();
@@ -745,6 +721,7 @@ public void disableDataSources(final ControllerClientConfig controllerClientConf
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#createDatasources(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, java.util.List, boolean, java.lang.String[])
      */
+    @Override
     public void createDatasources(final ControllerClientConfig controllerClientConfig, final List<JndiDataSource> dataSources,
         final boolean enable, final String... serverProfileNames) throws ControllerOperationException {
 
@@ -775,6 +752,7 @@ public void createDatasources(final ControllerClientConfig controllerClientConfi
      * @see uk.co.techblue.jboss.controller.ControllerOperationExecutor#removeDatasources(uk.co.techblue.jboss.controller.vo.
      * ControllerClientConfig, java.util.List, java.lang.String[])
      */
+    @Override
     public void removeDatasources(final ControllerClientConfig controllerClientConfig, final List<String> datasourceNames,
         final String... serverProfileNames) throws ControllerOperationException {