Refacto authentication and add tests suite

GitGuardian / gitguardian-vscode

Stop leaks. Safeguard your secrets with GitGuardian. GitGuardian actively prevents accidental exposure of sensitive information in your code, allowing you to code confidently and maintain the integrity of your data.

https://marketplace.visualstudio.com/items?itemName=gitguardian-secret-security.gitguardian

MIT License

21 stars 0 forks source link

Refacto authentication and add tests suite #38

Closed salome-voltz closed 3 weeks ago

salome-voltz commented 1 month ago

This PR has two goals:

Add a test suite, and especially tests for authentication
Refactor how the authentication status is checked.

gg-jonathangriffe commented 1 month ago

I think a better organisation for tests would be to put them in files that have the same name as the file in which the function we want to tests is. For example tests for ggshieldAuthStatus should be in a ggshield-api.test.ts. WDYT ?

gg-jonathangriffe commented 1 month ago

Can you also push the changes to the yarn.lock

gg-jonathangriffe commented 1 month ago

Also, we should add simple-mock as a dev dependency not a regular one.