Skip to content

Commit

Permalink
Read timeout from delegate definition. (#106)
Browse files Browse the repository at this point in the history
  • Loading branch information
dompuiu authored May 18, 2020
1 parent 2eec8bd commit f28b965
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@adobe/reactor-turbine",
"version": "26.0.0",
"version": "26.0.1",
"description": "Launch rule engine which processes rules on client websites and delegates logic to extensions.",
"license": "Apache-2.0",
"scripts": {
Expand Down
2 changes: 1 addition & 1 deletion src/rules/__tests__/createAddActionToQueue.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ describe('createAddActionToQueue returns a function that when called', function
normalizeRuleComponentError,
logActionError
)(
{ modulePath: 'action1', settings: { timeout: 10 } },
{ modulePath: 'action1', timeout: 10, settings: {} },
rule,
event,
lastPromiseInQueue
Expand Down
2 changes: 1 addition & 1 deletion src/rules/__tests__/createAddConditionToQueue.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ describe('createAddRuleToQueue returns a function that when called', function ()
var logConditionError = emptyFn;
var logConditionNotMet = emptyFn;
var lastPromiseInQueue = Promise.resolve();
var condition = { modulePath: 'condition1', settings: { timeout: 10 } };
var condition = { modulePath: 'condition1', timeout: 10, settings: {} };

return createAddConditionToQueue(
executeDelegateModule,
Expand Down
2 changes: 1 addition & 1 deletion src/rules/createAddActionToQueue.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ module.exports = function (
var timeoutId;

return new Promise(function (resolve, reject) {
var promiseTimeout = action.settings.timeout;
var promiseTimeout = action.timeout;

var moduleResult = executeDelegateModule(action, syntheticEvent, [
syntheticEvent
Expand Down
2 changes: 1 addition & 1 deletion src/rules/createAddConditionToQueue.js
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ module.exports = function (
var timeoutId;

return new Promise(function (resolve, reject) {
var promiseTimeout = condition.settings.timeout;
var promiseTimeout = condition.timeout;

timeoutId = setTimeout(function () {
// Reject instead of resolve to prevent subsequent
Expand Down

0 comments on commit f28b965

Please sign in to comment.