tools/check_uuid.py works incorect for tests with decorators
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
tempest |
Fix Released
|
Undecided
|
Marc Koderer |
Bug Description
There is a patch
https:/
It contains new tests as well as DDT lib enabling. In case of DDT test uses decorators. In that case tools/check_uuid.py does not check test ids correctly. I see following
tox -v -epep8
using tox.ini: /home/ymorkovn/
using tox-1.9.0 from /usr/local/
pep8 reusing: /home/ymorkovn/
/home/
pep8 develop-inst-noop: /home/ymorkovn/
pep8 runtests: PYTHONHASHSEED=
pep8 runtests: commands[0] | flake8
/home/
pep8 runtests: commands[1] | /home/ymorkovn/
/home/
pep8 runtests: commands[2] | python tools/check_uuid.py
/home/
2015-03-12 10:18:23.199 31877 CRITICAL tempest [-] AttributeError: 'Name' object has no attribute 'attr'
2015-03-12 10:18:23.199 31877 TRACE tempest Traceback (most recent call last):
2015-03-12 10:18:23.199 31877 TRACE tempest File "tools/
2015-03-12 10:18:23.199 31877 TRACE tempest run()
2015-03-12 10:18:23.199 31877 TRACE tempest File "tools/
2015-03-12 10:18:23.199 31877 TRACE tempest untagged = checker.
2015-03-12 10:18:23.199 31877 TRACE tempest File "tools/
2015-03-12 10:18:23.199 31877 TRACE tempest return self._filter_
2015-03-12 10:18:23.199 31877 TRACE tempest File "tools/
2015-03-12 10:18:23.199 31877 TRACE tempest if function(
2015-03-12 10:18:23.199 31877 TRACE tempest File "tools/
2015-03-12 10:18:23.199 31877 TRACE tempest tests[module_
2015-03-12 10:18:23.199 31877 TRACE tempest File "tools/
2015-03-12 10:18:23.199 31877 TRACE tempest decorator.func.attr == DECORATOR_NAME and
2015-03-12 10:18:23.199 31877 TRACE tempest AttributeError: 'Name' object has no attribute 'attr'
2015-03-12 10:18:23.199 31877 TRACE tempest
ERROR: InvocationError: '/home/
Add tags Tag help
See full activity log
Add comment
Changed in tempest: | |
assignee: | nobody → Chris Hoge (hoge) |
Changed in tempest: | |
assignee: | Chris Hoge (hoge) → Marc Koderer (m-koderer) |
status: | New → In Progress |
@Chris: Ups, I think I already fixed the issue without seeing this bug. /review. openstack. org/#/c/ 164647/).
Could you have a look? Sry for "stealing" it...
I will link the patch to this bug then (it's https:/