Rename 'map' method to 'reify', anticipating a future flag

Author: malthe

CHANGES.md

@@ -7,7 +7,7 @@ In next release ...
 - Query results are now generic with typing support for the `get`
   method on each row.
 
-  In addition, a new `map` method now produces records which conform
+  In addition, a new `reify` method now produces records which conform
   to the specified type. This method is available on all of the result
   objects.
 

src/result.ts

@@ -29,7 +29,7 @@ export class ResultRowImpl<T> {
         return this.data[i];
     }
 
-    map(): T {
+    reify(): T {
         const data = this.data;
         const result: Record<string, any> = {};
         this.names.forEach((key, i) => result[key] = data[i]);
@@ -71,7 +71,7 @@ export class Result<T = ResultRecord> {
         }
     }
 
-    map(): T[] {
+    reify(): T[] {
         return this.rows.map(data => {
             const result: Record<string, any> = {};
             this.names.forEach((key, i) => result[key] = data[i]);
@@ -116,15 +116,15 @@ export class ResultIterator<T = ResultRecord> extends Promise<Result<T>> {
         throw new Error('Query returned an empty result');
     }
 
-    map(): AsyncIterable<T> {
+    reify(): AsyncIterable<T> {
         const iterator: AsyncIterator<ResultRow<T>> = this[Symbol.asyncIterator]();
         return {
             [Symbol.asyncIterator]() {
                 return {
                     async next() {
                         const { done, value } = await iterator.next();
                         if (done) return { done, value: null };
-                        return { done, value: value.map() };
+                        return { done, value: value.reify() };
                     },
                     async return() {
                         if (iterator?.return) await iterator?.return();

test/result.test.ts

@@ -62,12 +62,12 @@ describe('Result', () => {
         expect(result.status).toEqual('SELECT 1');
         expect(result.names.length).toEqual(1);
         expect(result.names[0]).toEqual('message');
-        expect(result.map()).toEqual([{message: 'Hello world!'}]);
+        expect(result.reify()).toEqual([{message: 'Hello world!'}]);
         const rows = [...result];
         const row = rows[0];
         expect(row.get('message')).toEqual('Hello world!');
         expect(row.get('bad')).toEqual(undefined);
-        const mapped = row.map();
+        const mapped = row.reify();
         expect(mapped.message).toEqual('Hello world!');
     });
 
@@ -81,7 +81,7 @@ describe('Result', () => {
         const row = rows[0];
         const message: string = row.get('message');
         expect(message).toEqual('Hello world!');
-        const mapped: {message: string} = row.map();
+        const mapped: {message: string} = row.reify();
         expect(mapped.message).toEqual('Hello world!');
     });
 
@@ -121,7 +121,7 @@ describe('Result', () => {
         expect.assertions(1);
         const mapped = client.query(
             'select $1::text as message', ['Hello world!']
-        ).map();
+        ).reify();
         for await (const item of mapped) {
             expect(item).toEqual({message: 'Hello world!'});
         }