search

funginstitute / patentprocessor

BSD 2-Clause "Simplified" License
68 stars 31 forks source link

Consolidate script looking for gyear that's not passed in by integration tests? #71

Closed billyeh closed 10 years ago

billyeh commented 10 years ago

The integration test for consolidate.py doesn't work because consolidate.py is looking for 

gyear = sys.argv[1]

which isn't passed in from the bash script, consolidate_integration.sh. Am I just missing something here?

Otherwise, I can either add the arguments where consolidate is used, or take it out, since it's just used for filtering.

(The same problem also seems to occur in start.py just calling consolidate without passing in the year.)

gtfierro commented 10 years ago

This will be fixed in a pull request from my fork

gtfierro commented 10 years ago

Fixed as of #72