Try to get lgtm to stop complaining about sensitive info in localStorage

[imnasnainaec]

This change is 

[lgtm-com]

This pull request fixes 1 alert when merging dac473e into 344d7a4 - view on LGTM.com

fixed alerts:

• 1 for Clear text storage of sensitive information

[johnthagen]

---

src/goals/DefaultGoal/BaseGoalScreen/tests/BaseGoalScreen.test.tsx, line 41 at r2 (raw file):

  const div = document.createElement("div");
  let user: User = new User("TestUser", "TestUsername", "password");

Was LGTM flagging this because we used the name "password"?

[imnasnainaec]

Reviewable status: 0 of 4 files reviewed, all discussions resolved (waiting on @johnthagen)

---

src/goals/DefaultGoal/BaseGoalScreen/tests/BaseGoalScreen.test.tsx, line 41 at r2 (raw file):

Previously, johnthagen wrote…

Was LGTM flagging this because we used the name "password"?

That's my understanding.

[johnthagen]

Reviewed 4 of 4 files at r1.
Reviewable status: complete! all files reviewed, all discussions resolved

[johnthagen]

Reviewable status: complete! all files reviewed, all discussions resolved

[johnthagen]

Nice clean up included in this PR too. ✔️

[lgtm-com]

This pull request fixes 1 alert when merging 89c0b5a into 344d7a4 - view on LGTM.com

fixed alerts:

• 1 for Clear text storage of sensitive information