Add Type.forValue tests

packages/types/lib/types.js

@@ -1803,7 +1803,7 @@ function RecordType(schema, opts, scope) {
   // Force creation of the options object in case we need to register this
   // record's name.
   opts = opts || {};
-  scope = scope || Scope.forOptions(opts);
+  scope = scope ? scope.clone() : Scope.forOptions(opts);
 
   // Save the namespace to restore it as we leave this record's scope.
   if (schema.namespace !== undefined) {
@@ -2527,6 +2527,13 @@ function Resolver(writerType, readerType) {
 
 Resolver.prototype._peek = Type.prototype._peek;
 
+/**
+ * Type creation scope.
+ *
+ * Currently it holds the current namespace and the path to the type being
+ * generated (useful in type hooks when generating record names for
+ * `Type.forValue` for example).
+ */
 function Scope(namespace, path) {
   this.namespace = namespace;
   this.path = path || [];
@@ -2536,6 +2543,10 @@ Scope.forOptions = function (opts) {
   return new Scope(opts ? opts.namespace : undefined);
 };
 
+Scope.prototype.clone = function () {
+  return new Scope(this.namespace, this.path);
+};
+
 Scope.prototype.child = function (step) {
   return new Scope(this.namespace, this.path.concat('' + step));
 };

packages/types/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@avro/types",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Avro serialization",
   "homepage": "https://github.com/mtth/avsc",
   "keywords": [

packages/types/test/test-types.js

@@ -3721,6 +3721,34 @@ suite('types', function () {
         }
       }
     });
+
+    test('type hook array', function () {
+      var i = 1;
+      var t = infer([{foo: 2}, {foo: 3}], {typeHook: hook}).itemsType;
+      assert.equal(t.name, 'Foo3');
+      assert.equal(t.field('foo').type.typeName, 'int');
+      function hook(schema) {
+        if (schema.type !== 'record') {
+          return;
+        }
+        schema.name = 'Foo' + (i++);
+      }
+    });
+
+    test('type hook nested array', function () {
+      var i = 1;
+      var outer = infer([[{foo: 2}], [{foo: 3}]], {typeHook: hook});
+      var inner = outer.itemsType.itemsType;
+      assert.equal(inner.name, 'Foo3');
+      assert.equal(inner.field('foo').type.typeName, 'int');
+
+       function hook(schema) {
+        if (schema.type !== 'record') {
+          return;
+        }
+        schema.name = 'Foo' + (i++);
+      }
+    });
   });
 });