chore(tests): Move code in global test scope to inside beforeAll

[sandrina-p]

chore(tests): Move code in global test scope to inside beforeAll

Problem:

When we need to run only a specific test (it.only()), we were seeing logs of createHeadlessForm() outside that test unit. That can cause a lot of noise if we want to debug only a specific unit test, but also seeing logs from other instances.

Reason

That happens because we had 1 instance of createHeadlessForm() being called outside jest lifecycle.

Solution

Moved that 1 instance in the global scope (inside describe()) to an isolated scope, using beforeAll()

How to test.

A simple way can be to add console.log() to inside the createHeadlessForm()

export function createHeadlessForm(jsonSchema, customConfig = {}) {
+  console.log('Hello!')

And then run only one test at your choise. Eg it.only('returns empty result given no schema'

You'll see the logs saying "Hello" once. But if you go to main master and do the same thing, you'll see the logs twice (Because of the other global instance)

[ollyd]

Tested it, looks good 👍