diff options
| author | Robert Speicher <rspeicher@gmail.com> | 2017-09-02 01:13:00 +0300 |
|---|---|---|
| committer | Robert Speicher <rspeicher@gmail.com> | 2017-09-02 01:13:00 +0300 |
| commit | 7cc19cd5abb4ae8649e960c71d531adccb880a89 (patch) | |
| tree | 0e6e5d2cd9cae3b1b0ad6998d6647c38e9a7236a /spec/services | |
| parent | f0777fbb28a36b45035c235bf5a4c6c30f4c5db5 (diff) | |
Fix a spec failure dependent on load order
We were incorrectly stubbing `Issues::MoveService#execute`. This only
caused a failure when another spec caused it to be autoloaded --
`spec/controllers/projects/issues_controller_spec.rb` in this case.
Diffstat (limited to 'spec/services')
| -rw-r--r-- | spec/services/issues/update_service_spec.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/spec/services/issues/update_service_spec.rb b/spec/services/issues/update_service_spec.rb index 2176469aba6..85f46838351 100644 --- a/spec/services/issues/update_service_spec.rb +++ b/spec/services/issues/update_service_spec.rb @@ -519,7 +519,7 @@ describe Issues::UpdateService, :mailer do end it 'calls the move service with the proper issue and project' do - move_stub = class_double("Issues::MoveService").as_stubbed_const + move_stub = instance_double(Issues::MoveService) allow(Issues::MoveService).to receive(:new).and_return(move_stub) allow(move_stub).to receive(:execute).with(issue, target_project).and_return(issue) |