Nordstrom / serverless-artillery / 502

61%

master: 99%

push

travis-ci

Merge and Defaults Improvements + Bug Fixes

lib/lambda/funcHandle.js: improve merging of the event using lodash.merge and lodash.omit.  This enables the replacement of deep values in the script (so as to override portions rather than the entirety of an attribute in the script (e.g. a metrics sink configuration attribute within the script).  The alternative was duplicating the entirety of that attribute in the merge portion.
lib/lambda/funcHandle.spec.js: add deeper merge case, improve reflection of timeout as a milliseconds (not seconds) value.
lib/lambda/package.json: add lodash merge and omit

lib/lambda/taskDef.js: adjust default to compact the sampling traffic so as to reduce cost

lib/lambda/handler.js: the event executed is potentially not the event that specifies correct behavior.  Specifically, a single event produced from/by an acceptance/monitoring mode execution will look and act like a performance script irregardless of the expected treatment and behavior (i.e. report analysis and alerting), by adding the source event and using that this bug is fixed.
lib/lambda/handler.spec.js: add sourceEvent vs event check case.
lib/lambda/package.json: switch to a known license to avoid warning output from npm
lib/lambda/taskResult.js: fix bug where the settings had not been passed to analyzeMonitoring

lib/index.js: switch from using the ServerlessFiles array, indexing into it was fragile and caused bugs, to using ServerlessFile.  This duplicates the "serverless.yml" string but that's worth it.  Also fix function environment variables declarations that are references (rather than ending up with the reference function declarations as the values).  Resolve successful executions of invoke with the result returned by the target function (for use in tests or tools).  Add creation of `script.yml` if one doesn't exist, so that the serverless.yml's `${file(script.yml)}` reference is valid.  Last but not least, avoid new promise creation and synchronous write... (continued)

519 of 534 branches covered (97.19%)

Branch coverage included in aggregate %.

897 of 899 relevant lines covered (99.78%)

44.53 hits per line