diff --git a/Assets/Tests/Editor/CliSetupSectionTests.cs b/Assets/Tests/Editor/CliSetupSectionTests.cs index 54b6713188..0ec66b8fb2 100644 --- a/Assets/Tests/Editor/CliSetupSectionTests.cs +++ b/Assets/Tests/Editor/CliSetupSectionTests.cs @@ -85,14 +85,15 @@ public void IsUninstallCliAction_ReturnsExpectedValue( } [Test] - public void Update_WhenCliRefreshIsChecking_ShowsSkillsCheckingState() + public void Update_WhenSkillStateIsChecking_ShowsSkillsCheckingState() { - // Verifies that IsChecking routes the shared skills panel into ShowChecking. + // Verifies that IsSkillStateChecking routes the shared skills panel into ShowChecking. VisualElement root = CreateRootElement(); CliSetupSection section = new(root); CliSetupData data = CreateData( isCliInstalled: true, - isChecking: true, + isChecking: false, + isSkillStateChecking: true, selectedTargetInstallState: SkillInstallState.Missing); section.Update(data); @@ -113,6 +114,28 @@ public void Update_WhenCliRefreshIsChecking_ShowsSkillsCheckingState() Assert.That(groupSkillsToggle.enabledSelf, Is.False); } + [Test] + public void Update_WhenOnlyCliIsChecking_DoesNotShowSkillsCheckingState() + { + // Verifies CLI refresh checking does not force the skills panel into Checking... state. + VisualElement root = CreateRootElement(); + CliSetupSection section = new(root); + CliSetupData data = CreateData( + isCliInstalled: true, + isChecking: true, + isSkillStateChecking: false, + selectedTargetInstallState: SkillInstallState.Missing); + + section.Update(data); + + Button installAllSkillsButton = root.Q