Merge pull request #34 from viniciusam/feature/rename_base_package

Rename base package to org.utplsql

Author: viniciusam

pom.xml

@@ -2,7 +2,7 @@
   xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
   <modelVersion>4.0.0</modelVersion>
 
-  <groupId>io.github.utplsql</groupId>
+  <groupId>org.utplsql</groupId>
   <artifactId>cli</artifactId>
   <version>1.0-SNAPSHOT</version>
   <packaging>jar</packaging>
@@ -18,15 +18,15 @@
 
   <dependencies>
     <dependency>
-      <groupId>com.beust</groupId>
-      <artifactId>jcommander</artifactId>
-      <version>1.69</version>
+      <groupId>org.utplsql</groupId>
+      <artifactId>java-api</artifactId>
+      <version>1.0-SNAPSHOT</version>
       <scope>compile</scope>
     </dependency>
     <dependency>
-      <groupId>io.github.utplsql</groupId>
-      <artifactId>java-api</artifactId>
-      <version>1.0-SNAPSHOT</version>
+      <groupId>com.beust</groupId>
+      <artifactId>jcommander</artifactId>
+      <version>1.69</version>
       <scope>compile</scope>
     </dependency>
     <dependency>
@@ -50,7 +50,7 @@
           <repositoryLayout>flat</repositoryLayout>
           <programs>
             <program>
-              <mainClass>io.github.utplsql.cli.Cli</mainClass>
+              <mainClass>org.utplsql.cli.Cli</mainClass>
               <id>utplsql</id>
             </program>
           </programs>

src/main/java/io/github/utplsql/cli/Cli.java renamed to src/main/java/org/utplsql/cli/Cli.java

@@ -1,4 +1,4 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import com.beust.jcommander.JCommander;
 import com.beust.jcommander.Parameter;

src/main/java/io/github/utplsql/cli/ConnectionInfo.java renamed to src/main/java/org/utplsql/cli/ConnectionInfo.java

@@ -1,4 +1,4 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import com.beust.jcommander.ParameterException;
 

src/main/java/io/github/utplsql/cli/ConnectionStringConverter.java renamed to src/main/java/org/utplsql/cli/ConnectionStringConverter.java

@@ -1,4 +1,4 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import com.beust.jcommander.IStringConverter;
 

src/main/java/io/github/utplsql/cli/FileWalker.java renamed to src/main/java/org/utplsql/cli/FileWalker.java

@@ -1,4 +1,4 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import java.io.File;
 import java.util.ArrayList;

src/main/java/io/github/utplsql/cli/ReporterOptions.java renamed to src/main/java/org/utplsql/cli/ReporterOptions.java

