Skip to content

Commit a4d599c

Browse files
committed
test: teardown survive if cleanup fails
1 parent f95c609 commit a4d599c

File tree

9 files changed

+18
-9
lines changed

9 files changed

+18
-9
lines changed

test/TestAbsolute.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ function set_temp_wd(tc)
2525
methods(TestClassTeardown)
2626
function remove_temp_wd(tc)
2727
if isMATLABReleaseOlderThan('R2022a')
28-
rmdir(tc.td, 's');
28+
[s, m, i] = rmdir(tc.td, 's');
29+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
2930
end
3031
end
3132
end

test/TestHDF5.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,8 @@ function setup_file(tc)
7070
methods(TestClassTeardown)
7171
function remove_temp_wd(tc)
7272
if isMATLABReleaseOlderThan('R2022a')
73-
rmdir(tc.td, 's');
73+
[s, m, i] = rmdir(tc.td, 's');
74+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
7475
end
7576
end
7677
end

test/TestHash.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@ function set_temp_wd(tc)
2424
methods(TestMethodTeardown)
2525
function remove_temp_wd(tc)
2626
if isMATLABReleaseOlderThan('R2022a')
27-
rmdir(tc.td, 's');
27+
[s, m, i] = rmdir(tc.td, 's');
28+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
2829
end
2930
end
3031
end

test/TestJava.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,8 @@ function set_temp_wd(tc)
1818
methods(TestClassTeardown)
1919
function remove_temp_wd(tc)
2020
if isMATLABReleaseOlderThan('R2022a')
21-
rmdir(tc.td, 's');
21+
[s, m, i] = rmdir(tc.td, 's');
22+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
2223
end
2324
end
2425
end

test/TestNetCDF.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,8 @@ function setup_file(tc)
6464
methods(TestClassTeardown)
6565
function remove_temp_wd(tc)
6666
if isMATLABReleaseOlderThan('R2022a')
67-
rmdir(tc.td, 's');
67+
[s, m, i] = rmdir(tc.td, 's');
68+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
6869
end
6970
end
7071
end

test/TestPermissions.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,8 @@ function set_temp_wd(tc)
2424
methods(TestClassTeardown)
2525
function remove_temp_wd(tc)
2626
if isMATLABReleaseOlderThan('R2022a')
27-
rmdir(tc.td, 's');
27+
[s, m, i] = rmdir(tc.td, 's');
28+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
2829
end
2930
end
3031
end

test/TestResolve.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@ function set_temp_wd(tc)
2323
methods(TestClassTeardown)
2424
function remove_temp_wd(tc)
2525
if isMATLABReleaseOlderThan('R2022a')
26-
rmdir(tc.td, 's');
26+
[s, m, i] = rmdir(tc.td, 's');
27+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
2728
end
2829
end
2930
end

test/TestSubprocess.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,8 @@ function set_temp_wd(tc)
2323
methods(TestClassTeardown)
2424
function remove_temp_wd(tc)
2525
if isMATLABReleaseOlderThan('R2022a')
26-
rmdir(tc.td, 's');
26+
[s, m, i] = rmdir(tc.td, 's');
27+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
2728
end
2829
end
2930
end

test/TestSymlink.m

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,8 @@ function setup_symlink(tc)
3737
methods(TestClassTeardown)
3838
function remove_temp_wd(tc)
3939
if isMATLABReleaseOlderThan('R2022a')
40-
rmdir(tc.td, 's');
40+
[s, m, i] = rmdir(tc.td, 's');
41+
if ~s, warning(i, "Failed to remove temporary directory %s: %s", tc.td, m); end
4142
end
4243
end
4344
end

0 commit comments

Comments
 (0)