fix error when AdvertisingProvider is undefined (#13)

KijijiCA · Jun 18, 2019 · e01f62d · e01f62d
1 parent 618c940
commit e01f62d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/components/AdvertisingProvider.js b/src/components/AdvertisingProvider.js
@@ -50,7 +50,9 @@ export default class AdvertisingProvider extends Component {
     }
 
     componentWillUnmount() {
-        this.teardown();
+        if (this.props.config) {
+            this.teardown();
+        }
     }
 
     async teardown() {

diff --git a/src/components/AdvertisingProvider.test.js b/src/components/AdvertisingProvider.test.js
@@ -168,6 +168,12 @@ describe('The AdvertisingProvider component', () => {
             }, 0);
         });
 
+        it('does not tear down the Advertising module if the config is undefined', () => {
+            provider.unmount();
+
+            mockTeardown.should.not.have.been.calledOnce;
+        });
+
         afterEach(resetMocks);
     });