[Bug 1979724] Review Request: python-applicationinsights - Python support for Azure Application Insights API

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1979724



--- Comment #3 from Ben Beasley <code@xxxxxxxxxxxxxxxxxx> ---
> As for the tests, the azure tests require two bits of helper code from the git repository itself that are not installable on their own and they are not published to pypi. In addition, the majority of the tests need network access to download certain things and talk to Azure's API. That's why I haven't found a way to run tests here yet.

That makes sense, and if the tests prove to be un-runnable, I think that ought
to go in a spec file comment where the %check section would be.

> If I try your testing method, I get:
> 
>   ImportError: Start directory is not importable: './tests/'
> 
> pytest *does* work, but then I run into the network access problems and the missing helpers from the upstream repository. The whole upstream repository is about ~ 150-160MB and each of these little components are released independently from that repository.

This is strange. Here’s a koji build that worked for me
(https://koji.fedoraproject.org/koji/taskinfo?taskID=71966298). Maybe that will
help?


-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam on the list, report it: https://pagure.io/fedora-infrastructure




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux