feat(imaging): create basic imaging module

[AlexTan331]

Fixes #2242

Changes proposed in this pull request:

• add permissions to create new imaging and view all imagings
• add ability to request a new imaging
• add ability to view all requested imagings
• add imaging tabs to sidebar for "New Imaging Request" and "Imaging Requests"
• add imaging tab to quick add button in the navbar for "New Imaging Request"
• add imaging unit tests

No new dependency added

Side note:

• no ability to view a single imaging request/edit imaging request/update imaging request in this issue

[gitpod-io]

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hospitalrun/hospitalrun-frontend/82kmapk97
✅ Preview: https://hospitalrun-frontend-git-fork-alextan331-imaging.hospitalrun.vercel.app

[jackcmeyer]

Let's just go with /imaging for the route name

[jackcmeyer]

is this useEffect needed? it seems like it might not be since we initialize the state to be all

[AlexTan331]

This method is to display all the imaging with any status (requested, canceled, and completed). And this method can be used for searching imaging with specific status in the future.

[jackcmeyer]

by the comment, there appears like there may be some work left? if not, remove comment.

[jackcmeyer]

lately, we have been putting this kind of setup in a setup function. I think we should continue this pattern since it makes it easy to make sure each test is not dependent on a previous test.

Each describe can share the setup function. i.e. one setup function for the whole test suite.

[jackcmeyer]

we can use a setup function here as well.

It would like something like:

const setup = () => {
	const store = mockStore({ title: '', imaging: { status: 'loading', error: {} } } as any)
      history.push('/imagings/new')

      wrapper = mount(
        <Provider store={store}>
          <Router history={history}>
            <NewImagingRequest />
          </Router>
        </Provider>,
      )

	return { wrapper: wrapper as ReactWrapper, history, store }
}

This will eliminate the need for global variables in this file.

[jackcmeyer]

@AlexTan331 this is awesome work! let me know if you have any questions about getting the tests passing or any of the code review feedback.