Skip to content

Commit b4e6f42

Browse files
committed
file_size: require to be existing file
1 parent 71fbe7b commit b4e6f42

File tree

2 files changed

+3
-10
lines changed

2 files changed

+3
-10
lines changed

+stdlib/file_size.m

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,9 @@
77

88
function s = file_size(p)
99
arguments
10-
p (1,1) string
10+
p {mustBeTextScalar,mustBeFile}
1111
end
1212

13-
s = [];
14-
if ~isfile(p), return, end
15-
1613
if stdlib.isoctave()
1714
s = stat(p);
1815
if ~isempty(s)

test/TestFileImpure.m

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,19 +14,15 @@
1414

1515
ph = {{0, '"stdin"'}, {1, '"stdout"'}, {2, '"stderr"'}, {fopen(tempname()), ""}}
1616

17-
p_file_size = {mfilename("fullpath") + ".m", "", pwd()}
17+
p_file_size = {mfilename("fullpath") + ".m"}
1818
end
1919

2020

2121
methods (Test)
2222

2323
function test_file_size(tc, p_file_size)
2424
s = stdlib.file_size(p_file_size);
25-
if isfile(p_file_size)
26-
tc.verifyGreaterThan(s, 0)
27-
else
28-
tc.verifyEmpty(s)
29-
end
25+
tc.verifyGreaterThan(s, 0)
3026
end
3127

3228

0 commit comments

Comments
 (0)