workaround bug in matomo: showColumns=bounce_rate does not work with …

…API.get (#80)

* workaround bug in matomo: showColumns=bounce_rate does not work with API.get

* do not use showColumns for every Module.get API method, just to be sure no other core bugs surface

* add new test for querying a single metric from a ***.get method

* add new expected test file

src/data.ts

@@ -225,15 +225,16 @@ function getReportData(request: GoogleAppsScript.Data_Studio.Request<ConnectorPa
   }
 
   const SHOW_COLUMNS_UNSUPPORTED_METHODS = [
-    'VisitFrequency.get',
     'Contents.getContentPieces',
     'Contents.getContentNames',
     'Actions.getPageUrlsFollowingSiteSearch',
   ];
 
   let showColumns;
   // showColumns does not work correctly with some API methods
-  if (!SHOW_COLUMNS_UNSUPPORTED_METHODS.includes(apiMethod)) {
+  if (!SHOW_COLUMNS_UNSUPPORTED_METHODS.includes(apiMethod)
+    && reportParams.apiAction !== 'get' // showColumns does not work for API methods like API.get or VisitFrequency.get
+  ) {
     showColumns = (requestedFields.map(({name}) => name)).join(',');
   }
 
@@ -290,8 +291,8 @@ function getReportData(request: GoogleAppsScript.Data_Studio.Request<ConnectorPa
       filter_limit: `${limitToUse}`,
       filter_offset: `${offset}`,
       showColumns,
-      apiModule: null,
-      apiAction: null,
+      apiModule: undefined,
+      apiAction: undefined,
     };
 
     if (reportParams.apiModule !== 'Goals' && typeof params.idGoal === 'undefined') {

tests/appscript/data.spec.ts

@@ -438,6 +438,24 @@ describe('data', () => {
       expect(result).toEqual(getExpectedResponse(result, 'data', 'Events.getName_withDateDimension_date'));
     });
 
+    it('should correctly fetch a single column from a **.get API method', async () => {
+      let result = await Clasp.run('getData', {
+        configParams: {
+          idsite: env.APPSCRIPT_TEST_IDSITE,
+          report: JSON.stringify({ apiModule: 'API', apiAction: 'get' }),
+          filter_limit: 5,
+        },
+        dateRange: {
+          startDate: DATE_TO_TEST,
+          endDate: DATE_TO_TEST,
+        },
+        fields: [
+          { name: 'bounce_rate' },
+        ],
+      });
+      expect(result).toEqual(getExpectedResponse(result, 'data', `API.get_singleProcessedMetric`));
+    });
+
     it('should correctly fetch data for a date range spanning multiple days', async () => {
       await Clasp.run('setScriptProperties', {}, true);
 

tests/appscript/expected/data_API.get_singleProcessedMetric.json

@@ -0,0 +1,22 @@
+{
+  "filtersApplied": false,
+  "rows": [
+    {
+      "values": [
+        "0.73"
+      ]
+    }
+  ],
+  "schema": [
+    {
+      "dataType": "NUMBER",
+      "label": "Bounce Rate",
+      "name": "bounce_rate",
+      "semantics": {
+        "conceptType": "METRIC",
+        "isReaggregatable": false,
+        "semanticType": "PERCENT"
+      }
+    }
+  ]
+}