Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Errors in test #18

Open
Germano0 opened this issue Jan 26, 2016 · 4 comments
Open

Errors in test #18

Germano0 opened this issue Jan 26, 2016 · 4 comments
Milestone

Comments

@Germano0
Copy link

Hi, I submitted python-libcnml for Review Process in order to push it into Fedora, Fedora EPEL repositories.
On my machine the packaging process works correctly, but on Fedora Koji build system (that does not allow the package to connect to internet) fails. I attach here some error message in case you can help me find the problem. Thank you for your time

https://kojipkgs.fedoraproject.org//work/tasks/512/12690512/build.log

@PabloCastellano
Copy link
Owner

There is one particular test that tests parsing a CNML file from an URL (see https://github.com/PabloCastellano/libcnml/blob/master/libcnml/tests/test_libcnml.py#L148).

I'm not sure about what should we do in this case. What do you think @nemesisdesign ?

@Fale
Copy link

Fale commented Jan 27, 2016

Hi,
Does it make sense to test with an external url? What are the cases in which the local file succeed while the external fails, since this library is about parsing and not networking?

@nemesifier
Copy link
Contributor

The test request can be mocked (simulated). We introduced this feature to facilitate integration with netdiff and django-netjsongraph, so I'd prefer to keep it.

@PabloCastellano
Copy link
Owner

@nemesisdesign Mocking sounds good to me too.

PRs are welcome!

@PabloCastellano PabloCastellano added this to the v0.10.0 milestone Feb 2, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants