Show template problem ID in tests.

[somiaj]

For users who can see the single problem grader, also show the template problem ID if the problems are in random order or not enumerated from 1 to N. This way instructors can see what the real problem ID is in the case it is different from the problem number.

I'm unsure on the best label for this, I went with Template ID as Template Problem ID was a bit too long and I thought it was more descriptive than Problem ID.

This was mentioned in #2248.

[drgrice1]

This looks good. I am not entirely certain that tying this to the permission to use the problem grader is correct. I am fine with that though.

[Alex-Jordan]

I don't have an opinion, but alternative permission levels to use that occur to me are can_show_source_file and modify_problem_sets. The first one feels closer to semantically correct, but not perfectly so. The latter one is a bit of a misnomer because it grants access to the Set Details page, regardless of whether you would save any changes or just view what is there. But I guess if someone needs to act on the problem numbers they would see from this, they would be someone who has to go to the Set Details page.

[somiaj]

My logic was pigging back off of being able to see the problem grader, since I assume it would be most useful for someone grading a test to know what the real problem ID is when they are randomized. I have no strong opinions on which permission is best, and didn't see one that would clearly be preferred.

[drgrice1]

Yes, @somiaj is correct. The problems are ordered by problem_id when they are obtained from the database. So the last element in the @problems array will have the largest problem_id.