@trullock/firebase-common 1.2.0 → 1.3.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trullock/firebase-common",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "Collection of common firebase helpers",
   "homepage": "https://github.com/trullock/firebase#readme",
   "bugs": {

package/src/automapper.js

@@ -4,7 +4,7 @@ let mappings = [];
 /**
  * 
  * @param {function} toFirestore A function of the form (key: string, value: object/primitive) => object/primitive. Return a mapped value or undefined to fallback to default mapping
- * @param {function} fromFirestore A function of the form (key: string, value: object/primitive) => object/primitive. Return a mapped value or undefined to fallback to default mapping
+ * @param {function} fromFirestore A function of the form (key: string, value: object/primitive, dest: object) => object/primitive. Return a mapped value or undefined to fallback to default mapping
  */
 export function registerMapping(toFirestore, fromFirestore)
 {
@@ -237,7 +237,7 @@ export function autoMapFromFirestore(obj)
 			let handled = false;
 			for(let mapping of mappings)
 			{
-				let result = mapping.fromFirestore(key, value)
+				let result = mapping.fromFirestore(key, value, dest)
 				if(result !== undefined)
 				{
 					dest[key] = result;

package/tests/automapper.test.js

@@ -1,6 +1,6 @@
 import test from "node:test";
 import assert from "node:assert";
-import { autoMapFromFirestore, autoMapToFirestore, registerClassForPersistence } from "../src/automapper.js";
+import { autoMapFromFirestore, autoMapToFirestore, registerClassForPersistence, registerMapping } from "../src/automapper.js";
 import { Mixins } from "../src/utils.js";
 
 class AggregateRoot
@@ -70,6 +70,10 @@ class Baz extends Foo {
 
 class Academy extends Mixins([Baz, AggregateRoot])
 {
+	static Test = class {}
+
+	uniqueList = new Set();
+
 	changeName(name)
 	{
 		this.apply('academyNameChanged', {
@@ -86,9 +90,11 @@ class Academy extends Mixins([Baz, AggregateRoot])
 	{
 		super();
 		this.bar = new Baz.Bar();
+		this.uniqueList.add('a')
+		this.uniqueList.add('b')
+		this.uniqueList.add('b')
 	}
 
-	static Test = class {}
 }
 
 test('test', t => {
@@ -107,6 +113,17 @@ test('test', t => {
 test('automapping', t => {
 	registerClassForPersistence(Academy);
 
+	registerMapping((key, value) => {
+		if(value instanceof Set)
+			return Array.from(value)
+		return undefined;
+	}, (key, value, dest) => {
+		if(dest[key] instanceof Set)
+			return new Set(value);
+		return undefined;
+	})
+
+
 	let a = new Academy();
 	let mapped = autoMapToFirestore(a);
 	let unmapped = autoMapFromFirestore(mapped);
@@ -120,11 +137,14 @@ test('automapping', t => {
 	assert.equal(unmapped.bar.a, 1)
 
 	assert.equal(a.bar.barzy(), "barzy")
-	assert.equal(unmapped.bar.barzy(), "barzy")
+	assert.equal(unmapped.bar.barzy(), "barzy");
 
 	assert.equal(a.foozzy(), "foozzy")
 	assert.equal(unmapped.foozzy(), "foozzy")
 
 	assert.equal(a.bazzy(), "bazzy")
 	assert.equal(unmapped.bazzy(), "bazzy")
+
+	assert(a.uniqueList instanceof Set)
+	assert.equal(unmapped.uniqueList.size, 2)
 })