Fix Duplicate ID, Add Per-Problem Time Limit and Custom Answer Comparison (#545)

* Fix occasional duplicate testcase ids.

* Do an explicit null check.

Author: Acfboy

src/companion.ts

@@ -213,9 +213,10 @@ const handleNewProblem = async (problem: Problem) => {
 
     // Add fields absent in competitive companion.
     problem.srcPath = srcPath;
-    problem.tests = problem.tests.map((testcase) => ({
+    problem.tests = problem.tests.map((testcase, index) => ({
         ...testcase,
-        id: randomId(),
+        // Pass in index to avoid generating duplicate id
+        id: randomId(index),
     }));
     if (!existsSync(srcPath)) {
         writeFileSync(srcPath, '');

src/runTestCases.ts

@@ -64,7 +64,7 @@ const createLocalProblem = async (editor: vscode.TextEditor) => {
         url: srcPath,
         tests: [
             {
-                id: randomId(),
+                id: randomId(null),
                 input: '',
                 output: '',
             },

src/utils.ts

@@ -164,7 +164,13 @@ export const ocHide = () => {
     oc.hide();
 };
 
-export const randomId = () => Math.floor(Date.now() + Math.random() * 100);
+export const randomId = (index: number | null) => {
+    if (index !== null) {
+        return Math.floor(Date.now() + index);
+    } else {
+        return Math.floor(Date.now() + Math.random() * 100);
+    }
+};
 
 /**
  * Check if file is supported. If not, shows an error dialog. Returns true if