feat: Updated cert sending (#8)

Author: mamari90

src/utils.js

@@ -67,7 +67,7 @@ function telemetrySender(client){
             client.trackAvailability(apiTelemetryData);
         }
 
-        if (metricContext.certMetrics && Object.keys(metricContext.certMetrics).length > 0){
+        if (metricContext.certMetrics && Object.keys(metricContext.certMetrics).length > 0 && metricContext.monitoringConfiguration.checkCertificate){
             let certTelemetryData = statics.enrichData(metricContext.baseTelemetryData, metricContext.certMetrics, constants.keysForTelemetry);
             console.log(`tracking cert telemetry for ${metricContext.testId}: ${JSON.stringify(certTelemetryData)}`)
             client.trackAvailability(certTelemetryData);

tests/utils.test.js

@@ -238,6 +238,25 @@ describe('telemetrySender tests', () => {
             expect(trackAvailability).toHaveBeenCalledTimes(1);
         })
     });
+
+    test('trackAvailability not called when cert metric provided but checkCertificate is false', () => {
+
+        dummyMetricContex.certMetrics = {
+            'duration': 100, 
+            'targetStatus': 1,
+            'targetExpirationTimestamp': 1000,
+            'httpStatus': 200,
+            'targetTlsVersion': 1.3,
+            'targetExpireInDays': 8,
+            'domain': "foo",
+            'checkCert': true
+        }
+        dummyMetricContex.monitoringConfiguration.checkCertificate = false
+      
+        return utils.telemetrySender(dummyTelemetryClient)(dummyMetricContex).then(data =>{
+            expect(trackAvailability).toHaveBeenCalledTimes(0);
+        })
+    });
 
 })