Skip to content

Commit b7d668d

Browse files
authoredFeb 10, 2023
Merge pull request #137 from Financial-Times/actually-remove-pingdom
NOPS-488 fully remove pingdom check
2 parents c564c57 + 483efef commit b7d668d

File tree

7 files changed

+8
-59
lines changed

7 files changed

+8
-59
lines changed
 

‎src/checks/index.js

-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ module.exports = {
55
responseCompare: require('./responseCompare.check'),
66
json: require('./json.check'),
77
string: require('./string.check'),
8-
pingdom: require('./pingdom.check'),
98
graphiteSpike: require('./graphiteSpike.check'),
109
graphiteThreshold: require('./graphiteThreshold.check'),
1110
graphiteWorking: require('./graphiteWorking.check'),

‎src/checks/pingdom.check.js

-16
This file was deleted.

‎test/fixtures/config/aggregate.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -4,19 +4,19 @@ module.exports = {
44
description: '',
55
checks: [
66
{
7-
type: 'pingdom',
7+
type: 'graphiteThreshold',
88
name: 'test1',
9-
checkId: '5555',
9+
metric: 'next.fake.metric',
1010
severity: 2,
1111
businessImpact: 'blah',
1212
technicalSummary: 'god knows',
1313
panicGuide: 'Don\'t Panic',
1414
interval: '1s'
1515
},
1616
{
17-
type: 'pingdom',
17+
type: 'graphiteThreshold',
1818
name: 'test2',
19-
checkId: '5656',
19+
metric: 'next.fake.metric',
2020
severity: 2,
2121
businessImpact: 'blah',
2222
technicalSummary: 'god knows',

‎test/fixtures/config/paywall.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@ module.exports = {
33
description: ' ',
44
checks: [
55
{
6-
type: 'pingdom',
6+
type: 'graphiteThreshold',
77
name: 'test',
8-
checkID: '112345',
98
severity: 2,
9+
metric: 'next.fake.metric',
1010
businessImpact: 'blah',
1111
technicalSummary: 'god knows',
1212
panicGuide: 'Don\'t Panic',

‎test/fixtures/config/pingdomCheckFixture.js

-15
This file was deleted.

‎test/healthchecks.spec.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
'use strict';
22

33
const expect = require('chai').expect;
4-
const PingdomCheck = require('../src/checks/').pingdom;
4+
const GraphiteThresholdCheck = require('../src/checks/').graphiteThreshold;
55

66
describe('Healthchecks', function(){
77

@@ -31,7 +31,7 @@ describe('Healthchecks', function(){
3131

3232
it('Should create new checks as described in the config', function(){
3333

34-
expect(healthchecks.checks[0]).to.be.an.instanceOf(PingdomCheck);
34+
expect(healthchecks.checks[0]).to.be.an.instanceOf(GraphiteThresholdCheck);
3535
});
3636

3737
it("Should report its status correctly", function(){

‎test/pingdom.check.spec.js

-19
This file was deleted.

0 commit comments

Comments
 (0)
Please sign in to comment.