Remote Development: Fix all RSpec/ExpectInHook offenses
Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.
The following discussion from !105783 (merged) should be addressed:
-
@splattael started a discussion:
Suggestion (non-blocking) Thoughts using
allowinstead?🤔 Using expectations in hooks is considered bad practice and can lead to confusing spec failures.
allow(instance).to receive(:calculate_actual_state).with(Can we track fixing all
RSpec/ExpectInHookoffenses in a follow-up issue?🙏
Have a look in:
- ee/spec/lib/remote_development/workspaces/reconcile/agent_info_parser_spec.rb
- ee/spec/lib/remote_development/workspaces/reconcile/reconcile_processor_spec.rb
Edited by 🤖 GitLab Bot 🤖