fix: check if a user is signed in before attempting to sign them out

[mathieuartu]

Description

This PR adds a isUserSignedIn check before attempting to sign him out.

Related issues

Fixes: https://github.com/MetaMask/MetaMask-planning/issues/3709

Manual testing steps

1. Go to this page...

Screenshots/Recordings

Before

After

Pre-merge author checklist

• I've followed MetaMask Contributor Docs and MetaMask Extension Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[Prithpal-Sooriya]

LGTM - this ensures that we check the auth state before performing sign out 👍🏾

We can create a separate PR that decouples UI interactions and instead uses the background messaging stateChanged event - see gist

[metamaskbot]

Builds ready [b0bfe1b]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7
  • common: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (2448 ± 380 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	314	4328	2007	1100	528
		domContentLoaded	1633	4279	2406	782	376
		load	1676	4343	2448	790	380
		domInteractive	29	495	99	101	48
		backgroundConnect	9	121	47	36	17
		firstReactRender	83	586	152	112	54
		getState	4	75	30	23	11
		initialActions	0	1	0	0	0
		loadScripts	1193	3357	1796	649	312
		setupStore	6	57	16	15	7
		uiStartup	1828	5203	2749	954	458

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 43 Bytes (0.00%)
• common: 0 Bytes (0.00%)

[metamaskbot]

Builds ready [86b39c9]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6, 7
  • common: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9
  • content-script: 0
  • offscreen: 0
  • ui: 0, 1, 10, 11, 12, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (2057 ± 69 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	711	2351	1992	329	158
		domContentLoaded	1791	2334	2024	148	71
		load	1802	2348	2057	143	69
		domInteractive	24	83	37	15	7
		backgroundConnect	8	71	27	19	9
		firstReactRender	16	95	30	27	13
		getState	107	321	165	63	30
		initialActions	0	1	0	0	0
		loadScripts	1364	1857	1571	134	64
		setupStore	7	83	12	16	8
		uiStartup	2076	2805	2430	198	95

Bundle size diffs

• background: 0 Bytes (0.00%)
• ui: 43 Bytes (0.00%)
• common: 0 Bytes (0.00%)