Borrar usos de OpenStruct
#651
Conversation
test/models/user_test.rb
Outdated
| test 'User.from_omniauth of a user that doesn\'t exist, should create a new user' do | ||
| auth = OpenStruct.new( | ||
| auth = Auth.new( |
There was a problem hiding this comment.
otra opción puede ser usar OmniAuth::AuthHash y OmniAuth::AuthHash::InfoHash que parecería que es lo que usa internamente omniauth, y por lo tanto lo que se le termina pasando en nuestro caso a este método
There was a problem hiding this comment.
Ahh no es mala! Lo único que me preocupa es que podría darnos problemas si después la gema omniauth decide, por ejemplo, cambiar los nombres de esas clases – lo cual resultaría en que fallen nuevamente estos tests 🤔
There was a problem hiding this comment.
Me da igual cualquiera de las dos opciones.
Pero no me parece mal que se rompa so omniauth cambia algo, porque de hecho eso puede causarnos problemas en producción
There was a problem hiding this comment.
Actualicé con esos cambios en ac31a90
This commits changes from defining our own custom classes in favor of `OmniAuth::AuthHash` and `OmniAuth::AuthHash::InfoHash` which are `OmniAuth` internal classes. Instances of those classes end up being passed to our `User.from_omniauth` method in production.
Al intentar actualizar
rollbara la versiónv3.6.1se rompen los tests con el siguiente error:El problema es que nuestra aplicación usa
OpenStructpero nunca la requerimos directamente – de todas formas funcionaba dado querollbarla requería en su código.Sin embargo, la versión
v3.6.1derollbardeja de requerirOpenStructpor lo que al actualizar a esa versión empiezan a fallar los tests.Para solucionarlo simplemente nos definimos dos clases
AuthyAuth::InfousandoStructque nos proveen la misma funcionalidad que estaba ofreciendo elOpenStruct.