remove BoxLeds from toggle_pin command

FarmBot · Jun 9, 2022 · 1d4e5df · 1d4e5df
1 parent 9dc9665
commit 1d4e5df
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 3 deletions.
diff --git a/frontend/sequences/step_tiles/pin_support/__tests__/pin_and_peripheral_support_test.tsx b/frontend/sequences/step_tiles/pin_support/__tests__/pin_and_peripheral_support_test.tsx
@@ -132,6 +132,20 @@ describe("pinsAsDropdowns()", () => {
     const result = PinSupport.pinsAsDropdowns("read_pin")(ri.index, true);
     expect(JSON.stringify(result)).toContain("Pin 13");
   });
+
+  it("toggle_pin: displays peripherals and sensors", () => {
+    const p = fakePeripheral();
+    p.body.label = "displayed peripheral";
+    const ri = buildResourceIndex([p]);
+    const result = PinSupport.pinsAsDropdowns("toggle_pin")(ri.index, true);
+    expect(JSON.stringify(result)).toContain("displayed peripheral");
+  });
+
+  it("toggle_pin: displays pins", () => {
+    const ri = buildResourceIndex([]);
+    const result = PinSupport.pinsAsDropdowns("toggle_pin")(ri.index, true);
+    expect(JSON.stringify(result)).toContain("Pin 13");
+  });
 });
 
 describe("findByPinNumber()", () => {

diff --git a/frontend/sequences/step_tiles/pin_support/pin_and_peripheral_support.tsx b/frontend/sequences/step_tiles/pin_support/pin_and_peripheral_support.tsx
@@ -123,6 +123,14 @@ const pinsAsDropDownsWritePin = (
   ];
 };
 
+const pinsAsDropDownsTogglePin = (
+  resources: ResourceIndex, showPins: boolean,
+): DropDownItem[] =>
+  [
+    ...peripheralsAsDropDowns(resources),
+    ...(showPins ? pinDropdowns(n => n) : []),
+  ];
+
 const pinsAsDropDownsReadPin = (
   resources: ResourceIndex, showPins: boolean,
 ): DropDownItem[] =>
@@ -136,9 +144,8 @@ export const pinsAsDropdowns =
   (kind: "read_pin" | "write_pin" | "toggle_pin") => {
     switch (kind) {
       case "read_pin": return pinsAsDropDownsReadPin;
-      case "write_pin":
-      case "toggle_pin":
-        return pinsAsDropDownsWritePin;
+      case "write_pin": return pinsAsDropDownsWritePin;
+      case "toggle_pin": return pinsAsDropDownsTogglePin;
     }
 
   };