Merge branch 'legacy-undefined-noBatch' into legacy-undefined-noBatch…

…-altTemplates
Polymer · Jul 11, 2019 · 195323c · 195323c
2 parents 5311917 + d32d300
commit 195323c
Show file tree

Hide file tree

Showing 3 changed files with 23 additions and 11 deletions.
diff --git a/lib/mixins/property-effects.js b/lib/mixins/property-effects.js
@@ -539,7 +539,7 @@ function getComputedOrder(inst) {
   if (!ordered) {
     ordered = new Map();
     const effects = inst[TYPES.COMPUTE];
-    const {counts, ready} = dependencyCounts(inst);
+    let {counts, ready, total} = dependencyCounts(inst);
     let curr;
     while ((curr = ready.shift())) {
       ordered.set(curr, ordered.size);
@@ -548,12 +548,17 @@ function getComputedOrder(inst) {
         computedByCurr.forEach(fx => {
           // Note `methodInfo` is where the computed property name is stored
           const computedProp = fx.info.methodInfo;
+          --total;
           if (--counts[computedProp] === 0) {
             ready.push(computedProp);
           }
         });
       }
     }
+    if (total !== 0) {
+      const el = /** @type {HTMLElement} */ (inst);
+      console.warn(`Computed graph for ${el.localName} incomplete; circular?`);
+    }
     inst.constructor.__orderedComputedDeps = ordered;
   }
   return ordered;
@@ -563,7 +568,8 @@ function getComputedOrder(inst) {
  * Generates a map of property-to-dependency count (`counts`, where "dependency
  * count" is the number of dependencies a given property has assuming it is a
  * computed property, otherwise 0).  It also returns a pre-populated list of
- * `ready` properties that have no dependencies.
+ * `ready` properties that have no dependencies and a `total` count, which is
+ * used for error-checking the graph.
  *
  * Used by `orderedComputed: true` computed property algorithm.
  *
@@ -574,20 +580,25 @@ function getComputedOrder(inst) {
  *   dependencies.
  */
 function dependencyCounts(inst) {
-  const props = inst.constructor._properties;
   const infoForComputed = inst[COMPUTE_INFO];
   const counts = {};
+  const computedDeps = inst[TYPES.COMPUTE];
   const ready = [];
-  for (let p in props) {
+  let total = 0;
+  // Count dependencies for each computed property
+  for (let p in infoForComputed) {
     const info = infoForComputed[p];
-    if (info) {
-      // Be sure to add the method name itself in case of "dynamic functions"
-      counts[p] = info.args.length + (info.dynamicFn ? 1 : 0);
-    } else {
+    // Be sure to add the method name itself in case of "dynamic functions"
+    total += counts[p] =
+      info.args.filter(a => !a.literal).length + (info.dynamicFn ? 1 : 0);
+  }
+  // Build list of ready properties (that aren't themselves computed)
+  for (let p in computedDeps) {
+    if (!infoForComputed[p]) {
       ready.push(p);
     }
   }
-  return {counts, ready};
+  return {counts, ready, total};
 }
 
 /**

diff --git a/test/unit/property-effects-elements.js b/test/unit/property-effects-elements.js
@@ -1015,7 +1015,7 @@ customElements.define('x-computed-ordering', class extends PolymerElement {
   static get properties() {
     return {
       a: {type: Number, value: 1000},
-      b: {type: Number, value: 100},
+      // b: {type: Number, value: 100}, // Intentionally undeclared; init in ctor
       c: {type: Number, value: 10},
       d: {type: Number, value: 1},
       abbcd: {computed: 'computeABBCD(a, b, bcd)', observer: 'abbcdChanged'},
@@ -1034,6 +1034,7 @@ customElements.define('x-computed-ordering', class extends PolymerElement {
   }
   constructor() {
     super();
+    this.b = 100;
     sinon.spy(this, 'computeABBCD');
     sinon.spy(this, 'computeBCD');
     sinon.spy(this, 'computeBC');

diff --git a/test/unit/property-effects.html b/test/unit/property-effects.html
@@ -2091,7 +2091,7 @@
 
 if (orderedComputed) {
 
-  suite.only('computed property ordering', function() {
+  suite('computed property ordering', function() {
     var el;
 
     setup(function() {