The following discussion from !105783 should be addressed: - [ ] @splattael started a [discussion](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/105783#note_1379818908): > **Suggestion** (non-blocking) Thoughts using `allow` instead? :thinking: > > Using expectations in hooks is considered bad practice and can lead to confusing spec failures. > > ```suggestion:-0+0 > allow(instance).to receive(:calculate_actual_state).with( > ``` > > Can we track fixing all `RSpec/ExpectInHook` offenses in a follow-up issue? :pray: 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