Remove unnecessary dependencies and move others to dev-dependencies

[jmac105]

✏️ Changes

Removing dependencies which are not unused, and moving other packages from dependencies into dev-dependencies such as test assertion libraries.
Resolves #625

🎯 Testing

No tests added or removed, tests are all passing locally.

✅🚫 This change has unit test coverage

✅🚫 This change has integration test coverage

✅🚫 This change has been tested for performance

[willvedd]

Ha! You beat me to it, I already had this exact set of changes ready to push. Anyways, appreciate you addressing this.

[jmac105]

Not sure why e2e tests are failing, as far as I can tell it does not seem to be related to my changes?

[willvedd]

@jmac105 Yeah, I think it has to do with this being a forked PR. I'm looking into it now and will re-run locally but anticipate your changes to be valid. Once I confirm, I'll go ahead and merge.

[willvedd]

In addition to your changes, I removed cross-env dependency as it isn't being used anymore. Was able to confirm that e2e test failures are the result of secrets not being passed through for forked PRs, will need to address that sometime in the near future.