fix: unit tests

Author: maxceem

src/routes/projects/get.spec.js

@@ -526,7 +526,7 @@ describe('GET Project', () => {
         });
       });
 
-      it('should not return "email" for any invite which has userId field', (done) => {
+      it('should not return "userId" for any invite which has "email" field', (done) => {
         request(server)
         .get(`/v5/projects/${project1.id}`)
         .set({
@@ -541,8 +541,8 @@ describe('GET Project', () => {
             const resJson = res.body;
             should.exist(resJson);
             resJson.invites.length.should.be.eql(1);
-            resJson.invites[0].should.have.property('userId');
-            should.not.exist(resJson.invites[0].email);
+            resJson.invites[0].should.have.property('email');
+            should.not.exist(resJson.invites[0].userId);
             done();
           }
         });

src/routes/projects/list.spec.js

@@ -382,10 +382,6 @@ describe('LIST Project', () => {
               const resJson = res.body;
               should.exist(resJson);
               resJson.should.have.lengthOf(2);
-              resJson[0].invites[0].should.have.property('userId');
-              should.not.exist(resJson[0].invites[0].email);
-              resJson[1].invites[0].should.have.property('userId');
-              should.not.exist(resJson[1].invites[0].email);
               done();
             }
           });
@@ -1081,8 +1077,8 @@ describe('LIST Project', () => {
               resJson.should.have.lengthOf(1);
               resJson[0].name.should.equal('test1');
               resJson[0].invites.should.have.lengthOf(2);
-              resJson[0].invites[0].should.have.property('userId');
-              should.not.exist(resJson[0].invites[0].email);
+              resJson[0].invites[0].should.have.property('email');
+              should.not.exist(resJson[0].invites[0].userId);
               resJson[0].invites[1].email.should.equal('h***o@w***d.com');
               done();
             }

src/util.spec.js

@@ -190,22 +190,24 @@ describe('Util method', () => {
       util.postProcessInvites('$.invites[?(@.email)]', detail, res).should.deep.equal(detail2);
     });
 
-    it('should not return emails for invite with defined userId', () => {
+    it('should not return `userId` for invite by email', () => {
       const detail = {
         id: 1,
         invites: [{
           id: 2,
           email: 'abcd@aaaa.com',
           userId: 33,
+          createdBy: 2,
         },
         ],
       };
       const detail2 = {
         id: 1,
         invites: [{
           id: 2,
-          email: null,
-          userId: 33,
+          email: 'abcd@aaaa.com',
+          userId: null,
+          createdBy: 2,
         },
         ],
       };