pass more information and copy noxfile

Author: davidcavazos

.github/testing/src/config.ts

@@ -15,16 +15,27 @@
 import * as fs from 'node:fs';
 import * as git from './git';
 import * as path from 'path';
-import {List, Map, Set} from 'immutable';
-import {minimatch} from 'minimatch'; /* eslint-disable @typescript-eslint/no-explicit-any */
-import {Affected, TestAll, TestName, TestPath, mergeAffected} from './affected';
+import { List, Map, Set } from 'immutable';
+import { minimatch } from 'minimatch'; /* eslint-disable @typescript-eslint/no-explicit-any */
+import { Affected, TestAll, TestName, TestPath, mergeAffected } from './affected';
+
+type RunTestsAll = {
+  root: string
+  path: string
+}
+
+type RunTestsSome = {
+  root: string
+  path: string
+  tests: Map<TestPath, Set<TestName>>;
+}
 
 export class Config {
   match: List<string>;
   ignore: List<string>;
   packageFile: List<string>;
-  testAll: () => void;
-  testSome: (tests: Map<TestPath, Set<TestName>>) => void;
+  testAll: (_: RunTestsAll) => void;
+  testSome: (_: RunTestsSome) => void;
 
   constructor({
     match,
@@ -36,16 +47,14 @@ export class Config {
     match?: string[];
     ignore?: string[];
     packageFile?: string[];
-    testAll?: () => void;
-    testSome?: (tests: Map<TestPath, Set<TestName>>) => void;
+    testAll?: (_: RunTestsAll) => void;
+    testSome?: (_: RunTestsSome) => void;
   }) {
     this.match = List(match || ['**']);
     this.ignore = List(ignore);
     this.packageFile = List(packageFile);
-    this.testAll = testAll || (() => {});
-    this.testSome =
-      testSome ||
-      (_ => {}); /* eslint-disable @typescript-eslint/no-unused-vars */
+    this.testAll = testAll || (_ => { });
+    this.testSome = testSome || (_ => { });
   }
 
   affected = (head: string, main: string): List<Affected> =>
@@ -61,14 +70,19 @@ export class Config {
 
   test = (affected: Affected) => {
     const cwd = process.cwd();
-    const dir = path.join(git.root(), affected.path);
+    const root = git.root();
+    const dir = path.join(root, affected.path);
     console.log(`>> cd ${dir}`);
     process.chdir(dir);
     if ('TestAll' in affected) {
-      this.testAll();
+      this.testAll({ root: root, path: affected.path });
     }
     if ('TestSome' in affected) {
-      this.testSome(affected.TestSome);
+      this.testSome({
+        root: root,
+        path: affected.path,
+        tests: affected.TestSome,
+      });
     }
     process.chdir(cwd);
   };

.github/testing/src/config/python.ts

@@ -12,8 +12,9 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+import * as path from 'path'
 import * as subprocess from '../subprocess';
-import {Config} from '../config';
+import { Config } from '../config';
 
 export const python = new Config({
   match: ['**'],
@@ -25,10 +26,12 @@ export const python = new Config({
     'setup.py',
     'setup.cfg',
   ],
-  testAll: () => {
+  testAll: args => {
+    subprocess.run('cp', [path.join(args.root, 'noxfile-template.py'), 'noxfile.py'])
     subprocess.run('nox', ['-s', 'py-3.11']);
   },
-  testSome: tests => {
-    throw `TODO: config/python.ts testSome ${JSON.stringify(tests)}`;
+  testSome: args => {
+    subprocess.run('cp', [path.join(args.root, 'noxfile-template.py'), 'noxfile.py'])
+    throw `TODO: config/python.ts testSome ${JSON.stringify(args)}`;
   },
 });

.github/testing/src/git.ts

@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-import { List } from 'immutable';
+import {List} from 'immutable';
 import * as subprocess from './subprocess';
 
 export type Diff = {

.github/testing/src/subprocess.ts

@@ -12,10 +12,10 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-import { spawnSync } from 'child_process';
+import {spawnSync} from 'child_process';
 
 export function run(cmd: string, args: string[]) {
-  const p = spawnSync(cmd, args, { stdio: 'inherit' });
+  const p = spawnSync(cmd, args, {stdio: 'inherit'});
   process.exitCode = p.status || undefined;
 }
 
@@ -24,4 +24,3 @@ export function output(cmd: string, args: string[]): string {
   process.exitCode = p.status || undefined;
   return p.stdout.toString().trim();
 }
-