klajo / mockgyver

A mocking library for Erlang
BSD 3-Clause "New" or "Revised" License
38 stars 13 forks source link

Slight refactorisation for improved location of side-effects. #16

Closed tomas-abrahamsson closed 5 years ago

tomas-abrahamsson commented 5 years ago

I came to realize that the code I submitted in the #15 had some logic a odd places. (Sorry about the extra noise.) This PR is a small refactorization, to keep side-effectful decisions more central and helpers more free of side-effects: to move the unloading of a dynamic module from get_exported_fas_and_object_code up into mk_or_retrieve_mocked_mod, where it perhaps makes more sense.

klajo commented 5 years ago

Thanks!