From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <daniel.sangorrin@toshiba.co.jp>
Content-Transfer-Encoding: 7bit
From: Daniel Sangorrin <daniel.sangorrin@toshiba.co.jp>
Date: Wed, 13 Dec 2017 17:42:16 +0900
Message-Id: <1513154537-11933-2-git-send-email-daniel.sangorrin@toshiba.co.jp>
In-Reply-To: <1513154537-11933-1-git-send-email-daniel.sangorrin@toshiba.co.jp>
References: <1513154537-11933-1-git-send-email-daniel.sangorrin@toshiba.co.jp>
Subject: [Fuego] [PATCH 1/2] LTP:ltp_process: dont crash if no sheets are
	generated
List-Id: Mailing list for the Fuego test framework
	<fuego.lists.linuxfoundation.org>
List-Unsubscribe: <https://lists.linuxfoundation.org/mailman/options/fuego>,
	<mailto:fuego-request@lists.linuxfoundation.org?subject=unsubscribe>
List-Archive: <http://lists.linuxfoundation.org/pipermail/fuego/>
List-Post: <mailto:fuego@lists.linuxfoundation.org>
List-Help: <mailto:fuego-request@lists.linuxfoundation.org?subject=help>
List-Subscribe: <https://lists.linuxfoundation.org/mailman/listinfo/fuego>,
	<mailto:fuego-request@lists.linuxfoundation.org?subject=subscribe>
To: fuego@lists.linuxfoundation.org

This fixes the following error that occurs when running the rtonly spec,
which does not generate any sheet.

Traceback (most recent call last):
  File "ltp_process.py", line 338, in <module>
    book.save('results.xlsx')
  File "/usr/lib/pymodules/python2.7/openpyxl/workbook.py", line 232, in save
    save_workbook(self, filename)
  File "/usr/lib/pymodules/python2.7/openpyxl/writer/excel.py", line 159, in save_workbook
    writer.save(filename)
  File "/usr/lib/pymodules/python2.7/openpyxl/writer/excel.py", line 142, in save
    self.write_data(archive)
  File "/usr/lib/pymodules/python2.7/openpyxl/writer/excel.py", line 73, in write_data
    archive.writestr(ARC_WORKBOOK, write_workbook(self.workbook))
  File "/usr/lib/pymodules/python2.7/openpyxl/writer/workbook.py", line 170, in write_workbook
    SubElement(book_views, 'workbookView', {'activeTab': '%d' % workbook.get_index(workbook.get_active_sheet()),
  File "/usr/lib/pymodules/python2.7/openpyxl/workbook.py", line 124, in get_active_sheet
    return self.worksheets[self._active_sheet_index]
IndexError: list index out of range

Signed-off-by: Daniel Sangorrin <daniel.sangorrin@toshiba.co.jp>
---
 engine/tests/Functional.LTP/ltp_process.py | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

diff --git a/engine/tests/Functional.LTP/ltp_process.py b/engine/tests/Functional.LTP/ltp_process.py
index 1658943..867ef7a 100644
--- a/engine/tests/Functional.LTP/ltp_process.py
+++ b/engine/tests/Functional.LTP/ltp_process.py
@@ -88,9 +88,6 @@ def read_output (test_category, test_case):
 # Open a workbook
 book = Workbook()
 
-# Delete the default sheet
-book.remove_sheet(book.get_sheet_by_name("Sheet"))
-
 # Create style setting before the for loop
 # This is needed to avoid excel errors (too many font settings)
 medium_border = Border()
@@ -334,6 +331,10 @@ if os.path.exists('pts.log'):
         # last working sheet
         pts_set_style(ws)
 
+# if we have added sheets, remove the default one ("Sheet")
+sheets = book.get_sheet_names()
+if len(sheets) > 1:
+    book.remove_sheet(book.get_sheet_by_name("Sheet"))
+
 # save the results
 book.save('results.xlsx')
-
-- 
2.7.4