Skip to content

Add feature spec to test bulk delete

Miguel Rincon requested to merge add-bulk-delete-feature-spec into master

What does this MR do and why?

Add simple feature spec to test bulk delete

Why this change?

Our bulk delete feature will be extended to groups in #361721 (closed), so I want to add ensure it gets safely refactored with some more tests.

MR
Add feature spec to test bulk delete (!100941 - merged) You are here!
Refactor bulk delete frontend components (!101036 - merged)
Enable bulk delete for groups

Screenshots or screen recordings

This MR only updates tests, but this is the related feature:

  1. Visit admin runners
  2. Click the "Select all" checkbox http://gdk.test:3000/admin/runners
  3. Delete and confirm the modal
  4. Check all runners selected are removed.

image

How to set up and validate locally

No need, rspec is doing that for us now.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Miguel Rincon

Merge request reports

Loading