cleanup

Author: acao

packages/graphql-language-service-server/src/MessageProcessor.ts

@@ -975,8 +975,8 @@ export class MessageProcessor {
         const schemaText = await readFile(uri, 'utf8');
         await this._cacheSchemaText(schemaUri, schemaText, version);
       }
-    } catch {
-      // this._logger.error(String(err));
+    } catch (err) {
+      this._logger.error(String(err));
     }
   }
   private _getTmpProjectPath(
@@ -1184,7 +1184,6 @@ export class MessageProcessor {
       }),
     );
   }
-
   private _unwrapProjectSchema(project: GraphQLProjectConfig): string[] {
     const projectSchema = project.schema;
 

packages/graphql-language-service-server/src/__tests__/MessageProcessor.spec.ts

@@ -245,6 +245,8 @@ describe('project with simple config and graphql files', () => {
     expect(serializeRange(schemaDefinitionsAgain[0].range)).toEqual(
       fooLaterTypePosition,
     );
+    expect(project.lsp._logger.error).not.toHaveBeenCalled();
+
     // TODO: the position should change when a watched file changes???
   });
   it('caches files and schema with a URL config', async () => {
@@ -369,6 +371,7 @@ describe('project with simple config and graphql files', () => {
         character: 31,
       },
     });
+    expect(project.lsp._logger.error).not.toHaveBeenCalled();
   });
   it('caches multiple projects with files and schema with a URL config and a local schema', async () => {
     const project = new MockProject({
@@ -476,5 +479,7 @@ describe('project with simple config and graphql files', () => {
       position: { character: 21, line: 4 },
     });
     expect(serializeRange(schemaDefinition[0].range)).toEqual(fooTypePosition);
+
+    expect(project.lsp._logger.error).not.toHaveBeenCalled();
   });
 });