test: add a failing case

Author: JounQin

src/utils/resolve.ts

@@ -306,7 +306,7 @@ function fullResolve(
     const sourceFiles =
       context.physicalFilename === sourceFile
         ? [sourceFile]
-        : [sourceFile, context.physicalFilename]
+        : [context.physicalFilename, sourceFile]
 
     for (const sourceFile of sourceFiles) {
       for (const {
@@ -355,7 +355,7 @@ function fullResolve(
         }
 
         // else, counts
-        fileExistsCache.set(cacheKey, resolved.path as string | null)
+        fileExistsCache.set(cacheKey, resolved.path)
         return resolved
       }
     }

test/fixtures/config-helper.js

@@ -6,17 +6,19 @@ import { setTimeout } from 'node:timers/promises'
 import { jest } from '@jest/globals'
 import type { TSESLint } from '@typescript-eslint/utils'
 
-import { testContext, testFilePath } from '../utils.js'
+import { TEST_FILENAME, testContext, testFilePath } from '../utils.js'
 
-import { importXResolverCompat } from 'eslint-plugin-import-x'
+import { cjsRequire, importXResolverCompat } from 'eslint-plugin-import-x'
 import type {
   CjsRequire,
   NewResolver,
   NormalizedCacheSettings,
+  RuleContext,
 } from 'eslint-plugin-import-x'
 import {
   CASE_SENSITIVE_FS,
   fileExistsWithCaseSync,
+  relative,
   resolve,
 } from 'eslint-plugin-import-x/utils'
 
@@ -830,5 +832,15 @@ describe('resolve', () => {
         }),
       ).toBe(testFilePath('./bar.tsx'))
     })
+
+    it('sourceFile should take higher priority than context.physicalFilename', () => {
+      const sourceFile = cjsRequire.resolve('typescript-eslint')
+      expect(
+        relative('./config-helper', sourceFile, {}, {
+          physicalFilename: TEST_FILENAME,
+          settings: {},
+        } as RuleContext),
+      ).toBe(path.resolve(sourceFile, '../config-helper.js'))
+    })
   })
 })