Skip to content
This repository was archived by the owner on Dec 9, 2024. It is now read-only.

Commit 70282b3

Browse files
committed
Add a feed annotation to a trigger when creating a trigger
1 parent 251e7c4 commit 70282b3

File tree

2 files changed

+31
-2
lines changed

2 files changed

+31
-2
lines changed

deploy/lib/deployTriggers.js

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,12 @@ module.exports = {
88
if (this.options.verbose) {
99
this.serverless.cli.log(`Deploying Trigger: ${trigger.triggerName}`);
1010
}
11+
12+
const feed = this.getFeed(trigger)
13+
if (feed) {
14+
Object.assign(trigger, { annotations: [{ key: 'feed', value: feed }] });
15+
}
16+
1117
return ow.triggers.create(trigger)
1218
.then(() => {
1319
if (this.options.verbose) {
@@ -38,5 +44,9 @@ module.exports = {
3844
const trigger = { feed: undefined };
3945
return Object.keys(triggers)
4046
.map(t => Object.assign({}, triggers[t], trigger));
41-
}
47+
},
48+
49+
getFeed(trigger) {
50+
return trigger.feed;
51+
},
4252
};

deploy/tests/deployTriggers.js

Lines changed: 20 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,17 @@ describe('deployTriggers', () => {
1616
triggers: {
1717
myTrigger: {
1818
triggerName: 'myTrigger',
19-
namepspace: 'myNamespace',
19+
namespace: 'myNamespace',
2020
action: 'myAction',
2121
trigger: 'myTrigger',
2222
},
23+
feedTrigger: {
24+
triggerName: 'myTrigger',
25+
namespace: 'myNamespace',
26+
action: 'myAction',
27+
trigger: 'myTrigger',
28+
feed: '/whisk.system/alarms/alarm',
29+
},
2330
},
2431
};
2532

@@ -91,5 +98,17 @@ describe('deployTriggers', () => {
9198
})
9299
})
93100

101+
it('should deploy trigger with feed annotation to openwhisk', () => {
102+
sandbox.stub(openwhiskDeploy.provider, 'client', () => {
103+
const create = params => {
104+
expect(params).to.be.deep.equal(mockTriggerObject.triggers.feedTrigger);
105+
return Promise.resolve();
106+
};
107+
108+
return Promise.resolve({ triggers: { create } });
109+
});
110+
return expect(openwhiskDeploy.deployTrigger(mockTriggerObject.triggers.feedTrigger))
111+
.to.eventually.be.fulfilled;
112+
});
94113
});
95114
});

0 commit comments

Comments
 (0)