Reset FileInput value after load

[AustinMroz]

When a file is browsed for, the fileInput remains associated with the chosen file even after the associated workflow is loaded. If a user attempts to load the same file again, the onchange event does not fire and loading fails silently. This is fixed by clearing the fileInput after the workflow has been loaded.

Out of an abundance of caution, the onchange is made async to ensure the fileInput isn't cleared until after the workflow has loaded.

This is a long standing issue which had befuddled me for many months.

[huchenlei]

Can you add a playwright testcase for this change? See https://github.com/Comfy-Org/ComfyUI_frontend/tree/main/browser_tests

[huchenlei]

You can use

ComfyUI_frontend/browser_tests/ComfyPage.ts

Lines 326 to 331 in 2387a5e

	async loadWorkflow(workflowName: string) {
	await this.workflowUploadInput.setInputFiles(
	this.assetPath(`${workflowName}.json`)
	)
	await this.nextFrame()
	}

for file upload.

[huchenlei]

Please put the setting changes in before/after tests for proper setup/teardown, as if the test expectation fails, the setting side-effect can cause other tests to fail.

[AustinMroz]

Is there a way to have the before and after functions to apply to just this test and not the Can load workflow with string node id test other than creating a new test group?

[huchenlei]

Is there a way to have the before and after functions to apply to just this test and not the Can load workflow with string node id test other than creating a new test group?

I think the current way is to have a new describe group to hold the extra setup/teardown logic.