From f5a45ebc2d0a5baeee680374785756fc83f3b64f Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Thu, 11 Apr 2019 15:32:33 -0700
Subject: [PATCH 1/2] Ensure wildcard arguments get undefined treatment. Fixes
 #5428.

---
 lib/legacy/legacy-data-mixin.js |  3 ++-
 test/unit/legacy-data.html      | 48 +++++++++++++++++++++++++++++++--
 2 files changed, 48 insertions(+), 3 deletions(-)

diff --git a/lib/legacy/legacy-data-mixin.js b/lib/legacy/legacy-data-mixin.js
index b2a255386d..d3d84088e2 100644
--- a/lib/legacy/legacy-data-mixin.js
+++ b/lib/legacy/legacy-data-mixin.js
@@ -94,7 +94,8 @@ export const LegacyDataMixin = dedupingMixin(superClass => {
       // undefined or not. Multi-property observers must have all arguments defined
       if (this._legacyUndefinedCheck && vals.length > 1) {
         for (let i=0; i<vals.length; i++) {
-          if (vals[i] === undefined) {
+          if (vals[i] === undefined || 
+              (args[i].wildcard && vals[i].base === undefined)) {
             // Break out of effect's control flow; will be caught in
             // wrapped property effect function below
             const name = args[i].name;
diff --git a/test/unit/legacy-data.html b/test/unit/legacy-data.html
index 353efd1269..b77a76510b 100644
--- a/test/unit/legacy-data.html
+++ b/test/unit/legacy-data.html
@@ -51,13 +51,16 @@
           },
           computedMulti: {
             computed: 'computeMulti(computedMultiDep1, computedMultiDep2)'
-          }
+          },
+          wildcardProp: String,
+          wildcardObj: Object
         },
         observers: [
           'staticObserver("staticObserver")',
           'singlePropObserver(singleProp)',
           'multiPropObserver(multiProp1, multiProp2)',
-          'throws(throwProp)'
+          'throws(throwProp)',
+          'wildcardObserver(wildcardProp, wildcardObj.*)'
         ],
         created() {
           this.singlePropObserver = sinon.spy();
@@ -65,6 +68,7 @@
           this.staticObserver = sinon.spy();
           this.computeSingle = sinon.spy((inlineSingleDep) => `[${inlineSingleDep}]`);
           this.computeMulti = sinon.spy((inlineMultiDep1, inlineMultiDep2) => `[${inlineMultiDep1},${inlineMultiDep2}]`);
+          this.wildcardObserver = sinon.spy();
         },
         throws() {
           throw new Error('real error');
@@ -145,6 +149,18 @@
     </template>
   </test-fixture>
 
+  <test-fixture id="declarative-wildcard-one">
+    <template>
+      <x-data wildcard-prop='prop'></x-data>
+    </template>
+  </test-fixture>
+
+  <test-fixture id="declarative-wildcard-all">
+    <template>
+      <x-data wildcard-prop="prop" wildcard-obj='{"prop": "wildcardObj"}'></x-data>
+    </template>
+  </test-fixture>
+
   <script type="module">
     import {flush} from '../../lib/utils/flush.js';
 
@@ -192,6 +208,8 @@
       const inlineMultiDep2 = 'inlineMultiDep2';
       const inlineMultiIfDep1 = 'inlineMultiIfDep1';
       const inlineMultiIfDep2 = 'inlineMultiIfDep2';
+      const wildcardProp = 'wildcardProp';
+      const wildcardObj = {prop: 'wildcardObj'};
 
       suite('check disabled', () => {
         test('no arguments defined', () => {
@@ -270,6 +288,14 @@
           assertEffects({computeMulti: 1});
           assert.equal(el.$$('#ifChild').computedMulti, '[inlineMultiIfDep1,inlineMultiIfDep2]');
         });
+        test('one wildcard argument defined', () => {
+          setupElement(false, {wildcardProp});
+          assertEffects({wildcardObserver: 1});
+        });
+        test('all wildcard arguments defined', () => {
+          setupElement(false, {wildcardProp, wildcardObj});
+          assertEffects({wildcardObserver: 1});
+        });
       });
 
       suite('warn', () => {
@@ -349,6 +375,14 @@
           assertEffects({computeMulti: 1});
           assert.equal(el.$$('#ifChild').computedMulti, '[inlineMultiIfDep1,inlineMultiIfDep2]');
         });
+        test('one wildcard argument defined', () => {
+          setupElement(true, {wildcardProp});
+          assertEffects({warn: 1});
+        });
+        test('all wildcard arguments defined', () => {
+          setupElement(true, {wildcardProp, wildcardObj});
+          assertEffects({wildcardObserver: 1});
+        });
       });
     });
 
@@ -416,6 +450,16 @@
           assertEffects({computeMulti: 1});
           assert.equal(el.$$('#ifChild').computedMulti, '[b,c]');
         });
+        test.only('one wildcard argument defined', () => {
+          el = fixture('declarative-wildcard-one');
+          flush();
+          assertEffects({warn: 1});
+        });
+        test.only('all wildcard arguments defined', () => {
+          el = fixture('declarative-wildcard-all');
+          flush();
+          assertEffects({wildcardObserver: 1});
+        });
       });
     });
 

From ca124480a48d18fdf54159d95666d6a5a3dd1328 Mon Sep 17 00:00:00 2001
From: Kevin Schaaf <kschaaf@google.com>
Date: Thu, 11 Apr 2019 17:25:40 -0700
Subject: [PATCH 2/2] Remove test.only

---
 test/unit/legacy-data.html | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/unit/legacy-data.html b/test/unit/legacy-data.html
index b77a76510b..1cd5a06189 100644
--- a/test/unit/legacy-data.html
+++ b/test/unit/legacy-data.html
@@ -450,12 +450,12 @@
           assertEffects({computeMulti: 1});
           assert.equal(el.$$('#ifChild').computedMulti, '[b,c]');
         });
-        test.only('one wildcard argument defined', () => {
+        test('one wildcard argument defined', () => {
           el = fixture('declarative-wildcard-one');
           flush();
           assertEffects({warn: 1});
         });
-        test.only('all wildcard arguments defined', () => {
+        test('all wildcard arguments defined', () => {
           el = fixture('declarative-wildcard-all');
           flush();
           assertEffects({wildcardObserver: 1});