search

101companies / 101worker

4 stars 11 forks source link

Update targets101dir env var as well to fix path issue #223

Closed alexsr closed 8 years ago

alexsr commented 8 years ago

targets101dir is used in the env.py functions and by not updating this var the path.join operation does not work as intended because a different path is created.

kevin-klein commented 8 years ago

I have to check it out myself on a windows machine but why would you set targets101dir = repo101dir? You would loose all fragments downloaded from the production worker.

alexsr commented 8 years ago

Oh, now I get it. I'm sorry. I'll close this again. Sorry for the inconvenience. I just found it really confusing that the test module should create data inside the actual data dump folders. I actually thought it would create this data in the 101test folder itself, which is obviously not the intended behaviour. Still somewhat confusing to me in the sense that you override data in the 101web if you simply want to test your modules.