bam

Philip Guo · Philip Guo · commit 7027c8eb2bc3 · 2011-09-30T19:14:13.000-04:00
diff --git a/edu-python-questions.js b/edu-python-questions.js
@@ -131,13 +131,13 @@ function genTestResultHandler(idx) {
   return ret;
 }
 
-function genDebugLinkHandler(testResult) {
+function genDebugLinkHandler(failingTestIndex) {
   function ret() {
-    //console.log('DEBUG', testResult);
+    console.log('DEBUG', failingTestIndex);
 
-    // TODO: switch back to visualize mode, populating all the edit
-    // fields with the input and outputs from the failing test input
-    // but keeping the user's submitted solution code unchanged
+    // TODO: switch back to visualize mode, populating the "test" input
+    // field with the failing test case, and RE-RUN the back-end to
+    // visualize execution (with proper IDs)
 
     return false; // don't reload the page
   }
@@ -288,7 +288,7 @@ function readyToGradeSubmission() {
       $("#gradeMatrix tr.gradeMatrixRow:last").append('<td>' + sadFaceImg + debugMeSpan + '</td>');
 
       $('#' + linkID).unbind(); // unbind it just to be paranoid
-      $('#' + linkID).click(genDebugLinkHandler(res));
+      $('#' + linkID).click(genDebugLinkHandler(i));
     }
   }
 

Original file line number	Diff line number	Diff line change
`@@ -131,13 +131,13 @@ function genTestResultHandler(idx) {`
`131`	`131`	`return ret;`
`132`	`132`	`}`
`133`	`133`
`134`		`-function genDebugLinkHandler(testResult) {`
	`134`	`+function genDebugLinkHandler(failingTestIndex) {`
`135`	`135`	`function ret() {`
`136`		`- //console.log('DEBUG', testResult);`
	`136`	`+ console.log('DEBUG', failingTestIndex);`
`137`	`137`
`138`		`- // TODO: switch back to visualize mode, populating all the edit`
`139`		`- // fields with the input and outputs from the failing test input`
`140`		`- // but keeping the user's submitted solution code unchanged`
	`138`	`+ // TODO: switch back to visualize mode, populating the "test" input`
	`139`	`+ // field with the failing test case, and RE-RUN the back-end to`
	`140`	`+ // visualize execution (with proper IDs)`
`141`	`141`
`142`	`142`	`return false; // don't reload the page`
`143`	`143`	`}`
`@@ -288,7 +288,7 @@ function readyToGradeSubmission() {`
`288`	`288`	`$("#gradeMatrix tr.gradeMatrixRow:last").append('<td>' + sadFaceImg + debugMeSpan + '</td>');`
`289`	`289`
`290`	`290`	`$('#' + linkID).unbind(); // unbind it just to be paranoid`
`291`		`- $('#' + linkID).click(genDebugLinkHandler(res));`
	`291`	`+ $('#' + linkID).click(genDebugLinkHandler(i));`
`292`	`292`	`}`
`293`	`293`	`}`
`294`	`294`