Fix behat context & fixtures after latest merges

[stloyd]

No description provided.

[stloyd]

@Arn0d @pjedrzejewski After #1803 Behat context & user fixtures are broken.

[arnolanglade]

The build still failed :) but 👍

[arnolanglade]

@stloyd Do you know what happen for the feature "Users management" ?

[stloyd]

@Arn0d I'm trying to figure out that, but it seems that for Sylius we would need to revert part about setEmail() setting by default username... otherwise seems we need some "hacks" like overwriting user manager/provider services just to keep old "behavior".

[pjedrzejewski]

Why we need to do this?

[stloyd]

@pjedrzejewski @Arn0d Can you merge this? We can't fix this for Sylius in other way as we don't expose username anywhere. This change is simplest way to fix the problem, others would be IMO too hacky (additional user manager/provider or form handler).

[arnolanglade]

@pjedrzejewski I am for, what do you think? We need to find a way to fix travis (build exceeded more than 50minutes)

[stloyd]

@Arn0d I guess the only possible way is something like #1814.

[pjedrzejewski]

Thanks Joseph.

[arnolanglade]

👍