add coverage for subprocesses

[muryanto1]

adding coverage for subprocesses.
This PR depends on CDAT/TestsRunner#12

[doutriaux1]

There's more:

computeStdDailyMeansWrapped.py
fourierDiurnalAllGridWrapped.py
compositeDiurnalStatisticsWrapped.py
std_of_hourlyvaluesWrappedInOut.py
std_of_meandiurnalcycWrappedInOut.py
savg_fourierWrappedInOut.py
fourierDiurnalGridpoints.py
std_of_dailymeansWrappedInOut.py
mpindex_compute.py
driver_monsoon_sperber.py
check_available_models.py

[doutriaux1]

@muryanto1 in setup.py we have:
cripts = ['src/python/pcmdi/scripts/mean_climate_driver.py',
'src/python/pcmdi/scripts/pcmdi_compute_climatologies.py',
'src/python/misc/scripts/parallelize_driver.py',
'src/python/monsoon_wang/scripts/mpindex_compute.py',
'src/python/monsoon_sperber/scripts/driver_monsoon_sperber.py',
]
scripts += glob.glob("src/python/diurnal/scripts/*.py")

Maybe we can use this to write the coverage.json file?

[doutriaux1]

That way when we add scripts to test suite they would be automatically picked up?

[coveralls]

Pull Request Test Coverage Report for Build 580

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+9.6%) to 44.898%

---

Totals
Change from base Build 560:	9.6%
Covered Lines:	1805
Relevant Lines:	3691

---

💛 - Coveralls

[muryanto1]

I added the binaries to tests/coverage.json.
Coverage went up from 35% to 44.898%.

[doutriaux1]

Nice! Thanks @muryanto1

[doutriaux1]

@muryanto1 is it expected to fail?

[muryanto1]

Charles, no, it is not expected to fail.
Tests passed before merged to master.
macos_pmp_py3: https://circleci.com/gh/PCMDI/pcmdi_metrics/581
linux_pmp_py3: https://circleci.com/gh/PCMDI/pcmdi_metrics/580
macos_pmp_py2: https://circleci.com/gh/PCMDI/pcmdi_metrics/579
linux_pmp_py2: https://circleci.com/gh/PCMDI/pcmdi_metrics/578

failed when merged to 'master' -- looks like flake8 related
https://circleci.com/gh/PCMDI/pcmdi_metrics/583

[doutriaux1]

@muryanto1 I'll take a look, it doesnt seem to be in master, the PR is still opened