Don't error when ENGINE_API_KEY and APOLLO_KEY are found (#1893)

Author: JakeDawkins

CHANGELOG.md

@@ -19,6 +19,7 @@
 - `apollo-graphql`
   - <First `apollo-graphql` related entry goes here>
 - `apollo-language-server`
+  - Remove error from the case with old and new api keys present [#1893](https://github.com/apollographql/apollo-tooling/pull/1893)
   - Add Elixir support for vscode [#1971](https://github.com/apollographql/apollo-tooling/pull/1971)
 - `apollo-tools`
   - <First `apollo-tools` related entry goes here>

packages/apollo-language-server/src/config/__tests__/loadConfig.ts

@@ -339,18 +339,22 @@ describe("loadConfig", () => {
       );
     });
 
-    it("Throws when .env defined legacy and new key", async () => {
+    it("Uses new key when .env defined both legacy and new key", async () => {
       writeFilesToDir(dir, {
         "my.config.js": `module.exports = { client: { name: 'hello' } }`,
         ".env.local": `ENGINE_API_KEY=service:yoshi:65489061ko\nAPOLLO_KEY=service:yoshi:65489061ko`
       });
+      const spy = jest.spyOn(Debug, "warning");
 
-      const loadConfigPromise = loadConfig({
+      const config = await loadConfig({
         configPath: dirPath,
         configFileName: "my.config.js"
       });
 
-      await expect(loadConfigPromise).rejects.toThrow(/Cannot set both/);
+      expect(config.engine.apiKey).toEqual("service:yoshi:65489061ko");
+      expect(spy).toHaveBeenCalledWith(
+        expect.stringMatching(/Both ENGINE_API_KEY and APOLLO_KEY were found/i)
+      );
     });
 
     // this doesn't work right now :)

packages/apollo-language-server/src/config/loadConfig.ts

@@ -129,8 +129,8 @@ export async function loadConfig({
       const legacyKey = env[legacyKeyEnvVar];
       const key = env[keyEnvVar];
       if (legacyKey && key) {
-        throw new Error(
-          `Cannot set both ${legacyKeyEnvVar} and ${keyEnvVar}. Please only set ${keyEnvVar}`
+        Debug.warning(
+          `Both ${legacyKeyEnvVar} and ${keyEnvVar} were found. ${keyEnvVar} will take precedence.`
         );
       }
       if (legacyKey) {