Set -deleteTmpFiles true by default and formatted code

selendroid · May 4, 2015 · 61961a3 · 61961a3
1 parent 4b082bd
commit 61961a3
Show file tree

Hide file tree

Showing 3 changed files with 22 additions and 22 deletions.
diff --git a/selendroid-standalone/src/main/java/io/selendroid/standalone/SelendroidConfiguration.java b/selendroid-standalone/src/main/java/io/selendroid/standalone/SelendroidConfiguration.java
@@ -140,8 +140,8 @@ public class SelendroidConfiguration {
   @Parameter(names = "-folder", description = "The folder which contains Android applications under test. This folder will monitor and add new apps to the apps store during the lifetime of the selendroid node.")
   private String folder = null;
 
-    @Parameter(names = "-deleteTmpFiles", description = "Deletes temporary files created by the Selendroid Server.")
-    private boolean deleteTmpFiles = false;
+  @Parameter(names = "-deleteTmpFiles", description = "Deletes temporary files created by the Selendroid Server.")
+  private boolean deleteTmpFiles = true;
 
   public void setKeystore(String keystore) {
     this.keystore = keystore;
@@ -367,13 +367,13 @@ public String getAppFolderToMonitor() {
     return folder;
   }
 
-    public boolean isDeleteTmpFiles() {
-        return deleteTmpFiles;
-    }
+  public boolean isDeleteTmpFiles() {
+    return deleteTmpFiles;
+  }
 
-    public void setDeleteTmpFiles(boolean deleteTmpFiles){
-        this.deleteTmpFiles = deleteTmpFiles;
-    }
+  public void setDeleteTmpFiles(boolean deleteTmpFiles){
+    this.deleteTmpFiles = deleteTmpFiles;
+  }
 
   /**
    * @return {@code true} if Selendroid standalone should run in grid mode.

diff --git a/...id-standalone/src/main/java/io/selendroid/standalone/builder/SelendroidServerBuilder.java b/...id-standalone/src/main/java/io/selendroid/standalone/builder/SelendroidServerBuilder.java
@@ -96,9 +96,9 @@ public SelendroidServerBuilder(SelendroidConfiguration serverConfiguration) {
   /* package */void init(AndroidApp aut) throws IOException, ShellCommandException {
     applicationUnderTest = aut;
     File customizedServer = File.createTempFile("selendroid-server", ".apk");
-      if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
-          customizedServer.deleteOnExit(); //Deletes temporary files created
-      }
+    if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
+      customizedServer.deleteOnExit(); //Deletes temporary files created
+    }
     log.info("Creating customized Selendroid-server: " + customizedServer.getAbsolutePath());
     InputStream is = getResourceAsStream(selendroidPrebuildServerPath);
 
@@ -115,12 +115,12 @@ public AndroidApp createSelendroidServer(AndroidApp aut) throws IOException,
     cleanUpPrebuildServer();
     File selendroidServer = createAndAddCustomizedAndroidManifestToSelendroidServer();
     File outputFile =
-        new File(FileUtils.getTempDirectory(), String.format("selendroid-server-%s-%s.apk",
+      new File(FileUtils.getTempDirectory(), String.format("selendroid-server-%s-%s.apk",
                                                              applicationUnderTest.getBasePackage(),
                                                              getJarVersionNumber()));
-      if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
-          outputFile.deleteOnExit(); //Deletes file when done
-      }
+    if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
+      outputFile.deleteOnExit(); //Deletes file when done
+    }
     return signTestServer(selendroidServer, outputFile);
   }
 
@@ -150,9 +150,9 @@ public AndroidApp resignApp(File appFile) throws ShellCommandException, AndroidS
     deleteFileFromAppSilently(app, "META-INF/NDKEYSTO.RSA");
 
     File outputFile = new File(appFile.getParentFile(), "resigned-" + appFile.getName());
-      if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
-          outputFile.deleteOnExit();
-      }
+    if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
+      outputFile.deleteOnExit();
+    }
     return signTestServer(appFile, outputFile);
   }
 

diff --git a/...alone/src/main/java/io/selendroid/standalone/server/model/SelendroidStandaloneDriver.java b/...alone/src/main/java/io/selendroid/standalone/server/model/SelendroidStandaloneDriver.java
@@ -164,10 +164,10 @@ public void addToAppsStore(File file) throws AndroidSdkException {
         // using "android" as the app name, because that is the desired capability default in
         // selenium for
         // DesiredCapabilities.ANDROID
-          File androidAPK = androidDriverAPKBuilder.extractAndroidDriverAPK();
-          if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
-              androidAPK.deleteOnExit(); //Deletes temporary files if flag set
-          }
+        File androidAPK = androidDriverAPKBuilder.extractAndroidDriverAPK();
+        if(serverConfiguration != null && serverConfiguration.isDeleteTmpFiles()) {
+          androidAPK.deleteOnExit(); //Deletes temporary files if flag set
+        }
         AndroidApp app =
             selendroidApkBuilder.resignApp(androidAPK);
         appsStore.put(BrowserType.ANDROID, app);