itemadapter icon indicating copy to clipboard operation
itemadapter copied to clipboard

provide a test environment with missing dependencies

Open kmike opened this issue 4 years ago • 2 comments

This should allow to drop some of the mock usages, and test it for real that the package works. I think a single environment without all the dependencies could work; it would only support dict items.

kmike avatar May 20 '20 20:05 kmike

Sounds fine, although I don't think it should be a blocker for https://github.com/scrapy/scrapy/pull/3881.

One thing to consider is the coverage report. Currently those mocks allow to reach every line in a single environment. With this approach I'll need to do a few adjustments to combine reports from different tox envs. I'm pasting this here so I don't forget: https://help.github.com/en/actions/configuring-and-managing-workflows/persisting-workflow-data-using-artifacts

elacuesta avatar May 21 '20 00:05 elacuesta

@elacuesta I think if you use codecov they combine reports automatically, on their side.

kmike avatar May 21 '20 08:05 kmike