pure-orm 2.0.1 → 3.0.0

package/README.md

@@ -76,7 +76,7 @@ class PersonDAO extends BaseDAO {
 And use it like this:
 
 ```javascript
-const person = personDAO.get(55);
+const person = await personDAO.get({ id: 55 });
 console.log(person);
 ```
 
@@ -100,7 +100,7 @@ Person {
       Job {
         id: 278,
         personId: 55,
-        employerId: 26
+        employerId: 26,
         startDate: '2021-01-01',
         endDate: '2021-12-31',
         employer: Employer {
@@ -115,11 +115,37 @@ Person {
 
 > This is a quick showcase. To see how to wire up this code, see the full [Practical Example](#practical-example) below.
 
-Things to note:
+### Things to Note:
 
-- Our DAO returns a single Person business object which is properly structured from the relational row records!
+- Our DAO returns a single Person business object which is properly structured from the many relational row records!
 - Our query is executed with a `one` method. The DAO methods for `one`, `oneOrNone`, `many`, `any` ensure their count against the number of generated top level business objects - not the number of relational row records the sql expression returns!
-- Rather than manually specifying our columns in the sql select expression, we use the business object's `getSQLSelectClause`. This is purely a convenience method which namespaces each column with the table name prefix to ensure column names don't collide. (You are welcome to do this by hand if you don't mind the tedium.)
+- Rather than manually specifying our columns in the sql select expression, we use the business object's `getSQLSelectClause`. This is purely a convenience method which namespaces each column with the table name prefix to ensure column names don't collide (for example, the person, job, and employer `id`s would collide if not namespaced, as would person and employer `name`s). You are welcome to do this by hand instead of using the convenience methods (as were used above), if you don't mind the tedium:
+  ```javascript
+  class PersonDAO extends BaseDAO {
+    get({ id }) {
+      // Example showing you can manually specific the select expression fields
+      // instead of using a business object's `getSQLSelectClause` convenience
+      // method. Note: you must namespace the field with table name and hashtag.
+      const query = `
+        SELECT
+          person.id as "person#id",
+          person.name as "person#name",
+          job.id as "job#id",
+          job.person_id: "job#person_id",
+          job.employer_id: "job#employer_id",
+          job.start_date: "job#start_date",
+          job.end_date: "job#end_date",
+          employer.id as "employer#id",
+          employer.name as "employer#name"
+        FROM person
+        JOIN job on person.id = job.person_id
+        JOIN employer on job.employer_id = employer.id
+        WHERE id = $(id)
+      `;
+      return this.one(query, { id });
+    }
+  }
+  ```
 
 ## Usage
 
@@ -160,18 +186,18 @@ const renderProfile = (req, res) => {
           endDate: '2020-12-31',
           employer: {
             id: 17,
-            name: 'Good Corp',
+            name: 'Good Corp'
           }
         },
         {
           id: 278,
           personId: 55,
-          employerId: 26
+          employerId: 26,
           startDate: '2021-01-01',
           endDate: '2021-12-31',
           employer: {
             id: 26,
-            name: 'Better Corp',
+            name: 'Better Corp'
           }
         }
       ]
@@ -233,7 +259,7 @@ const renderProfile = (req, res) => {
 -       {
 -         id: 278,
 -         personId: 55,
--         employerId: 26
+-         employerId: 26,
 -         startDate: '2021-01-01',
 -         endDate: '2021-12-31',
 -         employer: {
@@ -244,8 +270,7 @@ const renderProfile = (req, res) => {
 -     ]
 -   }
 - };
-+ const personId = req.params.id;
-+ const person = personDAO.get({personId});
++ const person = personDAO.get({ id: req.params.id });
   res.render('profile.html', person);
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pure-orm",
-  "version": "2.0.1",
+  "version": "3.0.0",
   "main": "src/index.js",
   "engines": {
     "node": ">=6"

package/src/bo/base-bo.js

@@ -156,14 +156,19 @@ module.exports = ({ getBusinessObjects }) =>
           const bo = nodeAlreadySeen || _bo;
           const isNodeAlreadySeen = !!nodeAlreadySeen;
           const nodePointingToIt = nodes.find(node => {
-            const index = Object.values(node.constructor.references).indexOf(
-              bo.constructor
-            );
-            if (index === -1) {
+            const indexes = Object.values(node.constructor.references)
+              .map((x, i) => (x === bo.constructor ? i : null))
+              .filter(x => x != null);
+            if (!indexes.length) {
               return false;
             }
-            const property = Object.keys(node.constructor.references)[index];
-            return node[property] === bo.id;
+            for (const index of indexes) {
+              const property = Object.keys(node.constructor.references)[index];
+              if (node[property] === bo.id) {
+                return true;
+              }
+            }
+            return false;
           });
           // For first obj type which is has an instance in nodes array,
           // get its index in nodes array
@@ -180,8 +185,8 @@ module.exports = ({ getBusinessObjects }) =>
             return null;
           }, null);
           const parentHeirarchy = [
-            ...nodes.slice(0, indexOfOldestParent + 1).reverse(),
-            root
+            root,
+            ...nodes.slice(0, indexOfOldestParent + 1).reverse()
           ];
           const nodeItPointsTo = parentHeirarchy.find(parent => {
             const index = Object.values(bo.constructor.references).indexOf(
@@ -205,7 +210,7 @@ module.exports = ({ getBusinessObjects }) =>
             // shown to be the parent (of parcel_events).
             const ec = bo[nodePointingToIt.BoCollection.displayName];
             if (ec && ec.models.find(m => m === nodePointingToIt)) {
-              // nodes = [bo, ...nodes];
+              nodes = [bo, ...nodes];
               return;
             }
           }
@@ -278,13 +283,13 @@ module.exports = ({ getBusinessObjects }) =>
     getSqlInsertParts() {
       const columns = this.constructor.sqlColumns
         .filter(
-          (column, index) => this[this.constructor.columns[index]] != null
+          (column, index) => this[this.constructor.columns[index]] !== void 0
         )
         .map(col => `"${col}"`)
         .join(', ');
       const values = this.constructor.columns
         .map(column => this[column])
-        .filter(value => value != null);
+        .filter(value => value !== void 0);
       const valuesVar = values.map((value, index) => `$${index + 1}`);
       return { columns, values, valuesVar };
     }
@@ -292,14 +297,14 @@ module.exports = ({ getBusinessObjects }) =>
     getSqlUpdateParts(on = 'id') {
       const clauseArray = this.constructor.sqlColumns
         .filter(
-          (sqlColumn, index) => this[this.constructor.columns[index]] != null
+          (sqlColumn, index) => this[this.constructor.columns[index]] !== void 0
         )
         .map((sqlColumn, index) => `"${sqlColumn}" = $${index + 1}`);
       const clause = clauseArray.join(', ');
       const idVar = `$${clauseArray.length + 1}`;
       const _values = this.constructor.columns
         .map(column => this[column])
-        .filter(value => value != null);
+        .filter(value => value !== void 0);
       const values = [..._values, this[on]];
       return { clause, idVar, values };
     }

package/src/bo/base-bo.spec.js

@@ -17,6 +17,11 @@ const seven = require('../../test-utils/seven/results.json');
 const eight = require('../../test-utils/eight/results.json');
 const nine = require('../../test-utils/nine/results.json');
 const ten = require('../../test-utils/ten/results.json');
+const eleven = require('../../test-utils/eleven/results.json');
+const twelve = require('../../test-utils/twelve/results.json');
+const Prompt = require('../../test-utils/twelve/bo/prompt');
+const thirteen = require('../../test-utils/thirteen/results.json');
+const Member = require('../../test-utils/thirteen/bo/member');
 
 test('Bo#parseFromDatabase where multiple rows reduce to one nested object (with all one-to-one or one-to-many tables)', () => {
   const order = Order.createOneFromDatabase(one);
@@ -392,7 +397,7 @@ test('Bo#parseFromDatabase with just top level nodes', () => {
 // Problem is when oldest parent is an empty join record and is not included
 // which results in the oldest parent search returning -1 and parent heirarchy
 // is thus messed up.
-test('Bo#parseFromDatabase ', () => {
+test('Bo#parseFromDatabase 10', () => {
   let orders;
   try {
     // This failed when the bug was present
@@ -511,3 +516,70 @@ test('Bo#parseFromDatabase ', () => {
   expect(orders.models[4].lineItems.models[0].productVariant.productVariantImages.models.length).toEqual(1);
   expect(orders.models[4].lineItems.models[0].productVariant.productVariantImages.models[0].id).toEqual(829);
 });
+
+// Issue occcurs in nestClump
+// Problem from early returning not logging bo so parent hierarcy was missing it
+test('Bo#parseFromDatabase 11', () => {
+  let orders;
+  try {
+    // This failed when the bug was present
+    orders = OrderFull.createFromDatabase(eleven);
+  } catch (e) {
+    expect(e).not.toBeDefined();
+  }
+  expect(orders).toBeDefined();
+  // Lots of other assertions that are unrelated and shouldn't be here except
+  // I'm insecure about the lack of tests so just going at it cause I can.
+  // TODO add more later
+});
+
+// Issue occcurs in nestClump
+// Problem when a table references another model twice (two columns)
+test('Bo#parseFromDatabase 12', () => {
+  let prompt;
+  try {
+    // This failed when the bug was present
+    prompt = Prompt.createFromDatabase(twelve);
+  } catch (e) {
+    expect(e).not.toBeDefined();
+  }
+  expect(prompt).toBeDefined();
+  // Ideally the below should work
+  // expect(prompt.fromMember.id).toEqual(1);
+});
+
+// Issue occcurs in nestClump
+// Problem when a table has records that are supposed to nest under root
+// but nest under other stuff below it instead
+// Member
+//  Recommendations [1]
+//   Brand
+//    Recommendations [1]
+//   Passion
+//    Recommendations [2]
+// -- instead of correct --
+// Member
+//  Recommendations[4]
+//   Brand
+//   Passion
+test('Bo#parseFromDatabase 13', () => {
+  const members = Member.createFromDatabase(thirteen);
+  const member = members.models[0];
+  expect(member.recommendations.models.length).toEqual(4);
+  expect(member.recommendations.models[0].brand.id).toEqual(2);
+  expect(member.recommendations.models[0].product.id).toEqual(7);
+  expect(member.recommendations.models[0].category.id).toEqual(1);
+  expect(member.recommendations.models[0].recommendationAudiences.models[0].audience.id).toEqual(1);
+  expect(member.recommendations.models[1].brand.id).toEqual(2);
+  expect(member.recommendations.models[1].product.id).toEqual(1);
+  expect(member.recommendations.models[1].category.id).toEqual(2);
+  expect(member.recommendations.models[1].recommendationAudiences.models[0].audience.id).toEqual(1);
+  expect(member.recommendations.models[2].brand.id).toEqual(3);
+  expect(member.recommendations.models[2].product.id).toEqual(27);
+  expect(member.recommendations.models[2].category.id).toEqual(3);
+  expect(member.recommendations.models[2].recommendationAudiences.models[0].audience.id).toEqual(1);
+  expect(member.recommendations.models[3].brand.id).toEqual(6);
+  expect(member.recommendations.models[3].product.id).toEqual(75);
+  expect(member.recommendations.models[3].category.id).toEqual(4);
+  expect(member.recommendations.models[3].recommendationAudiences.models[0].audience.id).toEqual(1);
+});