Remove unnecessary use of the core Kolibri store in tests

[rtibbles]

When custom theming was first introduced to Kolibri, the theming state was made reactive by adding it to the core Vuex state.

This necessitated many component oriented frontend tests to inject the store into their tests in order to have the Vuex state available and prevent errors.

Since this happened, the reactive theming state has been extracted from Vuex and now lives independently. Therefore, injecting the store into these component tests is no longer needed.

Acceptance Criteria

Beta Give feedback

1. Find all instances of kolibri.coreVue.vuex.store imports in Javascript test files
2. In cases where the store is just injected as is into the component, and nothing in the component relies on any vuex state, remove the import and the injection, and ensure the test still passes.

For a starting point, many of the affected spec files were originally edited in this commit: 69856a1 but there may be others, or those files may have moved since!

[vaibhav-rm]

Can I work on this?

[LianaHarris360]

Hi @vaibhav-rm yes, I will assign you to this issue. Please reach out if you have any questions!

[adityagarg06]

Hi @vaibhav-rm I have almost solved this issue,if you don't mind can i raise a PR.

[vaibhav-rm]

Hi @vaibhav-rm I have almost solved this issue,if you don't mind can i raise a PR.

Sure

[MisRob]

@adityagarg06 and @vaibhav-rm Thanks both. @adityagarg06 kindly request you ask for an assignment of an issue and wait to be assigned next time, especially when someone has been assigned previously. Please have a look at the contributing guidelines.

[nick2432]

can i work on this?

[MisRob]

It seems no one is working on it now, so I think it's good to go @nick2432. Thank you.

[nick2432]

Hey @rtibbles , I'm raising a PR. Am I on the correct path?

[rtibbles]

I will take a look, thanks!