In AppVeyor.psm1 there should be an option to change how we detect the files that need code coverage

[TravisEz13]

The task to do this in #88 was added because I was going to write the code to do the detection of the files that needed code coverage. Since this was community provided. Let's assume it's sufficient. I'm filing thing just to keep the idea in mind and open it to a discussion.

[johlju]

This does not affect resource modules, like SharePointDsc, that uses the Harness-model. Those resource modules control the code coverage them self by having all testing logic in the harness function in the individual repo. Below is improving functionality for the default shared model.

Currently it catches all module files (*.psm1) recursively in the folders added here.

DscResource.Tests/AppVeyor.psm1

Lines 279 to 282 in 7fcd524

	$possibleModulePaths = @(
	'DSCResources',
	'DSCClassResources'
	)

And as a workaround for resource modules that (wrongly) has helper modules in the root folder this is added as well

DscResource.Tests/AppVeyor.psm1

Lines 260 to 262 in 7fcd524

	$codeCoveragePaths = @(
	"$env:APPVEYOR_BUILD_FOLDER\*.psm1"
	)

We should instead add a Modules folder to the list above, should be located in the root as well. In the Modules folder a resource module helper modules can be located (as .psm1). This should be updated in the DscResource.Template as well.

We could add the $possibleModulePath as a parameter instead so it's easier to control the module paths if they change in the future, with the following as default values.

$possibleModulePaths = @( 
     'DSCResources', 
     'DSCClassResources',
     'Modules'
 ) 

[johlju]

@PlagueHO This would help with the code coverage in for example CertificateDsc? To find if there are parts of the modules that are not covered by tests?

[johlju]

Did a test of this with CertificateDsc, and it seems it the helper function are covered. The code coverage percentage dropped, which is positive.
https://ci.appveyor.com/project/johlju/certificatedsc/build/2.3.6.0#L1486