Commit:    05e66bd4a07bb84e1190fc77ae9eb73e9534c958
Author:    Allen Truong <v-alt...@microsoft.com>         Fri, 28 Jun 2019 
11:49:57 -0700
Committer: Anatol Belski <a...@php.net>      Sun, 30 Jun 2019 20:26:10 +0200
Parents:   f6ddff3a4b96e4a36f2534ad4177900485538238
Branches:  master

Link:       
http://git.php.net/?p=pftt2.git;a=commitdiff;h=05e66bd4a07bb84e1190fc77ae9eb73e9534c958

Log:
Fixes #43: Modified what files get deleted

Bugs:
https://bugs.php.net/43

Changed paths:
  M  src/com/mostc/pftt/model/core/PhptSourceTestPack.java
  M  src/com/mostc/pftt/runner/AbstractPhptTestCaseRunner.java


Diff:
diff --git a/src/com/mostc/pftt/model/core/PhptSourceTestPack.java 
b/src/com/mostc/pftt/model/core/PhptSourceTestPack.java
index d167962..25dc969 100644
--- a/src/com/mostc/pftt/model/core/PhptSourceTestPack.java
+++ b/src/com/mostc/pftt/model/core/PhptSourceTestPack.java
@@ -139,10 +139,15 @@ public class PhptSourceTestPack extends 
SourceTestPack<PhptActiveTestPack, PhptT
                // clean out these files from the test-pack so they don't have 
to be stored or copied (to other hosts), etc...
                // (when installing test-pack on another host, it should just 
copy all files so its identical, less variability.
                //  also this improves performance for the installation process)
-               fs.deleteFileExtension(test_pack, ".skip.php");
+               fs.deleteFileExtension(test_pack, ".exp");
+               fs.deleteFileExtension(test_pack, ".out");
+               fs.deleteFileExtension(test_pack, ".log");
+               fs.deleteFileExtension(test_pack, ".zip");
+               fs.deleteFileExtension(test_pack, ".tar.gz");
                fs.deleteFileExtension(test_pack, ".cmd");
                fs.deleteFileExtension(test_pack, ".sh");
                fs.deleteFileExtension(test_pack, ".clean.php");
+               fs.deleteFileExtension(test_pack, ".skip.php");
                fs.deleteFileExtension(test_pack, ".tmp");
                // don't delete .php (specifically run-test.php) in root of 
test-pack (user may want it later)
                IFileChooser PHP_CHOOSER = new IFileChooser() {
diff --git a/src/com/mostc/pftt/runner/AbstractPhptTestCaseRunner.java 
b/src/com/mostc/pftt/runner/AbstractPhptTestCaseRunner.java
index 657c143..a73b638 100644
--- a/src/com/mostc/pftt/runner/AbstractPhptTestCaseRunner.java
+++ b/src/com/mostc/pftt/runner/AbstractPhptTestCaseRunner.java
@@ -228,6 +228,7 @@ public abstract class AbstractPhptTestCaseRunner extends 
AbstractTestCaseRunner<
        protected void removeTempFiles() throws IllegalStateException, 
IOException {
                fs.deleteIfExists(prep.test_clean);
                fs.deleteIfExists(prep.test_file);
+               fs.deleteIfExists(prep.skipif_file);
                fs.deleteIfExists(prep.test_dir + "\\" + prep.base_file_name + 
".php.cmd");
        }

Reply via email to