mirror of
https://github.com/grafana/grafana.git
synced 2025-02-25 18:55:37 -06:00
loki: adds proper error handling for config page
This handles three different error cases for the config page so that the user gets an error and not just a blank red error box.
This commit is contained in:
parent
721e1d7555
commit
40898e1cd6
98
public/app/plugins/datasource/loki/datasource.test.ts
Normal file
98
public/app/plugins/datasource/loki/datasource.test.ts
Normal file
@ -0,0 +1,98 @@
|
|||||||
|
import LokiDatasource from './datasource';
|
||||||
|
|
||||||
|
describe('LokiDatasource', () => {
|
||||||
|
const instanceSettings = {
|
||||||
|
url: 'myloggingurl',
|
||||||
|
};
|
||||||
|
|
||||||
|
describe('when performing testDataSource', () => {
|
||||||
|
let ds;
|
||||||
|
let result;
|
||||||
|
|
||||||
|
describe('and call succeeds', () => {
|
||||||
|
beforeEach(async () => {
|
||||||
|
const backendSrv = {
|
||||||
|
async datasourceRequest() {
|
||||||
|
return Promise.resolve({
|
||||||
|
status: 200,
|
||||||
|
data: {
|
||||||
|
values: ['avalue'],
|
||||||
|
},
|
||||||
|
});
|
||||||
|
},
|
||||||
|
};
|
||||||
|
ds = new LokiDatasource(instanceSettings, backendSrv, {});
|
||||||
|
result = await ds.testDatasource();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return successfully', () => {
|
||||||
|
expect(result.status).toBe('success');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('and call fails with 401 error', () => {
|
||||||
|
beforeEach(async () => {
|
||||||
|
const backendSrv = {
|
||||||
|
async datasourceRequest() {
|
||||||
|
return Promise.reject({
|
||||||
|
statusText: 'Unauthorized',
|
||||||
|
status: 401,
|
||||||
|
data: {
|
||||||
|
message: 'Unauthorized',
|
||||||
|
},
|
||||||
|
});
|
||||||
|
},
|
||||||
|
};
|
||||||
|
ds = new LokiDatasource(instanceSettings, backendSrv, {});
|
||||||
|
result = await ds.testDatasource();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return error status and a detailed error message', () => {
|
||||||
|
expect(result.status).toEqual('error');
|
||||||
|
expect(result.message).toBe('Loki: Unauthorized. 401. Unauthorized');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('and call fails with 404 error', () => {
|
||||||
|
beforeEach(async () => {
|
||||||
|
const backendSrv = {
|
||||||
|
async datasourceRequest() {
|
||||||
|
return Promise.reject({
|
||||||
|
statusText: 'Not found',
|
||||||
|
status: 404,
|
||||||
|
data: '404 page not found',
|
||||||
|
});
|
||||||
|
},
|
||||||
|
};
|
||||||
|
ds = new LokiDatasource(instanceSettings, backendSrv, {});
|
||||||
|
result = await ds.testDatasource();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return error status and a detailed error message', () => {
|
||||||
|
expect(result.status).toEqual('error');
|
||||||
|
expect(result.message).toBe('Loki: Not found. 404. 404 page not found');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
|
||||||
|
describe('and call fails with 502 error', () => {
|
||||||
|
beforeEach(async () => {
|
||||||
|
const backendSrv = {
|
||||||
|
async datasourceRequest() {
|
||||||
|
return Promise.reject({
|
||||||
|
statusText: 'Bad Gateway',
|
||||||
|
status: 502,
|
||||||
|
data: '',
|
||||||
|
});
|
||||||
|
},
|
||||||
|
};
|
||||||
|
ds = new LokiDatasource(instanceSettings, backendSrv, {});
|
||||||
|
result = await ds.testDatasource();
|
||||||
|
});
|
||||||
|
|
||||||
|
it('should return error status and a detailed error message', () => {
|
||||||
|
expect(result.status).toEqual('error');
|
||||||
|
expect(result.message).toBe('Loki: Bad Gateway. 502');
|
||||||
|
});
|
||||||
|
});
|
||||||
|
});
|
||||||
|
});
|
@ -140,7 +140,23 @@ export default class LokiDatasource {
|
|||||||
};
|
};
|
||||||
})
|
})
|
||||||
.catch(err => {
|
.catch(err => {
|
||||||
return { status: 'error', message: err.message };
|
let message = 'Loki: ';
|
||||||
|
if (err.statusText) {
|
||||||
|
message += err.statusText;
|
||||||
|
} else {
|
||||||
|
message += 'Cannot connect to Loki';
|
||||||
|
}
|
||||||
|
|
||||||
|
if (err.status) {
|
||||||
|
message += `. ${err.status}`;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (err.data && err.data.message) {
|
||||||
|
message += `. ${err.data.message}`;
|
||||||
|
} else if (err.data) {
|
||||||
|
message += `. ${err.data}`;
|
||||||
|
}
|
||||||
|
return { status: 'error', message: message };
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user