test(e2e): Add additional assertions to UI tests

[moduli]

Description

This PR is a follow-up to this comment: #2563 (comment)

This PR adds additional assertions to UI tests, specifically, ensuring that a session is successfully created after setting up resources via the Admin UI.

How to Test

Run e2e tests

Checklist

• I have added before and after screenshots for UI changes
• I have added JSON response output for API changes
• I have added steps to reproduce and test for bug fixes in the description
• I have commented on my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works

https://hashicorp.atlassian.net/browse/ICU-15765

[vercel]

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

Name	Status	Preview	Comments	Updated (UTC)
boundary-ui	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 2, 2025 8:22pm
boundary-ui-desktop	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 2, 2025 8:22pm

[ZedLi]

Looks good! Just had a few more comments

[ZedLi]

Does this fix the flakiness? I'm wondering if it might be better to wrap the exec in a promise and return that and that way you wait on any buffered text as a response and can inspect stdout/stderr?

We do something similar here

boundary-ui/ui/desktop/electron-app/src/helpers/spawn-promise.js

Lines 104 to 128 in 401ff35

	spawn(command, options) {
	return new Promise((resolve, reject) => {
	const childProcess = spawn(path, command, options);
	childProcess.stdout.on('data', (data) => {
	resolve({ childProcess, stdout: data.toString() });
	});
	childProcess.stderr.on('data', (data) => {
	resolve({ childProcess, stderr: data.toString() });
	});
	childProcess.on('error', (err) => reject(err));
	// In case the process has no stdio and didn't error out, resolve a
	// promise once the child process closes so we're not waiting forever.
	// Windows doesn't seem to return any error nor any output from stderr when
	// a timeout occurs so this guarantees we return a response to the caller.
	// Otherwise this should not get hit as we should be returning a response
	// from one of the handlers above.
	childProcess.on('close', () =>
	resolve({
	childProcess,
	stderr: JSON.stringify({ error: 'Process was closed.' }),
	}),
	);
	});
	},