Revert "[ENH] Move codecov to the end of each test #1889"

[oesteban]

CircleCI environment variables were not set in the docker container, and it wasn't working. This PR reverts that situation (and also tries to make the coverage reporting more reliable).

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (master@15f520d). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1895   +/-   ##
=========================================
  Coverage          ?   70.93%           
=========================================
  Files             ?     1057           
  Lines             ?    53341           
  Branches          ?     7718           
=========================================
  Hits              ?    37837           
  Misses            ?    14125           
  Partials          ?     1379

Flag	Coverage Δ
#smoketests	70.93% <ø> (?)
#unittests	70.38% <ø> (?)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 15f520d...037254a. Read the comment docs.

[oesteban]

@chrisfilo, since you were fighting with codecov some time ago, could you please take a look on this one? I think the codecov-io comment above is actually related to the fact that coverage has not been updated in master for a while.

[satra]

@chrisfilo and @oesteban - codecov is again doing something weird - it's down to 46%