feat(ResourceTable): Trigger re-fetch when rel, name or `embedded…

…` props change

src/components/ResourceTable.spec.tsx

@@ -102,7 +102,7 @@ describe('default behaviour', () => {
     });
   });
 
-  describe('when columns prop changes', () => {
+  describe('when `columns` prop changes', () => {
     it('fetches with new params', () => {
       fetch.mockClear();
 
@@ -150,6 +150,61 @@ describe('default behaviour', () => {
       expect(fetch).not.toHaveBeenCalled();
     });
   });
+
+  describe('when `rel` prop changes', () => {
+    it('fetches with new rel', () => {
+      fetch.mockClear();
+
+      root.setProps({
+        rel: 'users'
+      });
+
+      expect(fetch).toHaveBeenCalledWith(({
+        rel: 'users'
+      }), ({
+        where: undefined,
+        order: undefined,
+        page: undefined
+      }));
+    });
+  });
+
+  describe('when `name` prop changes', () => {
+    it('fetches with new rel', () => {
+      fetch.mockClear();
+
+      root.setProps({
+        name: 'item'
+      });
+
+      expect(fetch).toHaveBeenCalledWith(({
+        rel: 'widgets',
+        name: 'item'
+      }), ({
+        where: undefined,
+        order: undefined,
+        page: undefined
+      }));
+    });
+  });
+
+  describe('when `embedded` prop changes', () => {
+    it('fetches again (although not really necessary)', () => {
+      fetch.mockClear();
+
+      root.setProps({
+        embedded: 'something-else'
+      });
+
+      expect(fetch).toHaveBeenCalledWith(({
+        rel: 'widgets'
+      }), ({
+        where: undefined,
+        order: undefined,
+        page: undefined
+      }));
+    });
+  });
 });
 
 describe('when `name` is specified', () => {

src/components/ResourceTable.tsx

@@ -115,11 +115,24 @@ export class ResourceTable extends React.PureComponent<Props, State> {
   }
 
   componentWillReceiveProps(nextProps: Props) {
-    const params = this._nextParams(nextProps);
+    let shouldFetch = (
+      (nextProps.rel !== this.props.rel) ||
+      (nextProps.name !== this.props.name) ||
+      // @todo: Changing 'embedded' should not trigger a re-fetch
+      (nextProps.embedded !== this.props.embedded)
+    );
+
+    let params: any = this._nextParams(nextProps);
 
     if (params) {
       this.setState({ params });
-      this.fetch(params);
+      shouldFetch = true;
+    } else {
+      params = this.state.params;
+    }
+
+    if (shouldFetch) {
+      this.fetch(params, nextProps);
     }
   }
 
@@ -137,9 +150,7 @@ export class ResourceTable extends React.PureComponent<Props, State> {
     }
   }
 
-  fetch = (params = this.state.params) => {
-    const { resource, rel, name, embedded } = this.props;
-
+  fetch = (params = this.state.params, { resource, rel, name, embedded } = this.props) => {
     this.setState({ loading: true });
 
     const link = name ?