fix command bug

Author: recca0120

package.json

@@ -4,7 +4,7 @@
   "displayName": "PHPUnit Test Explorer",
   "icon": "img/icon.png",
   "publisher": "recca0120",
-  "version": "3.7.8",
+  "version": "3.7.9",
   "private": true,
   "license": "MIT",
   "repository": {

src/PHPUnit/CommandBuilder/Builder.test.ts

@@ -125,7 +125,7 @@ describe('Builder Test', () => {
             expect(runtime).toEqual('php');
             expect(args).toEqual([
                 'vendor/bin/pest',
-                `--filter="^.*::(it has user's email)(( with (data set )?.*)?)?$"`,
+                `--filter=^.*::(it has user's email)(( with (data set )?.*)?)?$`,
                 phpUnitProject('tests/AssertionsTest.php'),
                 '--colors=never',
                 '--teamcity',
@@ -184,7 +184,7 @@ describe('Builder Test', () => {
                 [
                     'php',
                     'vendor/bin/paratest',
-                    `--filter=^.*::(test_passed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed)( with data set .*)?$'`,
                     `--colors=never`,
                     `--teamcity`,
                     `--functional`,
@@ -263,7 +263,7 @@ describe('Builder Test', () => {
                     'php',
                     'artisan',
                     'test',
-                    `--filter=^.*::(test_passed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed)( with data set .*)?$'`,
                     `--colors=never`,
                     `--teamcity`,
                 ].join(' '),
@@ -292,7 +292,7 @@ describe('Builder Test', () => {
                     'php',
                     'artisan',
                     'test',
-                    `--filter=^.*::(test_passed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed)( with data set .*)?$'`,
                     '--colors=never',
                     '--teamcity',
                 ].join(' '),
@@ -394,7 +394,7 @@ describe('Builder Test', () => {
                     'php',
                     'artisan',
                     'test',
-                    `--filter=^.*::(test_passed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed)( with data set .*)?$'`,
                     `--colors=never`,
                     `--teamcity`,
                 ].join(' '),
@@ -418,7 +418,7 @@ describe('Builder Test', () => {
                 [
                     'php',
                     'vendor/bin/phpunit',
-                    `--filter=^.*::(test_passed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed)( with data set .*)?$'`,
                     '--colors=never',
                     '--teamcity',
                 ].join(' '),

src/PHPUnit/CommandBuilder/Builder.ts

@@ -162,22 +162,26 @@ export class Builder {
             const pattern = new RegExp('^(--filter)=(.*)');
 
             return input.replace(pattern, (_m, ...matched) => {
-                const value = btoa(matched[1]);
+                const value = Buffer.from(matched[1], 'utf-8').toString('base64');
 
                 return `${matched[0]}='${value}'`;
             });
         });
     }
 
     private decodeFilter(args: string[]) {
+        const command = args.join(' ');
+        const needsQuote = isSSH(command) || isShellCommand(command);
+
         return args.map((input) => {
             const pattern = new RegExp('(--filter)=["\'](.+)?["\']');
 
             return input.replace(pattern, (_m, ...matched) => {
-                const value = atob(matched[1]);
-                const quote = value.includes('\'') ? '"' : '';
+                const value = Buffer.from(matched[1], 'base64').toString('utf-8');
+                const quote = value.includes('\'') ? '"' : '\'';
+                const filter = `${matched[0]}=${value}`;
 
-                return `${matched[0]}=${quote}${value}${quote}`;
+                return needsQuote ? `${quote}${filter}${quote}` : filter;
             });
         });
     }

src/PHPUnit/TestRunner.test.ts

@@ -346,20 +346,6 @@ describe('TestRunner Test', () => {
 
             await shouldRunTestFailed(expected, builder, projectPath, appPath);
         });
-
-        it('should run test failed with phpvfscomposer for Docker', async () => {
-            const expected = [
-                'php',
-                appPath('vendor/bin/phpunit'),
-                `--configuration=${appPath('phpunit.xml')}`,
-                `--filter=^.*::(test_passed|test_failed)( with data set .*)?$`,
-                appPath('tests/AssertionsTest.php'),
-                '--colors=never',
-                '--teamcity',
-            ];
-
-            await shouldRunTestFailed(expected, builder, projectPath, appPath, true);
-        });
     });
 
     describe('SSH', () => {
@@ -441,7 +427,7 @@ describe('TestRunner Test', () => {
                     'php',
                     'vendor/bin/phpunit',
                     `--configuration=${appPath('phpunit.xml')}`,
-                    `--filter=^.*::(test_passed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed)( with data set .*)?$'`,
                     appPath('tests/AssertionsTest.php'),
                     `--colors=never`,
                     `--teamcity`,
@@ -467,7 +453,7 @@ describe('TestRunner Test', () => {
                     'php',
                     'vendor/bin/phpunit',
                     `--configuration=${appPath('phpunit.xml')}`,
-                    `--filter=^.*::(test_passed|test_failed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed|test_failed)( with data set .*)?$'`,
                     appPath('tests/AssertionsTest.php'),
                     `--colors=never`,
                     `--teamcity`,
@@ -493,7 +479,7 @@ describe('TestRunner Test', () => {
                     'php',
                     'vendor/bin/phpunit',
                     `--configuration=${appPath('phpunit.xml')}`,
-                    `--filter=^.*::(test_passed|test_failed)( with data set .*)?$`,
+                    `'--filter=^.*::(test_passed|test_failed)( with data set .*)?$'`,
                     appPath('tests/AssertionsTest.php'),
                     `--colors=never`,
                     `--teamcity`,