git#651-Return BA status in the GET billing account endpoint

— fixing unit test

Author: Vikas Agarwal

src/routes/billingAccounts/get.spec.js

@@ -114,16 +114,6 @@ describe('Project Billing Accounts list', () => {
         .expect(403, done);
     });
 
-    it('should return 403 for admin', (done) => {
-      request(server)
-        .get(`/v5/projects/${project1.id}/billingAccount`)
-        .set({
-          Authorization: `Bearer ${testUtil.jwts.admin}`,
-        })
-        .send()
-        .expect(403, done);
-    });
-
     it('should return 404 if the project is not found', (done) => {
       request(server)
         .get('/v5/projects/11223344/billingAccount')
@@ -163,5 +153,26 @@ describe('Project Billing Accounts list', () => {
             }
           });
       });
+
+    it('should return billing account details using user token but without markup field',
+      (done) => {
+        request(server)
+          .get(`/v5/projects/${project1.id}/billingAccount`)
+          .set({
+            Authorization: `Bearer ${testUtil.jwts.admin}`,
+          })
+          .send()
+          .expect(200)
+          .end((err, res) => {
+            if (err) {
+              done(err);
+            } else {
+              const resJson = res.body;
+              resJson.tcBillingAccountId.should.be.eql(billingAccountData.tcBillingAccountId);
+              resJson.markup.should.not.exist;
+              done();
+            }
+          });
+      });
   });
 });