Jeremy Evans

Results 267 comments of Jeremy Evans

I am also of the opinion that we should calculate this at boot, store it in a string constant, and return that in a route (using the HTTP caching headers)....

Add a Clover singleton attr_reader (`singleton_class.attr_reader :ips_v4`), call `self.class.ips_v4` in the route, and override `Clover.ips_v4` in the spec (with the appropriate `allow_mocking` change). An added bonus is removing the `instance_variable_get`...

> might need to cache that some day (setting a header might be enough to let some kind of middleware take care of it), but that day doesn't have to...

Some commits seem to overlap with the already merged #4173. Can you rebase? The test failure should be solvable by updating thawed mock: ```ruby allow_mocking(PrivateSubnetAwsResource, :create_with_id) ```

Thank you for taking the time to try to reproduce. With your `settings.json`, I am not able to trigger the issue. With the following `settings.json` (a one character difference from...

Doing more research on this, in the documentation for error 5040, it states: ``` Overloading a method (by providing multiple signatures for the method) is only allowed for methods defined...

I confirmed that adding all overrides fixes the issue. Thank you very much for looking into this, and for providing a detailed explanation of why it occurs.