Bug: Use v2 reports instead of v1 reports (#843)

This commit is contained in:
Pranav Raj S
2020-05-11 11:40:14 +05:30
committed by GitHub
parent 0d7c1e5e11
commit 70341df9b8
3 changed files with 19 additions and 3 deletions

View File

@@ -1,10 +1,10 @@
/* global axios */
const API_VERSION = `/api/v1`;
const DEFAULT_API_VERSION = 'v1';
class ApiClient {
constructor(resource, options = {}) {
this.apiVersion = API_VERSION;
this.apiVersion = `/api/${options.apiVersion || DEFAULT_API_VERSION}`;
this.options = options;
this.resource = resource;
}

View File

@@ -3,7 +3,7 @@ import ApiClient from './ApiClient';
class ReportsAPI extends ApiClient {
constructor() {
super('reports', { accountScoped: true });
super('reports', { accountScoped: true, apiVersion: 'v2' });
}
getAccountReports(metric, since, until) {

View File

@@ -0,0 +1,16 @@
import reports from '../reports';
import ApiClient from '../ApiClient';
describe('#Reports API', () => {
it('creates correct instance', () => {
expect(reports).toBeInstanceOf(ApiClient);
expect(reports.apiVersion).toBe('/api/v2');
expect(reports).toHaveProperty('get');
expect(reports).toHaveProperty('show');
expect(reports).toHaveProperty('create');
expect(reports).toHaveProperty('update');
expect(reports).toHaveProperty('delete');
expect(reports).toHaveProperty('getAccountReports');
expect(reports).toHaveProperty('getAccountSummary');
});
});