自動スクリプト

Author: Rozelin-dc

.github/workflows/fix-e2e.yml

@@ -71,22 +71,8 @@ jobs:
       # comment on PR with results
       - name: List and display file contents
         run: |
-          DIR="result"
-          OUTPUT="## Result of test fix\n\n"
-          FOUND_FILES=0
-
-          while IFS= read -r -d '' FILE; do
-            FOUND_FILES=1
-            echo "Processing $FILE"
-            CONTENT=$(cat "$FILE")
-            OUTPUT+="### Contents of \`$(basename "$FILE")\`\n\n\`\`\`\n$CONTENT\n\`\`\`\n\n"
-          done < <(find "$DIR" -maxdepth 1 -type f -print0)
-
-          if [ "$FOUND_FILES" -eq 0 ]; then
-            OUTPUT="No test fix required"
-          fi
-
-          echo -e "$OUTPUT" > comment.txt
+          chmod +x ./scripts/suggest-repair/index.sh
+          ./scripts/suggest-repair/index.sh
 
       - name: Comment on PR
         run: gh pr comment ${{ github.event.number }} --body-file comment.txt

scripts/suggest-repair/index.sh

@@ -0,0 +1,15 @@
+#!/bin/bash
+
+project_root_dir=$(pwd)
+result_file_dir="$project_root_dir/result"
+
+files=$(find $project_root_dir/e2e)
+
+for file in $files; do
+  if [ -d $file ]; then
+    # If $file directory, do nothing.
+    continue
+  fi
+
+  node ./scripts/suggest-repair/main.mjs --projectRootDir $project_root_dir --testFile $file --resultFileDir $result_file_dir
+done

scripts/suggest-repair/main.mjs

@@ -3,7 +3,7 @@ import _traverse from '@babel/traverse'
 import yargs from 'yargs'
 
 import { getAst } from '../util/get-ast.mjs'
-import { readFile } from '../util/file.mjs'
+import { readFile, writeFile } from '../util/file.mjs'
 import { TEST_ID_ATTRIBUTE_NAME } from '../util/constant.mjs'
 
 /** @type {typeof _traverse} */
@@ -36,6 +36,10 @@ const COLLECT_FUNCTIONS = new Set([
 
 const argv = await yargs(process.argv.slice(2))
   .options({
+    projectRootDir: {
+      demandOption: true,
+      string: true,
+    },
     testFile: {
       demandOption: true,
       string: true,
@@ -142,6 +146,7 @@ traverse(ast, {
   },
 })
 
+let comment = ''
 for (const testName in results) {
   const fixedResults = await readFile(
     path.join(argv.resultFileDir, `${testName}.json`),
@@ -318,9 +323,9 @@ for (const testName in results) {
     }
   }
 
-  console.log(
-    `### Suggested Repairs for test "${testName}" in "${
-      argv.testFile
-    }"\n${suggestedRepairs.join('\n')}`
-  )
+  comment += `### Suggested Repairs for test "${testName}" in "${
+    argv.testFile
+  }"\n${suggestedRepairs.join('\n')}\n`
 }
+
+await writeFile(path.join(argv.projectRootDir, 'comment.txt'), comment, true)

scripts/util/file.mjs

@@ -4,14 +4,18 @@ import path from 'path'
 /**
  * @param {string} filePath
  * @param {string} content
+ * @param {boolean} [append=false]
  * @return {void}
  */
-export function writeFile(filePath, content) {
+export function writeFile(filePath, content, append = false) {
   const dir = path.dirname(filePath)
   if (!fs.existsSync(dir)) {
     fs.mkdirSync(dir, { recursive: true })
   }
-  fs.writeFileSync(filePath, content, 'utf-8')
+  fs.writeFileSync(filePath, content, {
+    encoding: 'utf-8',
+    flag: append ? 'a' : 'w',
+  })
 }
 
 /**