Skip to content

Commit

Permalink
Fix type errors in integration tests
Browse files Browse the repository at this point in the history
  • Loading branch information
cnasikas committed Mar 29, 2024
1 parent c38cebc commit e59c542
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1140,7 +1140,7 @@ export default function createUpdateTests({ getService }: FtrProviderContext) {
.send(getTestRuleData())
.expect(200);

objectRemover.add(space.id, createdRule.body.id, 'rule', 'alerting');
objectRemover.add(space.id, createdRule.id, 'rule', 'alerting');

for (const propertyToAdd of [
{ group: 'default' },
Expand Down Expand Up @@ -1205,7 +1205,7 @@ export default function createUpdateTests({ getService }: FtrProviderContext) {
.send(getTestRuleData())
.expect(200);

objectRemover.add(space.id, createdRule.body.id, 'rule', 'alerting');
objectRemover.add(space.id, createdRule.id, 'rule', 'alerting');

await supertestWithoutAuth
.put(`${getUrlPrefix(space.id)}/api/alerting/rule/${createdRule.id}`)
Expand Down Expand Up @@ -1252,7 +1252,7 @@ export default function createUpdateTests({ getService }: FtrProviderContext) {
.send(getTestRuleData())
.expect(200);

objectRemover.add(space.id, createdRule.body.id, 'rule', 'alerting');
objectRemover.add(space.id, createdRule.id, 'rule', 'alerting');

await supertestWithoutAuth
.put(`${getUrlPrefix(space.id)}/api/alerting/rule/${createdRule.id}`)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -711,8 +711,6 @@ export default ({ getService }: FtrProviderContext) => {
)
.expect(200);

objectRemover.add(space.id, createdRule1.id, 'rule', 'alerting');

const response = await supertestWithoutAuth
.patch(`${getUrlPrefix(space.id)}/internal/alerting/rules/_bulk_delete`)
.set('kbn-xsrf', 'foo')
Expand Down

0 comments on commit e59c542

Please sign in to comment.