Don't log missing-module-attribute when module lookup errors with NoStubs or NoSource

Summary: If we fail to resolve `from x import y` due to `y` not being found, we should report `missing-module-attribute`. If `y` was found but we want to emit some other warning about missing source or stubs for it, we should not report `missing-module-attribute`.

Reviewed By: samwgoldman

Differential Revision: D85578780

fbshipit-source-id: 4f93e62c08c455491951affee4f7cb6681394016

Author: rchen152

pyrefly/lib/binding/stmt.rs

@@ -1025,7 +1025,7 @@ impl<'a> BindingsBuilder<'a> {
                         FindingOrError::Finding(finding) => (true, finding.error),
                         FindingOrError::Error(error) => (false, Some(error)),
                     };
-                    let error = error.is_some_and(|e| e.kind().is_some());
+                    let error = error.is_some_and(|e| matches!(e, FindError::NotFound(..)));
                     if error {
                         self.error(
                             x.range,