workbook: add check for blank worksheet name

Issue #442
This commit is contained in:
John McNamara 2024-05-08 19:59:22 +01:00
parent 5bd28dfccf
commit 284b61ba0b
5 changed files with 43 additions and 0 deletions

View file

@ -12,6 +12,7 @@
#include "../../../include/xlsxwriter/workbook.h"
#include "../../../include/xlsxwriter/shared_strings.h"
/* Test a valid sheet name. */
CTEST(workbook, validate_worksheet_name01) {
@ -145,3 +146,31 @@ CTEST(workbook, validate_worksheet_name09) {
lxw_workbook_free(workbook);
}
/* Test for blank sheet name. */
CTEST(workbook, validate_worksheet_name10) {
const char* sheetname = "";
lxw_workbook *workbook = workbook_new(NULL);
lxw_error exp = LXW_ERROR_SHEETNAME_IS_BLANK;
lxw_error got = workbook_validate_sheet_name(workbook, sheetname);
ASSERT_EQUAL(exp, got);
lxw_workbook_free(workbook);
}
/* Test for NULL sheet name. */
CTEST(workbook, validate_worksheet_name11) {
const char* sheetname = NULL;
lxw_workbook *workbook = workbook_new(NULL);
lxw_error exp = LXW_ERROR_NULL_PARAMETER_IGNORED;
lxw_error got = workbook_validate_sheet_name(workbook, sheetname);
ASSERT_EQUAL(exp, got);
lxw_workbook_free(workbook);
}