User OAuth missing tests?

[ilanbiala]

It seems like the User module is only being tested against local strategies, but the user can also be created through OAuth, which should probably be tested too, no?

[ilanbiala]

Any thoughts on how to do this? I can' think of any way to independently test the controller logic.

[sielay]

Controllers may need small refactor. I am looking in that at the moment, as I found some lacks in the flow.
#288

[ilanbiala]

@amoshaviv tests are probably more important than features, because I want to know that everything is always working. How can we test this?

[androidfred]

In this case, the issue referred to by sielay is arguably a bug and not a feature. @ilanbiala would you mean detaling more specifically what parts of the code you would like tests for?

[ilanbiala]

Oath link and callbacks and logic.

[lirantal]

Closing an old thread.
@ilanbiala @androidfred @sielay you're welcome to propose a PR if this issue is still relevant.