updates..

Author: John Donovan

src/com/gtnexus/appxpress/cli/option/BuilderOptionInterpreter.java

@@ -1,7 +1,5 @@
 package com.gtnexus.appxpress.cli.option;
 
-import static com.gtnexus.appxpress.AppXpressConstants.ZIP_EXTENSION;
-
 import java.io.File;
 import java.nio.file.Path;
 import java.util.Arrays;

src/com/gtnexus/appxpress/pmextractor/gitmap/GitMapper.java

@@ -55,18 +55,6 @@ public static GitMapper createMapper(AppXpressContext<ExtractorOption> context)
 		return new GitMapper(context);
 	}
 
-	
-//	public static GitMapper createMapper(Map<ExtractorOption, String> optionMap) {
-//		if (optionMap.containsKey(ExtractorOption.PLATFORM_ZIP)) {
-//			String platformZip = optionMap.get(ExtractorOption.PLATFORM_ZIP);
-//			if (!platformZip.endsWith(ZIP_EXTENSION)) {
-//				platformZip = platformZip + ZIP_EXTENSION;
-//				optionMap.put(ExtractorOption.PLATFORM_ZIP, platformZip);
-//			}
-//		}
-//		return new GitMapper(new GitMapVO(optionMap));
-//	}
-	
 	public GitMapper(AppXpressContext<ExtractorOption> context) {
 		this.ctx = context;
 		this.vo = new GitMapVO(context.getOptMap());
@@ -75,13 +63,6 @@ public GitMapper(AppXpressContext<ExtractorOption> context) {
 		this.fs = new FileService();
 	}
 
-//	public GitMapper(GitMapVO vo) {
-//		this.vo = vo;
-//		this.overwrittenScripts = new ArrayList<>();
-//		this.prep = new GitMapPrep();
-//		this.fs = new FileService();
-//	}
-
 	/**
 	 * Performs the appropriate actions for module extraction
 	 */

test/com/gtnexus/appxpress/cli/option/BuilderOptionInterpreterTest.java

@@ -31,10 +31,10 @@ public class BuilderOptionInterpreterTest {
 	PMProperties properties;
 	@Mock
 	PlatformSelector selector;
-	
+
 	@Mock
 	AppXpressDirResolver resolver;
-	
+
 	BuilderOptionInterpreter interpreter;
 	HashMap<BuilderOption, String> optMap = new HashMap<>();
 
@@ -44,21 +44,24 @@ public void setup() {
 	}
 
 	@Test(expected = NullPointerException.class)
-			public void testPerformCustomInterpretationWithNull() throws Exception {
-				interpreter.performCustomInterpretation(null);
-			}
+	public void testPerformCustomInterpretationWithNull() throws Exception {
+		interpreter.performCustomInterpretation(null);
+	}
 
 	@Test
-			public void testPerformCustomInterpretationWithoutSPFlag()
-					throws AppXpressException {
-				ParsedOptions<BuilderOption> parsedOptions = new ParsedOptions<>(new Options(), new HashMap<BuilderOption, String>(), EnumSet.allOf(BuilderOption.class));
-				BuilderOptionInterpreter interpeter = new BuilderOptionInterpreter(app, shutdown, parsedOptions, properties, selector, resolver);
-				fail("unimplemented");
-			}
+	public void testPerformCustomInterpretationWithoutSPFlag()
+			throws AppXpressException {
+		ParsedOptions<BuilderOption> parsedOptions = new ParsedOptions<>(
+				new Options(), new HashMap<BuilderOption, String>(),
+				EnumSet.allOf(BuilderOption.class));
+		BuilderOptionInterpreter interpeter = new BuilderOptionInterpreter(app,
+				shutdown, parsedOptions, properties, selector, resolver);
+		fail("unimplemented");
+	}
 
 	@Test
-			public void testPerformCustomInterpretationWithSPFlag()
-					throws AppXpressException {
-				fail("unimplemented");
-			}
+	public void testPerformCustomInterpretationWithSPFlag()
+			throws AppXpressException {
+		fail("unimplemented");
+	}
 }