iTowns / itowns / 1290

65%

master: 88%

Pull #1321

travis-ci-com

chore(test): move variables declaration in describe()

In some unit tests, variables were declared before the first
`describe()` section of the file. When calling `npm run test-unit`, all
files are loaded and executed at the same time, with all `describe`
content being moved to a queue. So this causes a problem with things
declared before `describe`: a lot of variables (noticeably `context`)
were being rewritten and shared between all tests. While all tests were
passing, some of them would be false positive.
Moving all those declarations in `describe` fixes this.

2206 of 3855 branches covered (57.22%)

Branch coverage included in aggregate %.

4499 of 6449 relevant lines covered (69.76%)

181.64 hits per line