Unittest TestBlockCommentsBeginWithASpace not pass

Bug #1844621 reported by Eric Xie
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Identity (keystone)
Invalid
Undecided
Unassigned

Bug Description

==============================
Failed 1 tests - output below:
==============================

keystone.tests.unit.test_hacking_checks.TestBlockCommentsBeginWithASpace.test
-----------------------------------------------------------------------------

Captured traceback:
~~~~~~~~~~~~~~~~~~~
    Traceback (most recent call last):
      File "keystone/tests/unit/test_hacking_checks.py", line 80, in test
        self.assert_has_errors(code, expected_errors=errors)
      File "keystone/tests/unit/test_hacking_checks.py", line 57, in assert_has_errors
        actual_errors = [e[:3] for e in self.run_check(code)]
      File "keystone/tests/unit/test_hacking_checks.py", line 52, in run_check
        checker.check_all()
      File "/opt/jenkins_work/workspace/unittest-keystone/.tox/py27/local/lib/python2.7/site-packages/pep8.py", line 1438, in check_all
        self.check_logical()
      File "/opt/jenkins_work/workspace/unittest-keystone/.tox/py27/local/lib/python2.7/site-packages/pep8.py", line 1328, in check_logical
        (start_row, start_col) = mapping[0][1]
    TypeError: 'NoneType' object has no attribute '__getitem__'

Version:
Rocky 14.1.0

Revision history for this message
Gage Hugo (gagehugo) wrote :

What version of python are you running? I see it's the Rocky release, were there any changes made?

Changed in keystone:
status: New → Incomplete
Revision history for this message
Eric Xie (eric-xie) wrote :

@Gage,i used python 2.7.
# python --version
Python 2.7.15+
# tox --version
3.6.0 imported from /usr/local/lib/python2.7/dist-packages/tox/__init__.pyc

Revision history for this message
Vishakha Agarwal (vishakha.agarwal) wrote :

I am unable to reproduce the issue on Master i.e. Train. Could you please check that in master. Also could you provide some extra details you might have changed in your cloud environment?

Revision history for this message
Eric Xie (eric-xie) wrote :

I use higher version of pep8, and this issue was gone.
So change it to invalid.

Colleen Murphy (krinkle)
Changed in keystone:
status: Incomplete → Invalid
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.