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

Odd behavior from script_usage_tests.py #74

Closed
wasade opened this issue Dec 19, 2012 · 3 comments
Closed

Odd behavior from script_usage_tests.py #74

wasade opened this issue Dec 19, 2012 · 3 comments

Comments

@wasade
Copy link
Contributor

wasade commented Dec 19, 2012

See below. To recreate, just call script_usage_tests.py twice without arguments

$ script_usage_tests.py

This script has been deprecated in favor of the more general add_metadata.py script. add_metadata.py is part of the biom-format package, a dependency of QIIME. For more details, call:

add_metadata.py -h

Traceback (most recent call last):
File "./script_usage_tests.py", line 105, in
main()
File "./script_usage_tests.py", line 99, in main
failure_log_fp=failure_log_fp)
File "/Users/mcdonald/ResearchWork/software/qiime/qiime/test.py", line 109, in run_script_usage_tests
usage_examples = script.script_info['script_usage']
AttributeError: 'module' object has no attribute 'script_info'
16:33:04 qiime_test_data@print_metadata_stats$ script_usage_tests.py
Traceback (most recent call last):
File "./script_usage_tests.py", line 105, in
main()
File "./script_usage_tests.py", line 99, in main
failure_log_fp=failure_log_fp)
File "/Users/mcdonald/ResearchWork/software/qiime/qiime/test.py", line 117, in run_script_usage_tests
copytree(test_input_dir,test_working_dir)
File "/Users/mcdonald/lib/python2.7/shutil.py", line 174, in copytree
os.makedirs(dst)
File "/Users/mcdonald/lib/python2.7/os.py", line 157, in makedirs
mkdir(name, mode)
OSError: [Errno 17] File exists: '/tmp/script_usage_tests/add_qiime_labels'

@gregcaporaso
Copy link
Contributor

I think you have two things going on here. First, you need to update your qiime_test_data repo as the add_taxa directory needs to be removed (since that script is deprecated and thus no longer implements the script_info dictionary. Then, since the run failed, the working directory '/tmp/script_usage_tests exists, so the tests fail. Both of these should result in more informative error messages, so I'll add issues for those specifically and close this one.

@gregcaporaso
Copy link
Contributor

See issues #78 and #77.

@wasade
Copy link
Contributor Author

wasade commented Dec 20, 2012

I had done a fresh pull on qiime_test_data, but can recheck

On Thu, Dec 20, 2012 at 7:21 AM, Greg Caporaso [email protected]:

See issues #78 #78 and
#77 #77.


Reply to this email directly or view it on GitHubhttps://github.com//issues/74#issuecomment-11574990.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants