@barchart/portfolio-api-common 1.0.99 → 1.0.103

package/lib/processing/PositionContainer.js

@@ -145,10 +145,13 @@ module.exports = (() => {
 						return list;
 					}, [ ]);
 
-					const missingGroups = array.difference(levelDefinition.requiredGroups.map(group => group.description), populatedGroups.map(group => group.description));
+					const missingGroups = array.difference(levelDefinition.requiredGroups.map(group => group.key), populatedGroups.map(group => group.key))
+						.map((key) => {
+							return levelDefinition.requiredGroups.find(g => g.key === key);
+						});
 
-					const empty = missingGroups.map((description) => {
-						return new PositionGroup(this, parent, [ ], levelDefinition.requiredGroups.find(group => group.description === description).currency, null, description);
+					const empty = missingGroups.map((group) => {
+						return new PositionGroup(this, parent, [ ], group.currency, group.key, group.description);
 					});
 
 					const compositeGroups = populatedGroups.concat(empty);
@@ -271,11 +274,7 @@ module.exports = (() => {
 	}
 
 	function findNode(tree, keys) {
-		return keys.reduce((tree, key) => {
-			tree = tree.findChild(group => group.description === key);
-
-			return tree;
-		}, tree);
+		return keys.reduce((tree, key) => tree.findChild(group => group.key === key), tree);
 	}
 
 	function getSummaryArray(ranges) {

package/lib/processing/PositionGroup.js

@@ -31,9 +31,16 @@ module.exports = (() => {
 
 			this._dataFormat = { };
 			this._dataActual = { };
-			
+
+			this._dataFormat.key = this._key;
 			this._dataFormat.description = this._description;
-			
+
+			if (this._single) {
+				this._dataFormat.instrument = items[0].position.instrument;
+			} else {
+				this._dataFormat.instrument = null;
+			}
+
 			this._dataActual.currentPrice = null;
 			this._dataActual.previousPrice = null;
 			this._dataActual.basis = null;

package/lib/processing/definitions/PositionLevelDefinition.js

@@ -14,7 +14,7 @@ module.exports = (() => {
 	 * @param {PositionLevelDefinition~keySelector} keySelector
 	 * @param {PositionLevelDefinition~descriptionSelector} descriptionSelector
 	 * @param {PositionLevelDefinition~currencySelector} currencySelector
-	 * @param {Array.<String>=} requiredGroups
+	 * @param {Array.<PositionLevelDefinition~RequiredGroup>=} requiredGroups
 	 * @param {Boolean=} single
 	 */
 	class PositionLevelDefinition {
@@ -139,5 +139,16 @@ module.exports = (() => {
 	 * @returns {Currency}
 	 */
 
+	/**
+	 * The data required to construct a group.
+	 *
+	 * @public
+	 * @typedef PositionLevelDefinition~RequiredGroup
+	 * @type {Object}
+	 * @property {String} key
+	 * @property {String} description
+	 * @property {Currency} currency
+	 */
+
 	return PositionLevelDefinition;
 })();

package/package.json

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

package/test/SpecRunner.js

@@ -861,10 +861,13 @@ module.exports = (() => {
 						return list;
 					}, [ ]);
 
-					const missingGroups = array.difference(levelDefinition.requiredGroups.map(group => group.description), populatedGroups.map(group => group.description));
+					const missingGroups = array.difference(levelDefinition.requiredGroups.map(group => group.key), populatedGroups.map(group => group.key))
+						.map((key) => {
+							return levelDefinition.requiredGroups.find(g => g.key === key);
+						});
 
-					const empty = missingGroups.map((description) => {
-						return new PositionGroup(this, parent, [ ], levelDefinition.requiredGroups.find(group => group.description === description).currency, null, description);
+					const empty = missingGroups.map((group) => {
+						return new PositionGroup(this, parent, [ ], group.currency, group.key, group.description);
 					});
 
 					const compositeGroups = populatedGroups.concat(empty);
@@ -987,11 +990,7 @@ module.exports = (() => {
 	}
 
 	function findNode(tree, keys) {
-		return keys.reduce((tree, key) => {
-			tree = tree.findChild(group => group.description === key);
-
-			return tree;
-		}, tree);
+		return keys.reduce((tree, key) => tree.findChild(group => group.key === key), tree);
 	}
 
 	function getSummaryArray(ranges) {
@@ -1035,9 +1034,16 @@ module.exports = (() => {
 
 			this._dataFormat = { };
 			this._dataActual = { };
-			
+
+			this._dataFormat.key = this._key;
 			this._dataFormat.description = this._description;
-			
+
+			if (this._single) {
+				this._dataFormat.instrument = items[0].position.instrument;
+			} else {
+				this._dataFormat.instrument = null;
+			}
+
 			this._dataActual.currentPrice = null;
 			this._dataActual.previousPrice = null;
 			this._dataActual.basis = null;
@@ -1578,7 +1584,7 @@ module.exports = (() => {
 	 * @param {PositionLevelDefinition~keySelector} keySelector
 	 * @param {PositionLevelDefinition~descriptionSelector} descriptionSelector
 	 * @param {PositionLevelDefinition~currencySelector} currencySelector
-	 * @param {Array.<String>=} requiredGroups
+	 * @param {Array.<PositionLevelDefinition~RequiredGroup>=} requiredGroups
 	 * @param {Boolean=} single
 	 */
 	class PositionLevelDefinition {
@@ -1703,6 +1709,17 @@ module.exports = (() => {
 	 * @returns {Currency}
 	 */
 
+	/**
+	 * The data required to construct a group.
+	 *
+	 * @public
+	 * @typedef PositionLevelDefinition~RequiredGroup
+	 * @type {Object}
+	 * @property {String} key
+	 * @property {String} description
+	 * @property {Currency} currency
+	 */
+
 	return PositionLevelDefinition;
 })();
 
@@ -6186,18 +6203,18 @@ describe('When a position container data is gathered', () => {
 		beforeEach(() => {
 			portfolios = [
 				{
-					portfolio: 'a',
+					portfolio: 'My First Portfolio',
 					name: 'a'
 				}, {
-					portfolio: 'b',
+					portfolio: 'My Second Portfolio',
 					name: 'b'
 				}
 			];
 
 			positions = [
-				getPosition('a', 'AAPL'),
-				getPosition('b', 'AAPL'),
-				getPosition('b', 'TSLA')
+				getPosition('My First Portfolio', 'AAPL'),
+				getPosition('My Second Portfolio', 'AAPL'),
+				getPosition('My Second Portfolio', 'TSLA')
 			];
 
 			summaries = [ ];
@@ -6211,7 +6228,7 @@ describe('When a position container data is gathered', () => {
 			beforeEach(() => {
 				definitions = [
 					new PositionTreeDefinition(name = 'the only tree', [
-						new PositionLevelDefinition('Total', x => true, x => 'Total', x => Currency.CAD),
+						new PositionLevelDefinition('Total', x => 'totals', x => 'Total', x => Currency.CAD),
 						new PositionLevelDefinition('Portfolio', x => x.portfolio.portfolio, x => x.portfolio.name, x => Currency.CAD),
 						new PositionLevelDefinition('Position', x => x.position.position, x => x.position.instrument.symbol.barchart, x =>  x.position.instrument.currency)
 					])
@@ -6225,27 +6242,27 @@ describe('When a position container data is gathered', () => {
 			});
 
 			it('the "Total" group should have two children groups', () => {
-				expect(container.getGroups(name, [ 'Total' ]).length).toEqual(2);
+				expect(container.getGroups(name, [ 'totals' ]).length).toEqual(2);
 			});
 
 			it('the "Total" group should have three items', () => {
-				expect(container.getGroup(name, [ 'Total' ]).items.length).toEqual(3);
+				expect(container.getGroup(name, [ 'totals' ]).items.length).toEqual(3);
 			});
 
 			it('The "a" portfolio group should have one child group', () => {
-				expect(container.getGroups(name, [ 'Total', 'a' ]).length).toEqual(1);
+				expect(container.getGroups(name, [ 'totals', 'My First Portfolio' ]).length).toEqual(1);
 			});
 
 			it('the "a" portfolio group should have one item', () => {
-				expect(container.getGroup(name, [ 'Total', 'a' ]).items.length).toEqual(1);
+				expect(container.getGroup(name, [ 'totals', 'My First Portfolio' ]).items.length).toEqual(1);
 			});
 
 			it('The "b" portfolio group should have two child groups', () => {
-				expect(container.getGroups(name, [ 'Total', 'b' ]).length).toEqual(2);
+				expect(container.getGroups(name, [ 'totals', 'My Second Portfolio' ]).length).toEqual(2);
 			});
 
 			it('the "b" portfolio group should have two items', () => {
-				expect(container.getGroup(name, [ 'Total', 'b' ]).items.length).toEqual(2);
+				expect(container.getGroup(name, [ 'totals', 'My Second Portfolio' ]).items.length).toEqual(2);
 			});
 		});
 	});

package/test/specs/processing/PositionContainerSpec.js

@@ -41,18 +41,18 @@ describe('When a position container data is gathered', () => {
 		beforeEach(() => {
 			portfolios = [
 				{
-					portfolio: 'a',
+					portfolio: 'My First Portfolio',
 					name: 'a'
 				}, {
-					portfolio: 'b',
+					portfolio: 'My Second Portfolio',
 					name: 'b'
 				}
 			];
 
 			positions = [
-				getPosition('a', 'AAPL'),
-				getPosition('b', 'AAPL'),
-				getPosition('b', 'TSLA')
+				getPosition('My First Portfolio', 'AAPL'),
+				getPosition('My Second Portfolio', 'AAPL'),
+				getPosition('My Second Portfolio', 'TSLA')
 			];
 
 			summaries = [ ];
@@ -66,7 +66,7 @@ describe('When a position container data is gathered', () => {
 			beforeEach(() => {
 				definitions = [
 					new PositionTreeDefinition(name = 'the only tree', [
-						new PositionLevelDefinition('Total', x => true, x => 'Total', x => Currency.CAD),
+						new PositionLevelDefinition('Total', x => 'totals', x => 'Total', x => Currency.CAD),
 						new PositionLevelDefinition('Portfolio', x => x.portfolio.portfolio, x => x.portfolio.name, x => Currency.CAD),
 						new PositionLevelDefinition('Position', x => x.position.position, x => x.position.instrument.symbol.barchart, x =>  x.position.instrument.currency)
 					])
@@ -80,27 +80,27 @@ describe('When a position container data is gathered', () => {
 			});
 
 			it('the "Total" group should have two children groups', () => {
-				expect(container.getGroups(name, [ 'Total' ]).length).toEqual(2);
+				expect(container.getGroups(name, [ 'totals' ]).length).toEqual(2);
 			});
 
 			it('the "Total" group should have three items', () => {
-				expect(container.getGroup(name, [ 'Total' ]).items.length).toEqual(3);
+				expect(container.getGroup(name, [ 'totals' ]).items.length).toEqual(3);
 			});
 
 			it('The "a" portfolio group should have one child group', () => {
-				expect(container.getGroups(name, [ 'Total', 'a' ]).length).toEqual(1);
+				expect(container.getGroups(name, [ 'totals', 'My First Portfolio' ]).length).toEqual(1);
 			});
 
 			it('the "a" portfolio group should have one item', () => {
-				expect(container.getGroup(name, [ 'Total', 'a' ]).items.length).toEqual(1);
+				expect(container.getGroup(name, [ 'totals', 'My First Portfolio' ]).items.length).toEqual(1);
 			});
 
 			it('The "b" portfolio group should have two child groups', () => {
-				expect(container.getGroups(name, [ 'Total', 'b' ]).length).toEqual(2);
+				expect(container.getGroups(name, [ 'totals', 'My Second Portfolio' ]).length).toEqual(2);
 			});
 
 			it('the "b" portfolio group should have two items', () => {
-				expect(container.getGroup(name, [ 'Total', 'b' ]).items.length).toEqual(2);
+				expect(container.getGroup(name, [ 'totals', 'My Second Portfolio' ]).items.length).toEqual(2);
 			});
 		});
 	});