A few suggested changes to the frontend CI added in #151... (Note that this PR targets rg/add-frontend-ci instead of main)
1f05a27c8ea3e725dec76a0670749455c5c515d7 removes unnecessary React imports from components and tests. Having React in scope when using JSX is not required with our Babel configuration.
c7915ce3907c41c344eb396117971f13a05270ba uses React Testing Library instead of React Test Renderer for snapshot tests. This lets us avoid adding a new dependency. To support this, had to add a mock for SignInButton to avoid an error in its useEffect hook.
051e9ed4f5360ceb04f7ae242ea4ff0b85183459 restructures tests to have a describe block for the component under test and an it block for the snapshot test. This lets us group related tests if/when we add additional tests for components.
5cfbd09ab5cd8861f637553c748d9f3ff5baefd1 reverts a change to spacing in one component. This change seemed unrelated to frontend CI.
A few suggested changes to the frontend CI added in #151... (Note that this PR targets
rg/add-frontend-ci
instead ofmain
)useEffect
hook.describe
block for the component under test and anit
block for the snapshot test. This lets us group related tests if/when we add additional tests for components.