Remove waitForElement from test case

Aside from being deprecated in later versions(
testing-library/dom-testing-library#416) it's also
not needed here, as we can just return `findByDisplayValue` directly

tests/integration/client/courseInstance/viewselector.test.tsx

@@ -1,5 +1,4 @@
 import React from 'react';
-import { waitForElement } from '@testing-library/react';
 import { render } from 'test-utils';
 import CoursesPage from 'client/components/pages/Courses/CoursesPage';
 
@@ -10,8 +9,6 @@ describe('View selection dropdown', function () {
     coursePage = render(<CoursesPage />);
   });
   it('defaults to the default view', async function () {
-    return waitForElement(
-      () => coursePage.findByDisplayValue(/Default/, { exact: false })
-    );
+    return coursePage.findByDisplayValue(/Default/, { exact: false });
   });
 });