search

seomoz / rep-cpp

Robot exclusion protocol in C++
MIT License
12 stars 5 forks source link

More wildcard tests #35

Closed b4hand closed 6 years ago

b4hand commented 6 years ago

These tests were added as part of the initial investigation into #34, but don't directly address the issue reported there. It turns out the bug is only triggered when the wildcard is in the leading position and only occurs in the Agent, not in the Directive.

I have some separate failing tests that directly address the bug mentioned in #34, but I'm waiting to submit a PR until I have a fix.

b4hand commented 6 years ago

It's unclear why the tests are failing in Travis. These pass locally and the build in Travis won't show the logs. I'll investigate more later.

dlecocq commented 6 years ago

It uses a python module gcovr to check coverage, but:

scripts/travis/before_install.sh: line 5: /home/travis/virtualenv/python3.5/bin/activate: No such file or directory

The .travis.yml lists language: cpp, but also lists 3.5 under python. Off hand I'm not sure what the Travis-recommended solution for this situation is.