brodieG / unitizer

Easy R Unit Tests
Other
39 stars 7 forks source link

Creating Active Bindings in Tests Breaks Things #295

Open brodieG opened 1 year ago

brodieG commented 1 year ago

The problem is that https://github.com/brodieG/unitizer/blob/5ff79c3103d81646b3c8ce971124519fc4133d81/R/heal.R#L157 causes the assign form of an active binding to be called, and if the active binding does not accept the extra parameter for setting things break.

A solution is probably to delete the binding before assigning to it.

Related there is the issue that someone could create an explosive binding and we aren't catching errors in this part of the code.