Fix bug, add tests, update i18n version

package.json

@@ -29,7 +29,7 @@
   "peerDependencies": {
     "@dojo/core": "~0.2.1",
     "@dojo/has": "~0.1.0",
-    "@dojo/i18n": "~0.1.1",
+    "@dojo/i18n": "~0.2.0",
     "@dojo/shim": "~0.2.2",
     "@dojo/widget-core": "~0.2.0"
   },

src/support/d.ts

@@ -129,21 +129,21 @@ function findByKey(
 	if (!target.children) {
 		return {};
 	}
-	let found: FoundNodeInfo<WNode | HNode> | undefined;
+	let nodeInfo: FoundNodeInfo<WNode | HNode> | undefined;
 	target.children
 		.forEach((child, index) => {
 			if (isNode(child)) {
-				if (found) {
+				if (nodeInfo && nodeInfo.found) {
 					if (findByKey(child, key, target, index).found) {
 						console.warn(`Duplicate key of "${typeof key === 'object' ? JSON.stringify(key) : key }" found.`);
 					}
 				}
 				else {
-					found = findByKey(child, key, target, index);
+					nodeInfo = findByKey(child, key, target, index);
 				}
 			}
 		});
-	return found || {};
+	return nodeInfo || {};
 }
 
 function  findByIndex(

tests/unit/support/d.ts

@@ -377,6 +377,7 @@ registerSuite('support/virtualDom', {
 
 		'key is in child'() {
 			assertRender(findKey(v('div', { key: 'bar' }, [ v('span', { key: 'foo' }), 'foo', null ]), 'foo')!, v('span', { key: 'foo' }), 'should find child node');
+			assertRender(findKey(v('div', { key: 'bar' }, [ v('span', { key: 'foo' }), v('span', { key: 'baz' }), 'foo', null ]), 'baz')!, v('span', { key: 'baz' }), 'should find child node');
 		},
 
 		'key is object'() {
@@ -426,7 +427,13 @@ registerSuite('support/virtualDom', {
 				])
 			]);
 
+			const noDuplicates = v('div', { key: 'foo' }, [
+				v('span', { key: 'parent1' }),
+				v('span', { key: 'parent2' })
+			]);
+
 			assertRender(findKey(fixture, key)!, v('i', { key, id: 'i1' }), 'should find first key');
+			assertRender(findKey(noDuplicates, 'parent2')!, v('span', { key: 'parent2' }), 'should find key');
 			assert.strictEqual(warnStub.callCount, 1, 'should have been called once');
 			assert.strictEqual(warnStub.lastCall.args[0], 'Duplicate key of "{}" found.', 'should have logged duplicate key');
 			warnStub.restore();