fix all TODOs

Author: prabalsingh24

test/e2e/RunScenario.spec.ts

@@ -85,8 +85,7 @@ describe('POST api/runs', () => {
             Accept: 'application/json'
         }).send(params);
 
-        expect(res.status).to.equal(200);
-        // TODO ? does not throw error.
+        expect(res.status).to.equal(400);
     });
 
     it('should throw 400 error when source is missing', async () => {

test/e2e/SubmitScenario.spec.ts

@@ -98,9 +98,7 @@ describe('POST api/submissions', () => {
             Accept: 'application/json'
         }).send(params);
 
-        // TODO WRONG, didn't throw error
-        expect(res.body.result).to.equal(expectedResult);
-        expect(res.status).to.equal(200);
+        expect(res.status).to.equal(400);
     });
 
     it('should throw error for source missing', async () => {

test/unit/validators/ApiKeyValidator.spec.ts

@@ -82,22 +82,25 @@ describe('API Key Validtors', async () => {
 
     it('should NOT reject api with a whitelist domain', async () => {
         const currentKey = 'SDLKJFLSJDKCWEKRJC';
+        const domainExample = 'www.google.com';
 
         await DB.apikeys.create({
             key: currentKey,
-            whitelist_domains: ['Referer']
+            whitelist_domains: [domainExample]
         });
 
         const req: Request = {
             header(name): any {
                 if (name === 'Authorization') {
-                    return `Bearer ${currentKey}`
+                    return `Bearer ${currentKey}`;
+                }
+                if (name === 'Referer') {
+                    return domainExample;
                 }
             }
         };
 
-        // TODO code is wrong
-        // expect(checkValidApiKey(req)).to.not.be.rejected;
+        expect(checkValidApiKey(req)).to.not.be.rejected;
     });
 
     it('should NOT reject api with whitelist ip/domain as "*"', async () => {

test/unit/validators/RunValidator.spec.ts

@@ -52,10 +52,8 @@ describe('RunValidator', async () => {
         };
 
         await runValidator.POST(req, res, nextSpy);
-        // TODO WRONG
-        // expect(sentStatus).to.equal(400);
-        // expect(sentData.err.message).to.equal('"lang" is required');
-        // expect(nextSpy.calledOnce).to.be.false;
+        expect(sentStatus).to.equal(400);
+        expect(nextSpy.calledOnce).to.be.false;
     });
 
     it('should throw an error when source is missing', async () => {

test/unit/validators/SubmitValidator.spec.ts

@@ -64,10 +64,9 @@ describe('SubmitValidator', async () => {
         };
 
         await submitValidator.POST(req, res, nextSpy);
-        // TODO WRONG
-        // expect(sentStatus).to.equal(400);
-        // expect(sentData.err.message).to.equal('"lang" is required');
-        // expect(nextSpy.calledOnce).to.be.false;
+
+        expect(sentStatus).to.equal(400);
+        expect(nextSpy.calledOnce).to.be.false;
     });
 
     it('should throw an error when source is missing', async () => {