fix(resolutions): Warn instead of erroring if unmatched dependency ma…

…tches resolutions with glob support

Summary

PR #4835 fixed `yarn check` while using simple resolutions keys.

This PR fixes `yarn check` while using resolutions keys with globs or dep trees.

Test plan

New test case added.

__tests__/commands/check.js

@@ -383,7 +383,9 @@ test.concurrent('should ignore bundled dependencies', async (): Promise<void> =>
   );
 });
 
-test.concurrent('should warn about mismatched dependencies if they match resolutions', async (): Promise<void> => {
+test.concurrent('should warn about mismatched dependencies if they match resolutions (simple)', async (): Promise<
+  void,
+> => {
   let mismatchError = false;
   let stdout = '';
   try {
@@ -401,6 +403,46 @@ test.concurrent('should warn about mismatched dependencies if they match resolut
   ).toBeGreaterThan(-1);
 });
 
+test.concurrent('should warn about mismatched dependencies if they match resolutions (tree)', async (): Promise<
+  void,
+> => {
+  let mismatchError = false;
+  let stdout = '';
+  try {
+    await runCheck([], {}, 'resolutions-tree', (config, reporter, check, getStdout) => {
+      stdout = getStdout();
+    });
+  } catch (err) {
+    mismatchError = true;
+  }
+  expect(mismatchError).toEqual(false);
+  expect(
+    stdout.search(
+      `warning.*"repeat-string@1.4.0" is incompatible with requested version "pad-left#repeat-string@\\^1.5.4"`,
+    ),
+  ).toBeGreaterThan(-1);
+});
+
+test.concurrent('should warn about mismatched dependencies if they match resolutions (glob)', async (): Promise<
+  void,
+> => {
+  let mismatchError = false;
+  let stdout = '';
+  try {
+    await runCheck([], {}, 'resolutions-glob', (config, reporter, check, getStdout) => {
+      stdout = getStdout();
+    });
+  } catch (err) {
+    mismatchError = true;
+  }
+  expect(mismatchError).toEqual(false);
+  expect(
+    stdout.search(
+      `warning.*"repeat-string@1.4.0" is incompatible with requested version "pad-left#repeat-string@\\^1.5.4"`,
+    ),
+  ).toBeGreaterThan(-1);
+});
+
 test.concurrent('--integrity should throw an error if top level patterns do not match', async (): Promise<void> => {
   let integrityError = false;
   try {