You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
A bug was reported (#737) that the padrino instrumentation was not working correctly. After looking into it we found that the padrino instrumentation was not being installed in such a way that it would work. The instrumentation was being installed too late for us to prepend, as the ancestor chain had already been established on the objects using the modules we prepended our instrumentation to.
While we were able to correct the issues with instrumentation installation, we were not able to figure out a way to update our padrino tests to load the same way as running a padrino app outside of multiverse does. The tests do still test the functionality of the instrumentation, however it is not able to accurately test IF the instrumentation actually gets installed.
We should update our multiverse tests to load padrino in the same way an application using padrino does.
The text was updated successfully, but these errors were encountered:
A bug was reported (#737) that the padrino instrumentation was not working correctly. After looking into it we found that the padrino instrumentation was not being installed in such a way that it would work. The instrumentation was being installed too late for us to prepend, as the ancestor chain had already been established on the objects using the modules we prepended our instrumentation to.
While we were able to correct the issues with instrumentation installation, we were not able to figure out a way to update our padrino tests to load the same way as running a padrino app outside of multiverse does. The tests do still test the functionality of the instrumentation, however it is not able to accurately test IF the instrumentation actually gets installed.
We should update our multiverse tests to load padrino in the same way an application using padrino does.
The text was updated successfully, but these errors were encountered: