Fix re-using a multirange codec already in codec cache (#765)

edgedb · Nov 7, 2023 · dc76cb6 · dc76cb6
1 parent d1f711b
commit dc76cb6
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/packages/driver/src/codecs/registry.ts b/packages/driver/src/codecs/registry.ts
@@ -198,6 +198,7 @@ export class CodecsRegistry {
         }
 
         case CTYPE_RANGE:
+        case CTYPE_MULTIRANGE:
         case CTYPE_SCALAR: {
           frb.discard(2);
           break;

diff --git a/packages/driver/test/client.test.ts b/packages/driver/test/client.test.ts
@@ -1372,6 +1372,15 @@ if (getEdgeDBVersion().major >= 4) {
       );
 
       expect(multiRangeRes).toEqual([expected]);
+
+      // query to make sure we exercise the code path where we re-use the
+      // multirange codec from the last query
+      await client.query(
+        `select {
+          multirange := <multirange<int32>>$mr
+        }`,
+        { mr: expected }
+      );
     } finally {
       await client.close();
     }