JavaScript (v3): GitHub - Run unit tests during JavaScript checks. (#6879)

* JavaScript (v3): GitHub - Run unit tests during JavaScript checks.

* JavaScript (v3): S3 - Add function comment to copy-object.

* JavaScript (v3): GitHub - Add /workflows to sparse checkout.

* JavaScript (v3): GitHub - Add format check

Author: cpyle0819

.github/workflows/lint-javascript.yml

@@ -16,6 +16,7 @@ jobs:
           sparse-checkout: |
             .github
             javascriptv3
+            workflows
       - name: Get changed files
         id: changed-files
         uses: tj-actions/changed-files@v41
@@ -37,3 +38,6 @@ jobs:
         run: |
           changed_files=(${{steps.changed-files.outputs.all_changed_files}})
           ./javascriptv3/github_scripts/lint.sh "${changed_files[@]}"
+      - name: Run unit tests
+        if: steps.changed-files.outputs.any_changed == 'true'
+        run: npm test --prefix javascriptv3

javascriptv3/example_code/s3/actions/copy-object.js

@@ -8,6 +8,9 @@ import { S3Client, CopyObjectCommand } from "@aws-sdk/client-s3";
 
 const client = new S3Client({});
 
+/**
+ * Copy an Amazon S3 object from one bucket to another.
+ */
 export const main = async () => {
   const command = new CopyObjectCommand({
     CopySource: "SOURCE_BUCKET/SOURCE_OBJECT_KEY",

javascriptv3/github_scripts/lint.sh

@@ -12,3 +12,6 @@ done
 
 # Run linting only on filtered files
 npm run --prefix javascriptv3 ci-lint -- "${filtered_files[@]//javascriptv3\/}"
+
+# Run format check
+npm run --prefix javascriptv3 ci-format-check -- "${@//javascriptv3\/}"

javascriptv3/package.json

@@ -7,14 +7,15 @@
     "merge-test-results": "jrm test-results.xml \"**/test_results/**/*.junit.xml\"",
     "log-tests": "cat test-results.xml",
     "lint": "lint-staged",
-    "ci-lint": "eslint"
+    "ci-lint": "eslint",
+    "ci-format-check": "prettier --check"
   },
   "lint-staged": {
     "*.{js,ts,jsx,tsx}": [
       "eslint",
       "prettier -w"
     ],
-    "*.{md}": [
+    "*.md": [
       "prettier -w"
     ]
   },