quattor / CCM

Configuration Cache Manager
www.quattor.org
Other
3 stars 12 forks source link

fetch.t depends on google being up #197

Open jrha opened 5 years ago

jrha commented 5 years ago

See: https://github.com/quattor/CCM/blob/c6c67c251e1ff50a49521c706d4f5cc4664af3ba/src/test/resources/ccm_tabcompletion.cfg#L4 https://github.com/quattor/CCM/blob/c6c67c251e1ff50a49521c706d4f5cc4664af3ba/src/test/resources/ccm.cfg#L7 https://github.com/quattor/CCM/blob/c6c67c251e1ff50a49521c706d4f5cc4664af3ba/src/test/resources/ccm_trust.cfg#L7 https://github.com/quattor/CCM/blob/c6c67c251e1ff50a49521c706d4f5cc4664af3ba/src/test/perl/fetch.t#L171 https://github.com/quattor/CCM/blob/c6c67c251e1ff50a49521c706d4f5cc4664af3ba/src/test/perl/fetch.t#L204

This was previously mentioned in #158.

It seems a little insane for unit tests to rely on external connectivity, but I can't immediately think of a better solution. Nevertheless I am opening this issue to document that we know about it.