WIP: Adopt calls to work with tox v3 as well as v4

Due to setups with i.e. minversion=v4 in tox.ini, it may happen, that
one is enforced to use this (although system install is still v3 ...).
This should support both versions at all tox calls.

Author: s-hemer

package.json

@@ -6,7 +6,7 @@
 		"python",
 		"tox"
 	],
-	"version": "1.0.0",
+	"version": "1.0.1-dev",
 	"license": "MIT",
 	"publisher": "the-compiler",
 	"repository": {

src/run.ts

@@ -6,7 +6,8 @@ import { getTerminal } from './utils';
 
 const exec = util.promisify(child_process.exec);
 
-export const commandToxListAllEnvs = 'tox -a';
+export const commandToxListAllEnvs = 'tox -a'; //for tox v4 may switch to
+//export const commandToxListAllEnvs = 'tox list --no-desc';
 
 export async function getToxEnvs(projDir: string) {
 	try {
@@ -34,13 +35,14 @@ export async function getToxEnvs(projDir: string) {
 	return undefined;
 }
 
-export const commandToxRun = 'tox -e';
+export const commandToxRun = 'tox -e'; //for tox v4 switch to
+//export const commandToxRun = 'tox run -e'; // does not work with a folder, needs to use --root/--workdir/..
 
 export function runTox(envs: string[], toxArguments: string, terminal: vscode.Terminal = getTerminal() ) {
 	const envArg = envs.join(",");
 	terminal.show(true);  // preserve focus
 
-	// FIXME In theory, there's a command injection here, if an environment name
+	// FIXME: In theory, there's a command injection here, if an environment name
 	// contains shell metacharacters. However:
 	// - Escaping the argument in a shell-agnostic way is hard:
 	//   https://github.com/microsoft/vscode/blob/1.57.0/src/vs/workbench/contrib/debug/node/terminals.ts#L84-L211

src/testController.ts

@@ -37,7 +37,7 @@ export function create() {
 
 			const start = Date.now();
 			try {
-				const cwd = vscode.workspace.getWorkspaceFolder(test.uri!)!.uri.path;
+				const cwd = "--workdir " + vscode.workspace.getWorkspaceFolder(test.uri!)!.uri.path + "/.tox";
 				runTox([test.label], cwd);
 				run.passed(test, Date.now() - start);
 			}