Testrail report script fails if docstring is null

Bug #1636226 reported by Alexander Kurenyshev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Released
Medium
Alexander Kurenyshev

Bug Description

When we have cases with an empty docstring test rail report uploading tool fails with:
Traceback (most recent call last):
  File "fuelweb_test/testrail/upload_cases_description.py", line 441, in <module>
    main()
  File "fuelweb_test/testrail/upload_cases_description.py", line 432, in main
    default_test_priority=testrail_default_test_priority
  File "fuelweb_test/testrail/upload_cases_description.py", line 70, in get_tests_descriptions
    title, steps, duration = _parse_docstring(docstring, case)
  File "fuelweb_test/testrail/upload_cases_description.py", line 266, in _parse_docstring
    title = _parse_title(title_r, case) if title_r else ''
  File "fuelweb_test/testrail/upload_cases_description.py", line 291, in _parse_title
    title = ' '.join(map(string.strip, s.split('\n')))
AttributeError: 'list' object has no attribute 'split'

Like here:
https://product-ci.infra.mirantis.net/view/9.x_acceptance/job/9.x.acceptance.test-reports/629/console

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/mitaka)

Fix proposed to branch: stable/mitaka
Review: https://review.openstack.org/390368

Changed in fuel:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (master)

Fix proposed to branch: master
Review: https://review.openstack.org/390369

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to fuel-qa (stable/newton)

Fix proposed to branch: stable/newton
Review: https://review.openstack.org/390370

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/mitaka)

Reviewed: https://review.openstack.org/390368
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=482998f2006b1a5091f31167545016e8704a2873
Submitter: Jenkins
Branch: stable/mitaka

commit 482998f2006b1a5091f31167545016e8704a2873
Author: Alexander Kurenyshev <email address hidden>
Date: Mon Oct 24 18:12:49 2016 +0300

    Fix testrail upload cases script

    When docstring is empty test rail report tool fails

    Change-Id: Iafee2ba4712ec9e86b9310c703b26bbf2f0b39e6
    Closes-Bug:1636226

tags: added: in-stable-mitaka
Changed in fuel:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (master)

Reviewed: https://review.openstack.org/390369
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=348e1b685650f2a5ebf6709bf64ca209aa0a42fe
Submitter: Jenkins
Branch: master

commit 348e1b685650f2a5ebf6709bf64ca209aa0a42fe
Author: Alexander Kurenyshev <email address hidden>
Date: Mon Oct 24 18:12:49 2016 +0300

    Fix testrail upload cases script

    When docstring is empty test rail report tool fails

    Change-Id: Iafee2ba4712ec9e86b9310c703b26bbf2f0b39e6
    Closes-Bug:1636226

tags: added: in-stable-newton
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-qa (stable/newton)

Reviewed: https://review.openstack.org/390370
Committed: https://git.openstack.org/cgit/openstack/fuel-qa/commit/?id=04741701a756331bbe0ed6aaf88464dc70a8207d
Submitter: Jenkins
Branch: stable/newton

commit 04741701a756331bbe0ed6aaf88464dc70a8207d
Author: Alexander Kurenyshev <email address hidden>
Date: Mon Oct 24 18:12:49 2016 +0300

    Fix testrail upload cases script

    When docstring is empty test rail report tool fails

    Change-Id: Iafee2ba4712ec9e86b9310c703b26bbf2f0b39e6
    Closes-Bug:1636226

Revision history for this message
TatyanaGladysheva (tgladysheva) wrote :

Verified on 9.2 snapshot #515.

Since https://product-ci.infra.mirantis.net/view/9.x_acceptance/job/9.x.acceptance.test-reports/630/ (Oct 28, 2016) all jobs are finished successfully.

Changed in fuel:
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/fuel-qa 11.0.0.0rc1

This issue was fixed in the openstack/fuel-qa 11.0.0.0rc1 release candidate.

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.