Merge pull request #19040 from github/run-lint-files-separately

Run lint-files test separately and test fewer developer redirects

Author: sarahs

.github/workflows/test-windows.yml

@@ -23,7 +23,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        test-group: [content, graphql, meta, rendering, routing, unit]
+        test-group: [content, graphql, meta, rendering, routing, unit, linting]
     steps:
       - name: Check out repo
         uses: actions/checkout@5a4ac9002d0be2fb38bd78e4b4dbde5606d7042f

.github/workflows/test.yml

@@ -25,7 +25,8 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        test-group: [content, graphql, meta, rendering, routing, unit]
+        # The same array lives in test-windows.yml, so make any updates there too.
+        test-group: [content, graphql, meta, rendering, routing, unit, linting]
     steps:
       # Each of these ifs needs to be repeated at each step to make sure the required check still runs
       # Even if if doesn't do anything

tests/content/lint-files.js tests/linting/lint-files.jstests/content/lint-files.js renamed to tests/linting/lint-files.js

@@ -15,7 +15,7 @@ describe('developer.github.com redirects', () => {
     ]
 
     // test a subset of the top paths
-    const pathsToCheck = 300
+    const pathsToCheck = 50
     const paths = require('../fixtures/top-old-developer-site-paths.json')
       .filter(path => !ignoredPatterns.some(pattern => path.match(pattern)))
       .slice(0, pathsToCheck)