fix(invoices): fix broken filter test

This commit fixes a broken filter test that was not actually clicking an
element.  It also optimizes the patient search tests slightly.

client/test/e2e/patient/invoice/registry.search.js

@@ -98,10 +98,10 @@ function InvoiceRegistrySearch() {
 
   it('filters by radio buttons', () => {
     FU.buttons.search();
-    element(by.id('no'));
+    element(by.id('no')).click();
     FU.buttons.submit();
 
-    expectNumberOfGridRows(4);
+    expectNumberOfGridRows(0);
     expectNumberOfFilters(1);
 
     // make sure to clear the filters for the next test

client/test/e2e/patient/registry.search.js

@@ -27,12 +27,12 @@ function PatientRegistrySearch() {
   };
 
   const defaultVisibleRowNumber = 3;
+  const grid = element(by.id('patient-registry'));
+  const rows = grid.element(by.css('.ui-grid-render-container-body'))
+    .all(by.repeater('(rowRenderIndex, row) in rowContainer.renderedRows track by $index'));
 
-  function expectNumberOfGridRows(number) {
-    const grid = element(by.id('patient-registry'));
-    const rows = grid.element(by.css('.ui-grid-render-container-body'))
-      .all(by.repeater('(rowRenderIndex, row) in rowContainer.renderedRows track by $index'));
 
+  function expectNumberOfGridRows(number) {
     expect(rows.count(),
       `Expected Patient Registry ui-grid's row count to be ${number}.`
     ).to.eventually.equal(number);