listDependenciesFromPom failed to include test-scoped plugin dependencies

[jglick]

…as promised in the documentation:

If the pom option is set, all dependencies will be added, including test ones.

I think this regressed in #35.

Also its technique of skipping dependencies with classifiers did not account for, say, core-assets.

[oleg-nenashev]

This is a case when we might need to adjust documentation to the reality. Needs input from @raul-arabaolaza

Also its technique of skipping dependencies with classifiers did not account for, say, core-assets.

Please create a ticket for that

[jglick]

OK, I might just skip CWP and rebuild a WAR manually…might actually be easier than getting this patch in.

[jglick]

Seems to be a build failure in test_war_from_pom_with_timeout; have not yet looked at what that means.