Skip to content

Add toggle behavior tests#4998

Merged
Bubballoo3 merged 14 commits intomasterfrom
test-toggle-behavior-4929
Feb 24, 2026
Merged

Add toggle behavior tests#4998
Bubballoo3 merged 14 commits intomasterfrom
test-toggle-behavior-4929

Conversation

@ahmed-mgd
Copy link
Contributor

Fixes #4929

Tests toggle behavior throughout the dashboard. This is probably not comprehensive but should be a decent starting point.

@github-project-automation github-project-automation bot moved this to Awaiting Review in PR Review Pipeline Jan 21, 2026
@ahmed-mgd ahmed-mgd marked this pull request as draft January 21, 2026 22:05
@ahmed-mgd ahmed-mgd marked this pull request as ready for review February 11, 2026 17:30
@Bubballoo3
Copy link
Contributor

Weirdly enough (since CI is passing), I cannot get any of these to pass in my dev dashboard due to some ood_core mismatch. Can anyone else replicate?

Removed sleep statements and replaced them with assertions to check visibility of elements after toggle actions.
end
end

test 'module browser toggles module details' do
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this a duplicate of the toggling test in module_browser_test.rb?

test 'module button expands and collapses the module details' do
with_modified_env({ OOD_MODULE_FILE_DIR: fixture_dir }) do
visit module_browser_url
first_module = find(MODULE_BTN_SELECT, match: :first)
collapse_id = first_module['data-bs-target']
assert_selector("#{collapse_id}", visible: :hidden)
first_module.click
assert_selector("#{MODULE_BTN_SELECT}.active")
assert_selector("#{collapse_id}", visible: :visible)
first_module.click
assert_selector("#{MODULE_BTN_SELECT}.collapsed")
assert_selector("#{collapse_id}", visible: :hidden)
end
end

@Bubballoo3 Bubballoo3 merged commit b89d4b6 into master Feb 24, 2026
46 of 48 checks passed
@github-project-automation github-project-automation bot moved this from Awaiting Review to Merged/Closed in PR Review Pipeline Feb 24, 2026
@Bubballoo3 Bubballoo3 deleted the test-toggle-behavior-4929 branch February 24, 2026 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Merged/Closed

Development

Successfully merging this pull request may close these issues.

Test toggle behavior across dashboard

3 participants