Feat: AddDataValidation() and DeleteDataValidation() are concurrency safe, and add some unit test and code optimization.

[JackMin1314]

PR Details

This pr contains two commits, non-breaking change which fixes an issue and adds a function.

1. fix: data validation is unsafe when used in concurrent condition#1825
2. feat: supports batch append data validation to the worksheet and optimse code#1826

Description

1. For the first commit which open related issue DataValidation is unsafe when used in concurrent condition #1825 and has detail description about this. Use the mutex's method Lock() and Unlock() which is in the *File object.

2. Optimse code: Extract some code about xlsxDataValidation creation and keep the contents of the AddDataValidation function clean

3. And a new function named BatchAddDataValidation() to batch append data validation to the worksheet for some typical excel data validation template scenario.

Related Issue

#1825

#1826

Motivation and Context

How Has This Been Tested

There are some test cases already written in test file datavalidation_test.go and two testing functions: TestConcurrentAddDataValidation() for high concurrency and TestBatchAddDataValidation() to batch append data validation.

Types of changes

• Docs change / refactoring / dependency upgrade
• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[xuri]

Thanks for your pull request. I've left some comments.

[codecov]

Codecov Report

Attention: Patch coverage is 95.45455% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 99.21%. Comparing base (7b4da39) to head (277df9f).

Files	Patch %	Lines
adjust.go	75.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1828   +/-   ##
=======================================
  Coverage   99.21%   99.21%           
=======================================
  Files          32       32           
  Lines       23926    23930    +4     
=======================================
+ Hits        23737    23741    +4     
  Misses        101      101           
  Partials       88       88           

Flag	Coverage Δ
unittests	99.21% <95.45%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[xuri]

LGTM, thanks for your contribution.