astro-cli icon indicating copy to clipboard operation
astro-cli copied to clipboard

astro dev pytest should be able to generate a codecov output

Open frankcash opened this issue 2 years ago • 4 comments

  • [x] I have checked that a similar feature request does not already exist.

✍️ Is your feature request related to a problem? Please describe.

No, just a feature gap. Currently you cannot use astro dev pytest to generate a coverage report

🧩 Describe the solution you'd like

Ability to use https://pytest-cov.readthedocs.io/en/latest/reporting.html if given flags were passed in to astro dev pytest cov-...

🤔 Describe alternatives you've considered

Is your feature request specific to a particular Astronomer Platform?

  • [ ] Astro
  • [ ] Software
  • [x] None/Unknown

💡 Additional context

frankcash avatar Apr 07 '23 18:04 frankcash

I don't believe this is solved as there is no way to eject the coverage report out of the running container.

To reproduce: astro dev pytest --args "--cov --cov-report xml"

stuart23 avatar Jun 28 '23 17:06 stuart23

You can if you write it to the include directory On Wed, Jun 28, 2023 at 1:44 PM Stuart Buckingham @.***> wrote:

Reopened #1142 https://github.com/astronomer/astro-cli/issues/1142.

— Reply to this email directly, view it on GitHub https://github.com/astronomer/astro-cli/issues/1142#event-9667601748, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABN32R5D4G44TWKTGAHI43LXNRUORANCNFSM6AAAAAAWW2UJ3E . You are receiving this because you authored the thread.Message ID: @.***>

-- Charles Frank Cash OSS Programmer https://github.com/frankcash https://keybase.io/frankcash https://github.com/frankcash

frankcash avatar Jun 28 '23 17:06 frankcash

astro dev pytest --args "--junitxml=/usr/local/airflow/include/report.xml"

frankcash avatar Jun 28 '23 17:06 frankcash

Great, I will put this in the docs. Would it make sense to mount a different directory for artifacts rather than use include?

stuart23 avatar Jun 28 '23 17:06 stuart23