Skip to content

Commit

Permalink
Release v1.1.5 support merge and count excel data
Browse files Browse the repository at this point in the history
  • Loading branch information
atthaboon committed Mar 21, 2020
1 parent d9fbb59 commit 8ea9cb1
Showing 1 changed file with 9 additions and 9 deletions.
18 changes: 9 additions & 9 deletions ExcelDataDriver/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
from ExcelDataDriver.Config.CaptureScreenShotOption import CaptureScreenShotOption


__version__ = '1.1.4'
__version__ = '1.1.5'


class ExcelDataDriver:
Expand Down Expand Up @@ -315,7 +315,7 @@ def update_test_result_if_keyword_fail(self, log_message=None, screenshot=None):
self.excelTestDataService.update_test_result(keyword_status, log_message, screenshot)

@keyword
def merged_excel_report(self, data_type='DefaultParserStrategy'):
def merged_excel_report(self, main_column_key, data_type='DefaultParserStrategy'):
"""
Merged all test data from report folder into summary_report.xlsx under summary_report_folder
Expand Down Expand Up @@ -344,8 +344,10 @@ def merged_excel_report(self, data_type='DefaultParserStrategy'):
overall_test_status_is_pass = True
summary_error_message = ''

CustomExcelParser = self.load_module(data_type)
parser_strategy = CustomExcelParser.CustomExcelParser()
parser_strategy = DefaultParserStrategy(main_column_key)
if data_type != 'DefaultParserStrategy':
CustomExcelParser = self.load_module(data_type)
parser_strategy = CustomExcelParser.CustomExcelParser()
parser_context = ParserContext(parser_strategy)

print('Parse wb')
Expand All @@ -359,8 +361,6 @@ def merged_excel_report(self, data_type='DefaultParserStrategy'):
for report in reports:
print('Merged ws test datas : '+report)
wb = OpenpyxlHelper.load_excel_file(report, data_only=False, keep_vba=False)
CustomExcelParser = self.load_module(data_type)
parser_strategy = CustomExcelParser.CustomExcelParser()
parser_context = ParserContext(parser_strategy)
wb_test_datas = parser_context.parse(wb)

Expand Down Expand Up @@ -390,7 +390,7 @@ def merged_excel_report(self, data_type='DefaultParserStrategy'):
#
####################################################
@keyword
def load_test_data(self, filename, data_type=None):
def load_test_data(self, filename, main_column_key, data_type=None):
"""
Load excel test data
Expand All @@ -405,10 +405,10 @@ def load_test_data(self, filename, data_type=None):
"""
if data_type is not None:
CustomExcelParser = self.load_module(data_type)
custom_parser = CustomExcelParser.CustomExcelParser()
custom_parser = CustomExcelParser.CustomExcelParser(main_column_key)
self.excelTestDataService.load_test_data(filename, custom_parser)
else:
self.excelTestDataService.load_test_data(filename, DefaultParserStrategy())
self.excelTestDataService.load_test_data(filename, DefaultParserStrategy(main_column_key))

@keyword
def get_all_test_data(self, rerun_only_failed=False, offset_row=0, maximum_row=None):
Expand Down

0 comments on commit 8ea9cb1

Please sign in to comment.