Commitd3f5438

authored

chore: throw FileNotFoundError for nonexistant files (#3014)

1 parent731b539 commitd3f5438Copy full SHA for d3f5438

File tree

-1

lines changed

-1

lines changed

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -14,6 +14,7 @@`
`14`	`14`	`importbase64`
`15`	`15`	`importcollections.abc`
`16`	`16`	`importos`
	`17`	`+importstat`
`17`	`18`	`frompathlibimportPath`
`18`	`19`	`fromtypingimport (`
`19`	`20`	`TYPE_CHECKING,`
`@@ -144,7 +145,8 @@ def resolve_paths_and_directory_for_input_files(`
`144`	`145`	`local_paths:Optional[List[str]]=None`
`145`	`146`	`local_directory:Optional[str]=None`
`146`	`147`	`foriteminitems:`
`147`		`-ifos.path.isdir(item):`
	`148`	`+item_stat=os.stat(item)# Raises FileNotFoundError if doesn't exist`
	`149`	`+ifstat.S_ISDIR(item_stat.st_mode):`
`148`	`150`	`iflocal_directory:`
`149`	`151`	`raiseError("Multiple directories are not supported")`
`150`	`152`	`local_directory=str(Path(item).resolve())`

Lines changed: 6 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -327,6 +327,12 @@ def test_locators_should_upload_a_file(page: Page, server: Server) -> None:`
`327`	`327`	`)`
`328`	`328`
`329`	`329`
	`330`	`+deftest_locators_upload_nonexistant_file(page:Page,server:Server)->None:`
	`331`	`+page.goto(server.PREFIX+"/input/fileupload.html")`
	`332`	`+withpytest.raises(FileNotFoundError):`
	`333`	`+page.locator("input[type=file]").set_input_files("nonexistant.html")`
	`334`	`+`
	`335`	`+`
`330`	`336`	`deftest_locators_should_press(page:Page)->None:`
`331`	`337`	`page.set_content("<input type='text' />")`
`332`	`338`	`page.locator("input").press("h")`

Comments

(0)