Fix eraseCookies domain bug and expand tests

[imnasnainaec]

Improvements in this pr:

• Expand docs for eraseCookies
• In eraseCookiesHelper, don't use erase(cookieName); because that causes a bug, deleting unintended cookies when a domain was specified; rather, allow erase(cookieName, customDomain); to cover the case when customDomain is empty
• In the erase function, only add a . to the start of the domain if it's nonempty and doesn't already start with a .
• Update cookie erasure tests to verify the cookie is there before it's erased
• Add more tests to ensure that a wrong path or wrong domain will not delete a cookie

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment

Name	Status	Preview	Comments	Updated (UTC)
cookieconsent-docs	⬜️ Ignored (Inspect)	Visit Preview		Aug 14, 2024 2:07pm

[netlify]

✅ Deploy Preview for cookieconsentv3-playground canceled.

Name	Link
🔨 Latest commit	bebdd6f
🔍 Latest deploy log	https://app.netlify.com/sites/cookieconsentv3-playground/deploys/66bcb9fec55c270008dafa7d

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[imnasnainaec]

@orestbida You had a comment about this pr here, which I responded to. I'm not sure if there were other things you wish to review or test.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

[imnasnainaec]

Improvements in this pr:

• Expand docs for eraseCookies
• In eraseCookiesHelper, don't use erase(cookieName); because that causes a bug, deleting unintended cookies when a domain was specified; rather, allow erase(cookieName, customDomain); to cover the case when customDomain is empty
• In the erase function, only add a . to the start of the domain if it's nonempty and doesn't already start with a .
• Update cookie erasure tests to verify the cookie is there before it's erased
• Add more tests to ensure that a wrong path or wrong domain will not delete a cookie

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.