iTowns / itowns / 1298

88%

push

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.

2209 of 3861 branches covered (57.21%)

Branch coverage included in aggregate %.

4518 of 6467 relevant lines covered (69.86%)

287.67 hits per line