Merge pull request #35 from github/joshmgross/validate-inputs-defined

Ensure `inputs` is defined

actions-languageservice/src/complete.expressions.test.ts

@@ -241,11 +241,33 @@ jobs:
     expect(result.map(x => x.label)).toEqual(["another-name", "name"]);
   });
 
+  it("no inputs", async () => {
+    const input = `
+on:
+  workflow_dispatch:
+jobs:
+  a:
+    outputs:
+      build_id: my-build-id
+    runs-on: ubuntu-latest
+    steps:
+    - run: echo hello a
+  b:
+    needs: [a]
+    runs-on: ubuntu-latest
+    steps:
+    - run: echo "hello \${{ inputs.|
+`;
+    const result = await complete(...getPositionFromCursor(input), undefined, contextProviderConfig);
+
+    expect(result).toEqual([]);
+  });
+  
   describe("steps context", () => {
     it("includes defined step IDs", async () => {
       const input = `
-  on: push
+on: push
-  jobs:
+jobs:
   one:
     runs-on: ubuntu-latest
     steps:
@@ -255,7 +277,7 @@ jobs:
       run: echo hello b
     - id: c
       run: echo "hello \${{ steps.|
-  `;
+`;
       const result = await complete(...getPositionFromCursor(input), undefined, contextProviderConfig);
 
       expect(result.map(x => x.label)).toEqual(["a", "b"]);
@@ -263,8 +285,8 @@ jobs:
 
     it("step.<step_id>", async () => {
       const input = `
-    on: push
+on: push
-    jobs:
+jobs:
   one:
     runs-on: ubuntu-latest
     steps:
@@ -279,8 +301,8 @@ jobs:
 
     it("ignores IDs from later steps", async () => {
       const input = `
-  on: push
+on: push
-  jobs:
+jobs:
   one:
     runs-on: ubuntu-latest
     steps:

actions-languageservice/src/context-providers/inputs.ts

@@ -13,6 +13,9 @@ export function getInputsContext(workflowContext: WorkflowContext): data.Diction
   }
 
   const inputs = event.inputs;
+  if (!inputs) {
+    return d;
+  }
   for (const inputName of Object.keys(inputs)) {
     const input = inputs[inputName];
     switch (input.type) {