Missing note on stubbing ServiceActor::Result about actor HOT 4 CLOSED

Great thorough article! I’ve added it to the list of resources on the wiki

from actor.

For those landing here but living on the minitest land: https://fidalgo.pt/2023/07/23/service-objects-in-the-wild.html
(shameless plug)

from actor.

I’m closing this issue, but feel free to discuss/reopen if you think of a better way to handle this.

from actor.

RSpec couldn’t be able to know what methods are defined on results since these methods only make sense at the end, once it’s passed through different actors.

For your example, it would make sense to define an error method on ServiceActor::Result for this specific use case, as it’s an encouraged output name:

class ServiceActor::Result
  def error
    to_h[:error]
  end
end

However, this still wouldn’t work for other defined outputs.

You could use an instance_double and stub expected outputs with allow:

let(:result) { instance_double ServiceActor::Result, success?: false }

before do
  allow(Google::AutocompletePlaces).to receive(:result).and_return(result)
  allow(result).to receive(:error).and_return("lol")
end

In my case, though, I simply use double in these cases and disable this cop, since it can’t know when we can and cannot check the methods beforehand:

RSpec/VerifiedDoubles:
  # Allow the use of `double`.
  Enabled: false

from actor.