Update node js and fix failing unit tests (#1119)

* fixed unhandled promise rejection

...which fixes locally failing jest test

* fix failing unit test for UserProfile

* udpate node js and npm for CI tests

* make test a bit more specific

* set node js to 16.x

* update node js to 16.x and npm to 8.15 in Dockerfile

* revert npm version change

Dockerfile

@@ -21,7 +21,7 @@ RUN apt-get update && apt-get install -y --no-install-recommends \
     python3 \
     time \
     vim \
-    && curl -sL https://deb.nodesource.com/setup_14.x | bash - \
+    && curl -sL https://deb.nodesource.com/setup_16.x | bash - \
     && apt-get install -y --no-install-recommends nodejs \
     && apt-get clean \
     && rm -rf /var/lib/apt/lists/* \

src/Submission/MapDataModel.jsx

@@ -151,6 +151,9 @@ class MapDataModel extends React.Component {
         if (data && data[this.state.parentNodeType]) {
           this.setState((prevState) => ({ validParentIds: data[prevState.parentNodeType] }));
         }
+      }).catch(error => {
+        // TODO - should we throw this error?
+        console.error(error);
       });
     } else {
       this.setState({ validParentIds: [] });

src/UserProfile/UserProfile.test.jsx

@@ -42,7 +42,8 @@ describe('the UserProfile component', () => {
         />
       </StaticRouter>,
     );
-    expect($vdom.find('tbody tr')).toHaveLength(testProps.userProfile.jtis.length + 1);
+    // we expect one delete button per jti:
+    expect($vdom.find(`button[name="${DELETE_BTN}"]`)).toHaveLength(testProps.userProfile.jtis.length);
   });
 
   it('triggers create-key events', (done) => {
@@ -79,7 +80,7 @@ describe('the UserProfile component', () => {
       </StaticRouter>,
     );
     const $deleteBtn = $vdom.find(`button[name="${DELETE_BTN}"]`);
-    expect($deleteBtn).toHaveLength(2);
+    expect($deleteBtn).toHaveLength(testProps.userProfile.jtis.length);
     $deleteBtn.at(0).simulate('click');
     // should invoke onRequestDeleteKey callback  (above - calls done()) ...
   });