Merge pull request #155 from wentout/proto

args fix

build/index.js

@@ -18,7 +18,7 @@ exports.lookup = function (TypeNestedPath) {
     const types = checkThis(this) ? exports.defaultTypes : this || exports.defaultTypes;
     return types.lookup(TypeNestedPath);
 };
-const apply = function (entity, Constructor, args) {
+const apply = function (entity, Constructor, args = []) {
     const result = new entity[Constructor.TypeName](...args);
     return result;
 };

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "mnemonica",
-	"version": "0.9.972",
+	"version": "0.9.973",
 	"description": "abstract technique that aids information retention : instance inheritance system",
 	"type": "commonjs",
 	"main": "./build/index.js",

src/index.ts

@@ -92,7 +92,7 @@ export const lookup = function (TypeNestedPath) {
 	return types.lookup(TypeNestedPath);
 } as TypeLookup;
 
-export const apply = function <E extends object, T extends object, S extends Proto<E, T>> (entity: E, Constructor: IDEF<T>, args?: unknown[]): {
+export const apply = function <E extends object, T extends object, S extends Proto<E, T>> (entity: E, Constructor: IDEF<T>, args: unknown[] = []): {
 	[key in keyof S]: S[key]
 } {
 	// const result = Constructor.apply(entity, args);

test/environment.js

@@ -49,6 +49,7 @@ const tests = (opts) => {
 		SubOfSomeADTCTypePre,
 		SubOfSomeADTCTypePost,
 		subOfSomeADTCInstanceA,
+		subOfSomeADTCInstanceANoArgs,
 		subOfSomeADTCInstanceC,
 		subOfSomeADTCInstanceB,
 		anotherNamespace,
@@ -351,7 +352,9 @@ const tests = (opts) => {
 
 				expect(SubOfSomeADTCTypePre.existentInstance).equal(someADTCInstance);
 				expect(SubOfSomeADTCTypePost.existentInstance).equal(someADTCInstance);
+				expect(subOfSomeADTCInstanceANoArgs.test).equal(123);
 				expect(subOfSomeADTCInstanceA.test).equal(123);
+				expect(subOfSomeADTCInstanceANoArgs.sub_test).equal(321);
 				expect(subOfSomeADTCInstanceA.sub_test).equal(321);
 				expect(subOfSomeADTCInstanceA.args).deep.to.equal([ 1, 2, 3 ]);
 

test/hookSamples.js

@@ -7,6 +7,8 @@ const namespaceFlowCheckerSample = [
 	'SubOfSomeADTCType',
 	'SubOfSomeADTCType',
 	'SubOfSomeADTCType',
+	'SubOfSomeADTCType',
+	'SubOfSomeADTCType',
 	'UserType',
 	'UserType',
 	'UserTypePL1',

test/hooks.js

@@ -37,7 +37,7 @@ const tests = (opts) => {
 			assert.equal(8, userTypeHooksInvocations.length);
 			debugger;
 			// +2
-			assert.equal(177, namespaceFlowCheckerInvocations.length);
+			assert.equal(179, namespaceFlowCheckerInvocations.length);
 			// +2
 			assert.equal(169, typesFlowCheckerInvocations.length);
 			// +1
@@ -48,10 +48,10 @@ const tests = (opts) => {
 			// +1
 			assert.equal(77, typesPostCreationInvocations.length);
 			// +1
-			assert.equal(96, namespacePreCreationInvocations.length);
+			assert.equal(97, namespacePreCreationInvocations.length);
 			// there are two registered Hooks, that is why
 			// +2
-			assert.equal(162, namespacePostCreationInvocations.length);
+			assert.equal(164, namespacePostCreationInvocations.length);
 		});
 	});
 

test/index.js

@@ -249,6 +249,7 @@ SubOfSomeADTCType.registerHook('postCreation', (opts) => {
 });
 
 // debugger;
+const subOfSomeADTCInstanceANoArgs = apply(someADTCInstance, SubOfSomeADTCType);
 const subOfSomeADTCInstanceA = apply(someADTCInstance, SubOfSomeADTCType, [ 1, 2, 3 ]);
 
 // debugger;
@@ -608,6 +609,7 @@ describe('Main Test', () => {
 		someADTCInstance,
 		SubOfSomeADTCTypePre,
 		SubOfSomeADTCTypePost,
+		subOfSomeADTCInstanceANoArgs,
 		subOfSomeADTCInstanceA,
 		subOfSomeADTCInstanceC,
 		subOfSomeADTCInstanceB,