| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
| |
The DocuSign service rejects email addresses without a TLD, even though
they are valid.
|
|
|
|
|
| |
All files now have the same range, beginning from the conception of this
project.
|
|
|
|
|
|
|
|
|
|
| |
Copyright notices updated. More casual references to "LoVullo
Associates" replaced with "RT Specialty / Lovullo", which will be "RT
Specialty Buffalo" in the future. Or "RT Specialty", depending on how
this is rolled out. Or "Ryan Specialty Group". Who knows.
"R-T Specialty, LLC." is the legal name, which includes the dash. Not
to be confused with a certain television network.
|
|
|
|
|
|
|
|
|
|
|
| |
Classifications often yield scalar results. Since those results are
used directly in the diff, we have the situation where the expected
diff format (an array) is not provided. Consistent with the rest of
the system, we should consider a scalar to affect every index.
* src/validate/ValidStateMonitor.js (_checkCauseFix): Consider scalar
diffs to affect every index when checking for fixes.
* test/validate/ValidStateMonitorTest.js: Add test.
|
|
|
|
|
|
| |
Didn't mean for this to be committed.
* test/validate/ValidStateMonitorTest.js: Remove `debugger'.
|
|
|
|
|
|
|
|
|
| |
This was not properly chain the promises---it was always chaining on
the original _pending (so, the first request) and never clearing
_pending after that point.
* src/validate/DataValidator.js (_onceReady): Properly chain.
* test/validate/DataValidatorTest.js: Updated test. Don't ask.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Since we maintain state (as a kluge for the time being to integrate
with the rest of the system), we need to be careful to protect against
concurrent requests that might mess with it before the original
request is complete.
* src/validate/DataValidator.js
(validate, updateFailures): Hold concurrent requests.
(_onceReady): Add method.
* test/validate/DataValidatorTest.js: Add tests.
|
|
|
|
|
|
|
| |
This breaks everything. :x
This reverts commit e0c2e4dc861d523c40b2d35a242e48b8b7c99662, reversing
changes made to e610372c8443ba5c5e73ed0791e642d82772d5cd.
|
|
|
|
|
|
|
|
| |
Tidy up a little bit; this thing is a mess and hard to work with.
* test/validate/DataValidatorTest
(createStubs): Add function.
Use it.
|
|
|
|
|
|
| |
Not sure how this was missing...!
* test/validate/DataValidatorTest.js: Add test.
|
|
|
|
|
|
|
|
|
| |
* src/validate/DataValidator.js (validate): If a validation is
ongoing, queue requests.
* test/validate/DataValidatorTest.js: Add test.
DEV-2299
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
In practice, not clearing the store and allowing it to use previous
state has the effect of instantly "fixing" failures if there happens
to be more than one validation call.
This was a log of debugging for a one-line change.
* src/validate/DataValidator.js (_populateStore): Always clear store.
* test/validate/DataValidatorTest.js: Add test.
DEV-2299
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This allows clearing only the specified failures.
* src/validate/ValidStateMonitor.js
(clearFailures): Add `fields' argument. Make method more concise.
(_fixFailure): Handle clearing `_failures' record.
* src/validate/DataValidator.js
(clearFailures): Add `fields' argument.
* test/validate/ValidStateMonitorTest.js: Add test.
* test/validate/DataValidatorTest.js: Add test.
|
|
|
|
|
|
|
|
| |
* src/validate/DataValidator.js (clearFailures): Add public method.
* test/validate/DataValidatorTest.js: Add #clearFailures test.
* src/validate/ValidStateMonitor.js (clearFailures): Add public method.
* test/validate/ValidStateMonitorTest.js: Add #clearFailures test.
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* src/validate/DataValidator.js
(validate): New `classes' parameter. API BC break from previous
commits.
(populateStore, updateFailures): Generalize methods.
* test/validate/DataValidatorTest.js:
Add associated test.
Refactor existing tests to adhere to new API/expectations.
DEV-2206
|
|
|
|
|
|
|
|
|
|
|
|
| |
See the description in the test case. This is a bug fix.
* src/validate/ValidStateMonitor.js
(_checkCauseFix): Do not consider an empty diff on a past failure to
be a fix.
* test/validate/ValidStateMonitorTest.js: Add test.
DEV-2296
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
ValidStateMonitor now uses a Store in place of the original primitive
object-based diff format. The original format is translated by
DataValidator. The code is in a transitional state, and considering
the amount of time we spend on various areas of this project, will
likely stay this way for a while.
* src/validate/DataValidator.js
(__construct): Accept Store factory parameter.
(_store_factory): Add field.
(_createStores): Add method.
(_validate): Handle Store.
(updateFailures): Add method.
(_populateStore): Add method.
* test/validate/DataValidatorTest.js: Add tests.
* src/validate/ValidStateMonitor.js
(update): Enforce Store diff.
Wait to process failures until fixes are calculated.
(_checkFailureFix): Handle asynchronous, Promise-based diff.
(_checkCauseFix): Extract logic from _checkCauseFix.
* test/validate/ValidStateMonitorTest.js:
Modify test cases to be Promise-based and handle async calls where
appropriate. That was a friggin' expensive mess.
DEV-2296
|
|
|
|
|
|
|
|
|
|
| |
This extracts some gross code from Client in the rating-fw repo, which
is responsible for gluing change validations together.
* src/validate/DataValidator.js: Add class.
* test/validate/DataValidatorTest.js: Add test case.
DEV-2296
|
|
|
|
|
|
|
|
|
|
| |
Stepping stone to async. fix checks.
* src/validate/ValidStateMonitor.js
(update, detectFixes, _checkFailureFix): Return Promise.
* test/validate/ValidStateMonitorTest.js: Update to use promises.
DEV-2296
|
|
|
|
|
|
|
|
|
|
| |
This is a much more general solution.
* src/validate/formatter/Currency.js: Remove trait.
* test/validate/formatter/CurrencyTest.js: Remove test case.
* src/validate/formatter/StringFormat.js: Add trait.
* test/validate/formatter/StringFormatTest.js: Add test case.
|
|
|
|
|
|
|
| |
* src/validate/formatter/Number.js
(parse): Validate number format.
* test/validate/formatter/NumberTest.js: Modify accordingly.
|
|
|
|
|
|
|
|
|
|
| |
* src/validate/formatter/Number.js
(__mixin): Add mixin ctor.
(parse): Handle scale.
(styleNumber): Handle scale.
(scale, split): Add methods.
* test/validate/formatter/NumberTest.js: Modified accordingly.
|
|
|
|
|
| |
* src/validate/formatter/Currency.js: Add trait.
* test/validate/formatter/CurrencyTest.js: Add test case.
|
|
|
|
|
| |
* src/validate/formatter/MultiDimension.js: Added
* test/validate/formatter/MultiDimensionTest.js: Added
|
|
|
|
|
| |
* src/validate/formatter/AcceptReject.js: Added.
* test/validate/formatter/AcceptRejectTest.js: Added.
|
|
|
|
|
| |
* src/validate/formatter/insurance/Limit.js: Added
* test/validate/formatter/insurance/LimitTest.js: Added
|
|
|
|
|
| |
* src/validate/formatter/Number.js: Added
* test/validate/formatter/NumberTest.js: Added
|
|
|
|
|
|
|
|
| |
This will simplify, through composition, a number of other
validator-formatters.
* src/validate/formatter/MultiDelimited.js: Added
* test/validate/formatter/MultiDelimitedTest.js: Added
|
|
|
|
|
|
|
|
|
|
|
| |
This adds a great deal of flexibility through composition via trait
stacking.
* src/validate/formatter/UnorderedList.js: Renamed from
UnorderedListFormatter; now a trait.
* test/validate/formatter/UnorderedListTest.js: Renamed from
UnorderedListFormatterTest and adjusted to instantiate trait.
|
|
|
|
| |
* test/validate/formatter/common.js (testMixin): Added.
|
|
|
|
|
| |
* src/validate/formatter/EchoFormatter.js: Added.
* test/validate/formatter/EchoFormatterTest.js: Added.
|
|
|
|
|
|
|
|
| |
These have been refacored from the original: rather than abusing what is now
the PatternFormatter, it is now its own class.
* src/validate/formatter/UnorderedListFormatter.js: Added.
* test/validate/formatter/UnorderedListFormatterTest.js: Added.
|
|
|
|
|
|
|
| |
This has been modified from the original to use Chai instead of the `assert'
module.
* test/validate/formatter/common.js: Added.
|
|
|
|
|
|
|
| |
* src/validate/ValidatorFormatter.js: Interface added.
* src/validate/formatter/PatternFormatter.js: Renamed from ../VFormat.js.
* test/validate/formatter/PatternFormatterTest.js: Renamed from
../VFormatTest.js.
|
|
|
|
|
|
|
| |
This will likely undergo some refactoring.
* src/validate/VFormat.js: Added.
* test/validate/VFormatTest.js: Added.
|
|
|
|
|
|
|
|
|
| |
* src/validate/ValidStateMonitor.js (mergeFailures):
Another error on a field that previously failed will no longer overwrite the
previous failure, which caused issue when the causes changed (leaving fields
potentially unfixed).
* test/validate/ValidStateMonitorTest.js: Respective tests added.
|
|
|
|
|
| |
* src/validate/Failure.js (merge): Added
* test/validate/FailureTest.js: Respective tests added
|
|
|
|
|
|
|
|
|
|
|
| |
* src/validate/Failure.js (__construct): Takes an array of causes.
(getCauses): Now returns an array of causes.
* src/validate/ValidStateMonitor.js: Recognize fixes on array of causes.
* test/validate/FailureTest.js: Updated
* test/validate/ValidStateMonitorTest.js: Updated
|
|
|
|
|
|
|
| |
This maintains BC with the old string-based system.
* src/validate/ValidStateMonitor.js (_getCause): Added.
(detectFixes): Consider failure cause if available when checking for fixes.
|
|
|
|
|
|
|
| |
This is to facilitate a transition to Failure without a BC break.
* test/validate/ValidStateMonitorTest.js: Ensure that replacing messages
with Failure will continue to operate as expected.
|
|
|
|
|
| |
* src/validate/Failure.js: Added
* test/validate/FailureTtest.js: Added
|
|
|
|
|
|
|
| |
* src/validate/ValidStateMonitor.js (mergeFailures): Return count of new
failures
* test/validate/ValidStateMonitorTest.js: Respective test added
|
|
The intent is to ultimately replace ClientFieldValidator with this and
individual validators that interact with it.
* src/validate/ValidStateMonitor.js: Added
* test/validate/ValidStateMonitorTest.js: Added
|