@barchart/portfolio-api-common 1.2.40 → 1.2.44

package/lib/data/CorporateActionType.js

@@ -4,6 +4,15 @@ const Enum = require('@barchart/common-js/lang/Enum'),
 module.exports = (() => {
 	'use strict';
 
+	/**
+	 * Types of corporate actions.
+	 *
+	 * @public
+	 * @extends {Enum}
+	 * @param {String} code
+	 * @param {String} description
+	 * @param {Boolean} internal
+	 */
 	class CorporateActionType extends Enum {
 		constructor(code, description, internal) {
 			super(code, description);
@@ -11,23 +20,75 @@ module.exports = (() => {
 			this._internal = is.boolean(internal) && internal;
 		}
 
+		/**
+		 * If true, the corporate action is fictitious -- used only for internal
+		 * system purposes.
+		 *
+		 * @public
+		 * @returns {Boolean}
+		 */
 		get internal() {
 			return this._internal;
 		}
 
+		/**
+		 * A symbol change.
+		 *
+		 * @public
+		 * @static
+		 * @returns {CorporateActionType}
+		 */
+		static get SYMBOL_CHANGE() {
+			return symbolChange;
+		}
+
+		/**
+		 * A name change.
+		 *
+		 * @public
+		 * @static
+		 * @returns {CorporateActionType}
+		 */
+		static get NAME_CHANGE() {
+			return nameChange;
+		}
+
+		/**
+		 * A dividend.
+		 *
+		 * @public
+		 * @static
+		 * @returns {CorporateActionType}
+		 */
 		static get DIVIDEND() {
 			return dividend;
 		}
 
+		/**
+		 * A split.
+		 *
+		 * @public
+		 * @static
+		 * @returns {CorporateActionType}
+		 */
 		static get SPLIT() {
 			return split;
 		}
 
+		/**
+		 * A fictitious event, used for internal system purposes.
+		 *
+		 * @public
+		 * @static
+		 * @returns {CorporateActionType}
+		 */
 		static get JOB() {
 			return job;
 		}
 	}
 
+	const symbolChange = new CorporateActionType('SYMBOL_CHANGE', 'Symbol Change', false);
+	const nameChange = new CorporateActionType('NAME_CHANGE', 'Name Change', false);
 	const dividend = new CorporateActionType('DIVIDEND', 'Dividend', false);
 	const split = new CorporateActionType('SPLIT', 'Split', false);
 	const job = new CorporateActionType('JOB', 'Job', true);

package/lib/data/TransactionValidator.js

@@ -1,5 +1,6 @@
 const assert = require('@barchart/common-js/lang/assert'),
-	array = require('@barchart/common-js/lang/array');
+	array = require('@barchart/common-js/lang/array'),
+	is = require('@barchart/common-js/lang/is');
 
 const InstrumentType = require('./InstrumentType'),
 	PositionDirection = require('./PositionDirection'),
@@ -19,7 +20,7 @@ module.exports = (() => {
 		}
 
 		/**
-		 * Given a set of transaction, ensures that all sequence numbers and dates
+		 * Given an array of transactions, ensures that all sequence numbers and dates
 		 * are properly ordered.
 		 *
 		 * @public
@@ -32,7 +33,45 @@ module.exports = (() => {
 		}
 
 		/**
-		 * Given a set of transaction, returns the index of the first transaction that with an invalid
+		 * Given an array of transactions, when transaction references are present, ensures
+		 * that no transactions within the set reference the same transaction.
+		 *
+		 * @public
+		 * @static
+		 * @param {Array.<Object>} transactions
+		 * @returns {Boolean}
+		 */
+		static validateReferences(transactions) {
+			assert.argumentIsArray(transactions, 'transactions');
+
+			const references = { };
+
+			return transactions.every((t) => {
+				let valid = true;
+
+				if (is.object(t.reference) && is.string(t.reference.root) && is.number(t.reference.sequence)) {
+					const root = t.reference.root;
+					const sequence = t.reference.sequence;
+
+					if (!references.hasOwnProperty(root)) {
+						references[root] = [ ];
+					}
+
+					const sequences = references[root];
+
+					if (sequences.some(s => s === sequence)) {
+						valid = false;
+					} else {
+						sequences.push(sequence);
+					}
+				}
+
+				return valid;
+			});
+		}
+
+		/**
+		 * Given an array of transactions, returns the index of the first transaction that with an invalid
 		 * sequence number or date.
 		 *
 		 * @public

package/lib/serialization/TransactionSchema.js

@@ -211,6 +211,8 @@ module.exports = (() => {
 		.withField('sequence', DataType.NUMBER, true)
 		.withField('type', DataType.forEnum(TransactionType, 'TransactionType'))
 		.withField('instrument.name', DataType.STRING, true)
+		.withField('instrument.exchange', DataType.STRING, true)
+		.withField('instrument.code', DataType.NUMBER, true)
 		.withField('instrument.type', DataType.forEnum(InstrumentType, 'InstrumentType'), true)
 		.withField('instrument.currency', DataType.forEnum(Currency, 'Currency'), true)
 		.withField('instrument.symbol.barchart', DataType.STRING, true)
@@ -257,6 +259,8 @@ module.exports = (() => {
 		.withField('sequence', DataType.NUMBER, true)
 		.withField('type', DataType.forEnum(TransactionType, 'TransactionType'))
 		.withField('instrument.name', DataType.STRING, true)
+		.withField('instrument.exchange', DataType.STRING, true)
+		.withField('instrument.code', DataType.NUMBER, true)
 		.withField('instrument.type', DataType.forEnum(InstrumentType, 'InstrumentType'), true)
 		.withField('instrument.currency', DataType.forEnum(Currency, 'Currency'), true)
 		.withField('instrument.symbol.barchart', DataType.STRING, true)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@barchart/portfolio-api-common",
-  "version": "1.2.40",
+  "version": "1.2.44",
   "description": "Common classes used by the Portfolio system",
   "author": {
     "name": "Bryan Ingle",

package/test/SpecRunner.js

@@ -1027,7 +1027,8 @@ module.exports = (() => {
 
 },{"@barchart/common-js/lang/Enum":21,"@barchart/common-js/lang/assert":24}],5:[function(require,module,exports){
 const assert = require('@barchart/common-js/lang/assert'),
-	array = require('@barchart/common-js/lang/array');
+	array = require('@barchart/common-js/lang/array'),
+	is = require('@barchart/common-js/lang/is');
 
 const InstrumentType = require('./InstrumentType'),
 	PositionDirection = require('./PositionDirection'),
@@ -1047,7 +1048,7 @@ module.exports = (() => {
 		}
 
 		/**
-		 * Given a set of transaction, ensures that all sequence numbers and dates
+		 * Given an array of transactions, ensures that all sequence numbers and dates
 		 * are properly ordered.
 		 *
 		 * @public
@@ -1060,7 +1061,45 @@ module.exports = (() => {
 		}
 
 		/**
-		 * Given a set of transaction, returns the index of the first transaction that with an invalid
+		 * Given an array of transactions, when transaction references are present, ensures
+		 * that no transactions within the set reference the same transaction.
+		 *
+		 * @public
+		 * @static
+		 * @param {Array.<Object>} transactions
+		 * @returns {Boolean}
+		 */
+		static validateReferences(transactions) {
+			assert.argumentIsArray(transactions, 'transactions');
+
+			const references = { };
+
+			return transactions.every((t) => {
+				let valid = true;
+
+				if (is.object(t.reference) && is.string(t.reference.root) && is.number(t.reference.sequence)) {
+					const root = t.reference.root;
+					const sequence = t.reference.sequence;
+
+					if (!references.hasOwnProperty(root)) {
+						references[root] = [ ];
+					}
+
+					const sequences = references[root];
+
+					if (sequences.some(s => s === sequence)) {
+						valid = false;
+					} else {
+						sequences.push(sequence);
+					}
+				}
+
+				return valid;
+			});
+		}
+
+		/**
+		 * Given an array of transactions, returns the index of the first transaction that with an invalid
 		 * sequence number or date.
 		 *
 		 * @public
@@ -1258,7 +1297,7 @@ module.exports = (() => {
 	return TransactionValidator;
 })();
 
-},{"./InstrumentType":1,"./PositionDirection":2,"./TransactionType":4,"@barchart/common-js/lang/array":23,"@barchart/common-js/lang/assert":24}],6:[function(require,module,exports){
+},{"./InstrumentType":1,"./PositionDirection":2,"./TransactionType":4,"@barchart/common-js/lang/array":23,"@barchart/common-js/lang/assert":24,"@barchart/common-js/lang/is":26}],6:[function(require,module,exports){
 const array = require('@barchart/common-js/lang/array'),
 	assert = require('@barchart/common-js/lang/assert'),
 	ComparatorBuilder = require('@barchart/common-js/collections/sorting/ComparatorBuilder'),
@@ -4635,8 +4674,8 @@ module.exports = function () {
    * Compares two strings (in ascending order), using {@link String#localeCompare}.
    *
    * @static
-   * @param {Number} a
-   * @param {Number} b
+   * @param {String} a
+   * @param {String} b
    * @returns {Number}
    */
 		compareStrings: function compareStrings(a, b) {
@@ -5240,7 +5279,7 @@ module.exports = function () {
     * @returns {String}
     */
 			value: function format() {
-				return this._year + '-' + leftPad(this._month) + '-' + leftPad(this._day);
+				return leftPad(this._year, 4, '0') + '-' + leftPad(this._month, 2, '0') + '-' + leftPad(this._day, 2, '0');
 			}
 
 			/**
@@ -5357,7 +5396,7 @@ module.exports = function () {
     *
     * @static
     * @public
-    * @return {Day}
+    * @returns {Day}
     */
 
 		}, {
@@ -5449,8 +5488,11 @@ module.exports = function () {
 
 	var dayRegex = /^([0-9]{4}).?([0-9]{2}).?([0-9]{2})$/;
 
-	function leftPad(value) {
-		return value < 10 ? '0' + value : '' + value;
+	function leftPad(value, digits, character) {
+		var string = value.toString();
+		var padding = digits - string.length;
+
+		return '' + character.repeat(padding) + string;
 	}
 
 	var comparator = ComparatorBuilder.startWith(function (a, b) {
@@ -5805,7 +5847,7 @@ module.exports = function () {
     *
     * @public
     * @param {Decimal} instance
-    * @return {Boolean}
+    * @returns {Boolean}
     */
 			value: function getIsZero(instance) {
 				assert.argumentIsRequired(instance, 'instance', Decimal, 'Decimal');
@@ -5818,7 +5860,7 @@ module.exports = function () {
     *
     * @public
     * @param {Decimal} instance
-    * @return {Boolean}
+    * @returns {Boolean}
     */
 
 		}, {
@@ -5834,7 +5876,7 @@ module.exports = function () {
     *
     * @public
     * @param {Decimal} instance
-    * @return {Boolean}
+    * @returns {Boolean}
     */
 
 		}, {
@@ -5850,7 +5892,7 @@ module.exports = function () {
     *
     * @public
     * @param {Decimal} instance
-    * @return {Boolean}
+    * @returns {Boolean}
     */
 
 		}, {
@@ -5866,7 +5908,7 @@ module.exports = function () {
     *
     * @public
     * @param {Decimal} instance
-    * @return {Boolean}
+    * @returns {Boolean}
     */
 
 		}, {
@@ -5882,7 +5924,7 @@ module.exports = function () {
     *
     * @public
     * @param {Decimal} instance
-    * @return {Boolean}
+    * @returns {Boolean}
     */
 
 		}, {
@@ -6272,6 +6314,7 @@ module.exports = function () {
 		/**
    * The unique code.
    *
+   * @public
    * @returns {String}
    */
 
@@ -6284,6 +6327,7 @@ module.exports = function () {
     * Returns true if the provided {@link Enum} argument is equal
     * to the instance.
     *
+    * @public
     * @param {Enum} other
     * @returns {boolean}
     */
@@ -6308,6 +6352,7 @@ module.exports = function () {
     * Looks up a enumeration item; given the enumeration type and the enumeration
     * item's value. If no matching item can be found, a null value is returned.
     *
+    * @public
     * @param {Function} type - The enumeration type.
     * @param {String} code - The enumeration item's code.
     * @returns {*|null}
@@ -6327,6 +6372,7 @@ module.exports = function () {
 			/**
     * The description.
     *
+    * @public
     * @returns {String}
     */
 
@@ -6346,6 +6392,7 @@ module.exports = function () {
 			/**
     * Returns all of the enumeration's items (given an enumeration type).
     *
+    * @public
     * @param {Function} type - The enumeration to list.
     * @returns {Array}
     */
@@ -7193,7 +7240,7 @@ module.exports = function () {
 
 				if (typeof itemConstraint === 'function' && itemConstraint !== Function) {
 					itemValidator = function itemValidator(value, index) {
-						return value instanceof itemConstraint || itemConstraint(value, variableName + '[' + index + ']');
+						return itemConstraint.prototype !== undefined && value instanceof itemConstraint || itemConstraint(value, variableName + '[' + index + ']');
 					};
 				} else {
 					itemValidator = function itemValidator(value, index) {
@@ -9317,6 +9364,30 @@ describe('When validating transaction order', () => {
 	});
 });
 
+describe('When validating transaction references', () => {
+	'use strict';
+
+	const build = (root, sequence) => {
+		return { reference: { root: root, sequence: sequence } };
+	};
+
+	it('An array of zero transactions should be valid', () => {
+		expect(TransactionValidator.validateReferences([])).toEqual(true);
+	});
+
+	it('An array with no references should be valid', () => {
+		expect(TransactionValidator.validateReferences([ { }, { } ])).toEqual(true);
+	});
+
+	it('An array with distinct references should be valid', () => {
+		expect(TransactionValidator.validateReferences([ build('a', 1), build('a', 2), build('b', 1) ])).toEqual(true);
+	});
+
+	it('An array with non-distinct references should be not valid', () => {
+		expect(TransactionValidator.validateReferences([ build('a', 1), build('a', 2), build('b', 1), build('a', 2) ])).toEqual(false);
+	});
+});
+
 describe('When requesting all the user-initiated transaction types', () => {
 	'use strict';
 
@@ -9330,13 +9401,6 @@ describe('When requesting all the user-initiated transaction types', () => {
 		expect(userInitiated.length).toEqual(9);
 	});
 });
-
-describe('When validating direction', () => {
-	'use strict';
-
-
-});
-
 },{"./../../../lib/data/TransactionValidator":5,"@barchart/common-js/lang/Day":18}],37:[function(require,module,exports){
 const Currency = require('@barchart/common-js/lang/Currency'),
 	Decimal = require('@barchart/common-js/lang/Decimal');

package/test/specs/data/TransactionValidatorSpec.js

@@ -42,6 +42,30 @@ describe('When validating transaction order', () => {
 	});
 });
 
+describe('When validating transaction references', () => {
+	'use strict';
+
+	const build = (root, sequence) => {
+		return { reference: { root: root, sequence: sequence } };
+	};
+
+	it('An array of zero transactions should be valid', () => {
+		expect(TransactionValidator.validateReferences([])).toEqual(true);
+	});
+
+	it('An array with no references should be valid', () => {
+		expect(TransactionValidator.validateReferences([ { }, { } ])).toEqual(true);
+	});
+
+	it('An array with distinct references should be valid', () => {
+		expect(TransactionValidator.validateReferences([ build('a', 1), build('a', 2), build('b', 1) ])).toEqual(true);
+	});
+
+	it('An array with non-distinct references should be not valid', () => {
+		expect(TransactionValidator.validateReferences([ build('a', 1), build('a', 2), build('b', 1), build('a', 2) ])).toEqual(false);
+	});
+});
+
 describe('When requesting all the user-initiated transaction types', () => {
 	'use strict';
 
@@ -54,10 +78,4 @@ describe('When requesting all the user-initiated transaction types', () => {
 	it('Only nine types should be returned', () => {
 		expect(userInitiated.length).toEqual(9);
 	});
-});
-
-describe('When validating direction', () => {
-	'use strict';
-
-
-});
+});