ShutlAdmin/webmock — comment on commit 2c324ed5ee01ab31335183314ddfb64f4f91bc80
File:  line 0
By bblimke:

Hi Mark,

Very cool idea with using the diff tool to spot the differences. It can be very useful in some circumstances.

I'm not sure webmock should depend on any external tools, but I can see you made it configurable, so it's an option.

Maybe it would be better to declare a need to use diff tool as webmock configuration. I.e WebMock.use_diff_tool or something.
Then this could be added temporarily in the test you're trying to debug, instead of enabling it for all test with env variable.
I don't think I would use this functionality for more than one test at once, since I wouldn't know which diff window is from which test.

I'm not sure the diff tool should be initialized within exception constructor. I would rather imagine that the exception is
independent and it has to be intercepted somewhere to initialize the diff tool.

If I have 5 stubs declared, it is going to open 5 windows at once, right? Did you test it with just one registered stub or many?

Bartosz