Remove code related to Pearson Testing Centers

[jzoldak]

@brianhw

[singingwolfboy]

🔥 KILL IT WITH FIRE 🔥

Er, I mean. Looks good to me. 👍

[wedaly]

@jzoldak Saw one of the Pearson tests fail on my branch. So there's an additional reason to merge this :)

[ormsbee]

The TimeLimitModule actually seems like it would be pretty useful to keep around. Does that need to be killed here?

[brianhw]

Okay, I didn't think that the TimeLimitModule was useful to anyone -- or at least that after eight months or so it had not been used by anyone other than a pearson exam. But we could be conservative and leave it around. Problem with it is that there is little in the way of tests, so if it doesn't work (after various changes to xblocks, for instance), we probably woudn't notice.

[brianhw]

Correction -- seems there are some TimeLimitModule tests. So if the TimeLimitModule stuff were restored, there's still lots of Pearson stuff to go, and it doesn't look too entwined.

[ichuang]

The TImeLimitModule has been of interest to instructors at MIT, though we
currently do not use it. This is for proctored exams, where students take
assessments online for credit in residential courses. If TimeLimitModule
were functional and documented we'd probably use it in the future.

On Tue, Oct 29, 2013 at 11:32 AM, brianhw notifications@github.com wrote:

Okay, I didn't think that the TimeLimitModule was useful to anyone -- or
at least that after eight months or so it had not been used by anyone other
than a pearson exam. But we could be conservative and leave it around.
Problem with it is that there is little in the way of tests, so if it
doesn't work (after various changes to xblocks, for instance), we probably
woudn't notice.

—
Reply to this email directly or view it on GitHubhttps://github.com/edx/edx-platform/pull/1299#issuecomment-27313284
.

[sarina]

ping? What's going on with this old PR?

[brianhw]

The PR needs to be modified to not include the removal of TimeLimitModule.

[sarina]

OK - if that's not going to be done anytime soon, can this PR be closed and re-opened when it's ready for re-consideration. Thanks.

[jzoldak]

Rebasing now to get this re-ready to merge.

Nobody that I know of now is using the current TimeLimitModule, and there are no integration tests for it. It could be broken by the ongoing XBlock / XModule shim work, and is one more thing to remember to test manually. If we need something like this it should be rewritten as an XBlock once XBlocks are in.

[ormsbee]

Point taken. Let's go ahead with this PR to kill off Pearson-related code and put it on the backlog to bring back just the TimeLimitModule then. Thank you.

[sarina]

@jzoldak I think one more rebase then it can be merged!

[jzoldak]

@ormsbee I need someone to help with a migration. I've never done one, and that's what's been holding me back from completing this work. Can you help?