[go: up one dir, main page]

Skip to content

Merge Request quick actions could use additional specs

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

Proposal

via @mayra-cabrera here

It'd be great if we could split the tests on this file somehow, or perhaps create a dedicate spec file for merge_request_actions.rb (e.g merge_request_actions_spec.rb), since this file is getting quite large (over 2k lines!).

The quick actions for merge requests have specs to interpret/parse the commands but don't actually have tests to make sure the commands worked. Ex: Does /assign_reviewer actually assign the reviewer to the MR?

Edited by 🤖 GitLab Bot 🤖