Is using a JSON data type here appropriate?
Simply no, with a single index on UserID you can easy and fast request the groups and delete also simlpy user from  groups. 

That is all posible with json, but needs much more code and so can cause much more problems.

As long you don't need it in your application and also dont change it much, json is good to store a lot of data, but that is xml also.