I think using constants here loses us more than we gain. Although constants help with re-factoring, they make the declarative request-form syntax harder to read. Since these references are encapsulated within the class itself (and so are neatly encapsulated from the outside world), and since if they weren't changed en-masse then the tests would fail anyway since the token names must actually exist, then I think it's easy to argue for keeping readability.