[V2] userIsInRole should check that id is a string

[Floriferous]

The following code doesn't work as I would expect:

const userId = Meteor.users.insert({});
Roles.setUserRoles(userId, 'admin');

Roles.userIsInRole(() => {}, 'admin'); // Returns true

The line here should check that the id is a string.

This used to fail properly in Roles V1.

[SimonSimCity]

Is this only related to v2 or also to v3?

[Floriferous]

Not sure, haven’t looked at v3 code.

[SimonSimCity]

Fixed in #324.