#5186 Tests are run against pagure global repo and not local branch/copy
Opened 2 years ago by abitrolly. Modified 5 months ago

./dev/run-tests-container.py always runs against main https://pagure.io/pagure.git repo. So no forks or local modifications are tested with its run as recommended in README.md.

It does if you specify the --repo and --branch argument. This is documented in the --help but not in the README indeed.

Metadata Update from @pingou:
- Issue tagged with: doc, easyfix

2 years ago

I would say the deafult behavior should test local modifications.

Metadata Update from @wombelix:
- Issue assigned to wombelix

5 months ago

In addition to the fact that the doc could be improved, even with --repo and --branch specified, I have to push the code first to a public repo. If I already have a PR open that means it I will trigger a CI run with every push, even if I just wanted to test something quickly on my own.
I think it would safe a lot resources and time to have a way to test changes fully locally. The way it's implemented right now, I could achieve it with tox but not when I want to user containers. I'm working on some adjustments right now to keep the existing features and behavior but also make testing of local code modifications possible in an easy way.

Metadata Update from @wombelix:
- Issue set to the milestone: 5.13

5 months ago

Login to comment on this ticket.