@dhyasama/totem-models 12.28.0 → 12.30.0

package/lib/Financials.js

@@ -120,6 +120,12 @@ module.exports = function(mongoose, config) {
             type: Schema.Types.ObjectId,
             refPath: 'metrics.sources.model',
           },
+          // On a derived/formula metric, the formula component (group) this
+          // cell belongs to — "this cell is part of Operating Expenses".
+          // Component totals live once per metric in `breakdown`
+          // ({ group: value }), keyed by these group names. Unset on raw
+          // extracted metrics.
+          group: { type: String, trim: true },
           position: {
             _id: false,
             page: { type: Number },
@@ -162,6 +168,7 @@ module.exports = function(mongoose, config) {
             _id: false,
             model: { type: String, enum: ['Document', 'Meeting', 'Message', 'Note'] },
             ref: { type: Schema.Types.ObjectId },
+            group: { type: String, trim: true },
             position: {
               _id: false,
               page: { type: Number },

package/lib/Organization.js

@@ -218,7 +218,8 @@ module.exports = function(mongoose, config) {
         },
 
         inbound: {
-          active: { type: Boolean, default: false }
+          active: { type: Boolean, default: false },
+          forward: [{ type: String, trim: true, lowercase: true }]
         },
 
         carta: {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dhyasama/totem-models",
-  "version": "12.28.0",
+  "version": "12.30.0",
   "author": "Jason Reynolds",
   "license": "UNLICENSED",
   "description": "Models for Totem platform",

package/test/Financials.js

@@ -130,6 +130,39 @@ describe('Financials', function() {
 
   });
 
+  it('getOrCreateByOrg returns one document under concurrent calls', async function() {
+
+    var customer = new mongoose.Types.ObjectId();
+    var org = new mongoose.Types.ObjectId();
+
+    // mirror production's unique index so the losing writer of a race can never insert a duplicate
+    await Financials.collection.createIndex({ customer: 1, organization: 1 }, { unique: true });
+
+    var results = await Promise.all(
+      Array.from({ length: 10 }, function() { return Financials.getOrCreateByOrg(customer, org); })
+    );
+
+    var ids = new Set(results.map(function(r) { return r._id.toString(); }));
+    ids.size.should.equal(1);
+
+    var count = await Financials.countDocuments({ customer: customer, organization: org });
+    count.should.equal(1);
+
+  });
+
+  it('getOrCreateByOrg returns the existing document', function(done) {
+
+    Financials.getOrCreateByOrg(customerA, orgA, function(err, result) {
+      should.not.exist(err);
+      should.exist(result);
+      result.customer.toString().should.equal(customerA.toString());
+      result.organization.toString().should.equal(orgA.toString());
+      should.exist(result.asOfDate); // proves it found the doc created above, not a fresh one
+      done();
+    });
+
+  });
+
   describe.skip('old so skip for now', function() {
 
     it('gets financials for an org', function(done) {