@@ -1,6 +1,6 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
-import io.github.utplsql.api.reporter.Reporter;
+import org.utplsql.api.reporter.Reporter;
 
 /**
  * Created by Vinicius on 20/05/2017.

src/main/java/io/github/utplsql/cli/RunCommand.java renamed to src/main/java/org/utplsql/cli/RunCommand.java

@@ -1,11 +1,11 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import com.beust.jcommander.Parameter;
 import com.beust.jcommander.Parameters;
-import io.github.utplsql.api.*;
-import io.github.utplsql.api.exception.SomeTestsFailedException;
-import io.github.utplsql.api.reporter.Reporter;
-import io.github.utplsql.api.reporter.ReporterFactory;
+import org.utplsql.api.*;
+import org.utplsql.api.exception.SomeTestsFailedException;
+import org.utplsql.api.reporter.Reporter;
+import org.utplsql.api.reporter.ReporterFactory;
 
 import java.io.File;
 import java.io.FileNotFoundException;
@@ -202,17 +202,24 @@ public List<ReporterOptions> getReporterOptionsList() {
     public FileMapperOptions getMapperOptions(List<String> mappingParams, List<String> filePaths) {
         FileMapperOptions mapperOptions = new FileMapperOptions(filePaths);
 
+        final String OPT_OWNER="-owner=";
+        final String OPT_REGEX="-regex_expression=";
+        final String OPT_TYPE_MAPPING="-type_mapping=";
+        final String OPT_OWNER_SUBEX="-owner_subexpression=";
+        final String OPT_NAME_SUBEX="-name_subexpression=";
+        final String OPT_TYPE_SUBEX="-type_subexpression=";
+
         for (String p : mappingParams) {
-            if (p.startsWith("-object_owner=")) {
-                mapperOptions.setObjectOwner(p.substring("-object_owner=".length()));
+            if (p.startsWith(OPT_OWNER)) {
+                mapperOptions.setObjectOwner(p.substring(OPT_OWNER.length()));
             }
             else
-            if (p.startsWith("-regex_pattern=")) {
-                mapperOptions.setRegexPattern(p.substring("-regex_pattern=".length()));
+            if (p.startsWith(OPT_REGEX)) {
+                mapperOptions.setRegexPattern(p.substring(OPT_REGEX.length()));
             }
             else
-            if (p.startsWith("-type_mapping=")) {
-                String typeMappingsParam = p.substring("-type_mapping=".length());
+            if (p.startsWith(OPT_TYPE_MAPPING)) {
+                String typeMappingsParam = p.substring(OPT_TYPE_MAPPING.length());
 
                 List<KeyValuePair> typeMappings = new ArrayList<>();
                 for (String mapping : typeMappingsParam.split("/")) {
@@ -223,23 +230,20 @@ public FileMapperOptions getMapperOptions(List<String> mappingParams, List<Strin
                 mapperOptions.setTypeMappings(typeMappings);
             }
             else
-            if (p.startsWith("-owner_subexpression=")) {
-                mapperOptions.setOwnerSubExpression(Integer.parseInt(p.substring("-owner_subexpression=".length())));
+            if (p.startsWith(OPT_OWNER_SUBEX)) {
+                mapperOptions.setOwnerSubExpression(Integer.parseInt(p.substring(OPT_OWNER_SUBEX.length())));
             }
             else
-            if (p.startsWith("-name_subexpression=")) {
-                mapperOptions.setNameSubExpression(Integer.parseInt(p.substring("-name_subexpression=".length())));
+            if (p.startsWith(OPT_NAME_SUBEX)) {
+                mapperOptions.setNameSubExpression(Integer.parseInt(p.substring(OPT_NAME_SUBEX.length())));
             }
             else
-            if (p.startsWith("-type_subexpression=")) {
+            if (p.startsWith(OPT_TYPE_SUBEX)) {
                 mapperOptions.setTypeSubExpression(Integer.parseInt(p.substring("-type_subexpression=".length())));
             }
         }
 
-        if (mapperOptions.getRegexPattern() == null || mapperOptions.getRegexPattern().isEmpty())
-            return null;
-        else
-            return mapperOptions;
+        return mapperOptions;
     }
 
 }

src/test/java/io/github/utplsql/cli/ConnectionInfoTest.java renamed to src/test/java/org/utplsql/cli/ConnectionInfoTest.java

@@ -1,4 +1,4 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import com.beust.jcommander.ParameterException;
 import org.junit.Assert;

src/test/java/io/github/utplsql/cli/FileWalkerTest.java renamed to src/test/java/org/utplsql/cli/FileWalkerTest.java

@@ -1,4 +1,4 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import org.junit.Assert;
 import org.junit.Test;

src/test/java/io/github/utplsql/cli/RunCommandTest.java renamed to src/test/java/org/utplsql/cli/RunCommandTest.java

@@ -1,9 +1,9 @@
-package io.github.utplsql.cli;
+package org.utplsql.cli;
 
 import com.beust.jcommander.JCommander;
-import io.github.utplsql.api.CustomTypes;
 import org.junit.Assert;
 import org.junit.Test;
+import org.utplsql.api.CustomTypes;
 
 import java.util.List;