Add more cases to escapeFormulae, allow to pass RegExp

[caub]

See #796 #793

@pokoli @wcerfgba Hey guys, could you check this PR?

It adds more cases for escapeFormulae option, cf https://owasp.org/www-community/attacks/CSV_Injection

[caub]

I added possibility to pass a RegExp as escapeFormulae option, but not sure it's needed, just tell me, I can remove that

[pokoli]

Hi @caub!

Thanks for your suggestion. Could you please keep the current tests cases as is (so we ensure nothing is borken) and add a new test case for the regexp validation (the new feature) ?

Thanks in advance!

[caub]

Hey @pokoli
Current tests are actually changed, https://owasp.org/www-community/attacks/CSV_Injection recommanded to escape values anyway as soon as a formulae is used

Not sure if really necessary but what's your thought? can we revert my last commit if you prefer? (and then previous tests will be identical)

[caub]

I'll start separating anyway so it's clearer, you're right

[pokoli]

@caub feel free to remove some test suit if it is no longer valid (but please keep the valid ones)

[caub]

ok done, https://github.com/mholt/PapaParse/pull/904/files you can see 2 of the previous tests are affected, because for example =test was converted in '=test as it doesn't require escaping the whole value, with owasp recommandation we get either "'=test" or '''=test'

[pokoli]

Hi @caub,

Tests failed due to node15 (which seems unrelated). I removed the version from github actions in 26a86fd

Could you please rebase the PR so I can relauch the test suite to ensure everything is in place.

Sorry for the inconvenience.

[caub]

yep, done

[pokoli]

Merged it. Thanks