More types (tests)

Related: #1225
ansible · Apr 24, 2024 · 6daae4b · 6daae4b
1 parent cf5fb47
commit 6daae4b
Show file tree

Hide file tree

Showing 3 changed files with 267 additions and 230 deletions.
diff --git a/packages/ansible-language-server/test/providers/completionResolver.test.ts b/packages/ansible-language-server/test/providers/completionResolver.test.ts
@@ -216,92 +216,97 @@ describe("doCompletionResolve()", () => {
   const context = workspaceManager.getContext(fixtureFileUri);
 
   const textDoc = getDoc(fixtureFilePath);
-  const docSettings = context.documentSettings.get(textDoc.uri);
+  expect(context).be.not.undefined;
+  if (context) {
+    const docSettings = context.documentSettings.get(textDoc.uri);
 
-  describe("Resolve completion for module names", () => {
-    describe("Resolve completion for module names when FQCN is enabled", function () {
-      describe("with EE enabled @ee", () => {
-        before(async () => {
-          setFixtureAnsibleCollectionPathEnv(
-            "/home/runner/.ansible/collections:/usr/share/ansible",
-          );
-          await enableExecutionEnvironmentSettings(docSettings);
+    describe("Resolve completion for module names", () => {
+      describe("Resolve completion for module names when FQCN is enabled", function () {
+        describe("with EE enabled @ee", () => {
+          before(async () => {
+            setFixtureAnsibleCollectionPathEnv(
+              "/home/runner/.ansible/collections:/usr/share/ansible",
+            );
+            await enableExecutionEnvironmentSettings(docSettings);
+          });
+          testFQCNEnabled(context);
+
+          after(async () => {
+            setFixtureAnsibleCollectionPathEnv();
+            await disableExecutionEnvironmentSettings(docSettings);
+          });
         });
-        testFQCNEnabled(context);
 
-        after(async () => {
-          setFixtureAnsibleCollectionPathEnv();
-          await disableExecutionEnvironmentSettings(docSettings);
+        describe("with EE disabled", () => {
+          before(async () => {
+            setFixtureAnsibleCollectionPathEnv();
+            await disableExecutionEnvironmentSettings(docSettings);
+          });
+          testFQCNEnabled(context);
         });
       });
 
-      describe("with EE disabled", () => {
-        before(async () => {
-          setFixtureAnsibleCollectionPathEnv();
-          await disableExecutionEnvironmentSettings(docSettings);
+      describe("Resolve completion for module names when FQCN is disabled", function () {
+        describe("with EE enabled @ee", () => {
+          before(async () => {
+            setFixtureAnsibleCollectionPathEnv(
+              "/home/runner/.ansible/collections:/usr/share/ansible",
+            );
+            await enableExecutionEnvironmentSettings(docSettings);
+            (await docSettings).ansible.useFullyQualifiedCollectionNames =
+              false;
+          });
+          testFQCNDisabled(context);
+
+          after(async () => {
+            setFixtureAnsibleCollectionPathEnv();
+            await disableExecutionEnvironmentSettings(docSettings);
+            (await docSettings).ansible.useFullyQualifiedCollectionNames = true;
+          });
+        });
+
+        describe("with EE disabled", () => {
+          before(async () => {
+            setFixtureAnsibleCollectionPathEnv();
+            await disableExecutionEnvironmentSettings(docSettings);
+            (await docSettings).ansible.useFullyQualifiedCollectionNames =
+              false;
+          });
+          testFQCNDisabled(context);
+
+          after(async () => {
+            setFixtureAnsibleCollectionPathEnv();
+            (await docSettings).ansible.useFullyQualifiedCollectionNames = true;
+          });
         });
-        testFQCNEnabled(context);
       });
     });
 
-    describe("Resolve completion for module names when FQCN is disabled", function () {
+    describe("Resolve completion for module options and suboptions", () => {
       describe("with EE enabled @ee", () => {
         before(async () => {
           setFixtureAnsibleCollectionPathEnv(
             "/home/runner/.ansible/collections:/usr/share/ansible",
           );
           await enableExecutionEnvironmentSettings(docSettings);
-          (await docSettings).ansible.useFullyQualifiedCollectionNames = false;
         });
-        testFQCNDisabled(context);
+
+        testResolveModuleOptionCompletion(context);
 
         after(async () => {
           setFixtureAnsibleCollectionPathEnv();
           await disableExecutionEnvironmentSettings(docSettings);
-          (await docSettings).ansible.useFullyQualifiedCollectionNames = true;
         });
       });
 
       describe("with EE disabled", () => {
         before(async () => {
           setFixtureAnsibleCollectionPathEnv();
           await disableExecutionEnvironmentSettings(docSettings);
-          (await docSettings).ansible.useFullyQualifiedCollectionNames = false;
-        });
-        testFQCNDisabled(context);
-
-        after(async () => {
-          setFixtureAnsibleCollectionPathEnv();
-          (await docSettings).ansible.useFullyQualifiedCollectionNames = true;
         });
-      });
-    });
-  });
 
-  describe("Resolve completion for module options and suboptions", () => {
-    describe("with EE enabled @ee", () => {
-      before(async () => {
-        setFixtureAnsibleCollectionPathEnv(
-          "/home/runner/.ansible/collections:/usr/share/ansible",
-        );
-        await enableExecutionEnvironmentSettings(docSettings);
-      });
-
-      testResolveModuleOptionCompletion(context);
-
-      after(async () => {
-        setFixtureAnsibleCollectionPathEnv();
-        await disableExecutionEnvironmentSettings(docSettings);
+        testResolveModuleOptionCompletion(context);
       });
     });
-
-    describe("with EE disabled", () => {
-      before(async () => {
-        setFixtureAnsibleCollectionPathEnv();
-        await disableExecutionEnvironmentSettings(docSettings);
-      });
-
-      testResolveModuleOptionCompletion(context);
-    });
-  });
+  }
 });