Add test: DynamicContentCheckbox can toggle "IsDisabled" attribute on ContentScript (#189)

Author: Christian-Toney

src/DialogueEditor/components/Checkbox/init.luau

@@ -24,22 +24,28 @@ local function Checkbox(properties: CheckboxProperties)
   local colors = useStudioColors();
   local icons = useStudioIcons();
   local buttonRef = React.useRef(nil :: TextButton?);
-  
-  React.useEffect(function()
+
+  React.useEffect(function(): ()
 
     local button = buttonRef.current;
     assert(button, "Button reference should not be nil.");
 
-    if isDisabled then return end;
+    local activatedConnection: RBXScriptConnection? = nil;
 
-    local activatedConnection = VirtualService.events.GuiButton.Activated:getSignal(button):Connect(function()
+    if not isDisabled then
 
-      onChanged(not isChecked);
+      activatedConnection = VirtualService.events.GuiButton.Activated:getSignal(button):Connect(function()
 
-    end);
+        onChanged(not isChecked);
+
+      end);
+
+    end;
+
+    button:SetAttribute("IsChecked", isChecked);
 
     return function()
-    
+
       if activatedConnection then
 
         activatedConnection:Disconnect();
@@ -48,7 +54,7 @@ local function Checkbox(properties: CheckboxProperties)
 
     end;
 
-  end, {onChanged :: unknown, isDisabled});
+  end, {onChanged :: unknown, isDisabled, isChecked});
 
   return React.createElement("Frame", {
     AutomaticSize = Enum.AutomaticSize.XY;

src/DialogueEditor/components/Explorer/components/Preview/components/DialogueTypeDropdown/init.luau

@@ -70,7 +70,6 @@ local function DialogueTypeDropdown(properties: DialogueTypeDropdownProperties)
 
           end;
 
-          task.wait();
           Selection:Set({selectedScript});
 
           finishHistoryRecording(historyIdentifier);

src/DialogueEditor/components/Explorer/components/Preview/components/DynamicContentCheckbox/DynamicContentCheckbox.test.luau

@@ -125,6 +125,39 @@ return {
         end).toFinishBeforeSeconds(1);
 
       end);
+
+      it(`can toggle "IsDisabled" attribute on ContentScript`, function()
+
+        expect(function()
+
+          local selectedScript = render("Message");
+
+          assert(screenGui, "ScreenGui should be initialized before running tests.");
+          local dynamicContentCheckbox = screenGui:FindFirstChildOfClass("Frame");
+          assert(dynamicContentCheckbox, "DynamicContentCheckbox should be rendered.");
+
+          local checkbox = dynamicContentCheckbox:FindFirstChild("Checkbox");
+          assert(checkbox and checkbox:IsA("TextButton"), "Checkbox should be rendered inside DynamicContentCheckbox.");
+
+          VirtualService.events.GuiButton.Activated:fireEvent(checkbox);
+          local contentScript = selectedScript:WaitForChild("ContentScript");
+          verifyReactStatus();
+          checkbox:GetAttributeChangedSignal("IsChecked"):Wait();
+          expect(contentScript:GetAttribute("IsDisabled")).toBe(false);
+
+          VirtualService.events.GuiButton.Activated:fireEvent(checkbox);
+          checkbox:GetAttributeChangedSignal("IsChecked"):Wait();
+          verifyReactStatus();
+          expect(contentScript:GetAttribute("IsDisabled")).toBe(true);
+
+          VirtualService.events.GuiButton.Activated:fireEvent(checkbox);
+          checkbox:GetAttributeChangedSignal("IsChecked"):Wait();
+          verifyReactStatus();
+          expect(contentScript:GetAttribute("IsDisabled")).toBe(false);
+
+        end).toFinishBeforeSeconds(1);
+
+      end);
 
     }
 

src/DialogueEditor/components/Explorer/components/Preview/hooks/useDialogueContentScript.luau

@@ -31,7 +31,7 @@ local function useDialogueContentScript(dialogueScript: ModuleScript): (ModuleSc
 
     local childAddedConnection = dialogueScript.ChildAdded:Connect(updateDialogueContentScript);
     local childRemovedConnection = dialogueScript.ChildRemoved:Connect(updateDialogueContentScript);
-    updateDialogueContentScript();
+    task.spawn(updateDialogueContentScript);
 
     return function()