[PATCH v5 2/6] robocopy/tests: Add basic syntax tests
Florian Eder
others.meder at gmail.com
Mon Sep 20 13:51:22 CDT 2021
Basic files and required changes to configure(.ac) to create basic syntax tests and
a scaffolding for the conformance tests to follow for the robocopy utility
Signed-off-by: Florian Eder <others.meder at gmail.com>
---
v5: Added *very* basic syntax tests, as the unit tests following the implementation of each switch / flag will
automatically test the correct syntax also
Made execute_robocopy only accept args instead of the whole commandline, dynamically allocate the commandline string,
changed the return value of execute_robocopy from -1 to explicitly MAXDWORD and reduced the size of temp_path to MAX_PATH,
which is the max size that GetTempPathW can return
---
configure | 1 +
configure.ac | 1 +
programs/robocopy/tests/Makefile.in | 4 ++
programs/robocopy/tests/robocopy.c | 87 +++++++++++++++++++++++++++++
4 files changed, 93 insertions(+)
create mode 100644 programs/robocopy/tests/Makefile.in
create mode 100644 programs/robocopy/tests/robocopy.c
diff --git a/configure b/configure
index a5767af3c60..d63321522f4 100755
--- a/configure
+++ b/configure
@@ -21235,6 +21235,7 @@ wine_fn_config_makefile programs/regini enable_regini
wine_fn_config_makefile programs/regsvcs enable_regsvcs
wine_fn_config_makefile programs/regsvr32 enable_regsvr32
wine_fn_config_makefile programs/robocopy enable_robocopy
+wine_fn_config_makefile programs/robocopy/tests enable_tests
wine_fn_config_makefile programs/rpcss enable_rpcss
wine_fn_config_makefile programs/rundll.exe16 enable_win16
wine_fn_config_makefile programs/rundll32 enable_rundll32
diff --git a/configure.ac b/configure.ac
index 97c46565db3..6d89ceea089 100644
--- a/configure.ac
+++ b/configure.ac
@@ -3959,6 +3959,7 @@ WINE_CONFIG_MAKEFILE(programs/regini)
WINE_CONFIG_MAKEFILE(programs/regsvcs)
WINE_CONFIG_MAKEFILE(programs/regsvr32)
WINE_CONFIG_MAKEFILE(programs/robocopy)
+WINE_CONFIG_MAKEFILE(programs/robocopy/tests)
WINE_CONFIG_MAKEFILE(programs/rpcss)
WINE_CONFIG_MAKEFILE(programs/rundll.exe16,enable_win16)
WINE_CONFIG_MAKEFILE(programs/rundll32)
diff --git a/programs/robocopy/tests/Makefile.in b/programs/robocopy/tests/Makefile.in
new file mode 100644
index 00000000000..d86e70f4c43
--- /dev/null
+++ b/programs/robocopy/tests/Makefile.in
@@ -0,0 +1,4 @@
+TESTDLL = robocopy.exe
+
+C_SRCS = \
+ robocopy.c
diff --git a/programs/robocopy/tests/robocopy.c b/programs/robocopy/tests/robocopy.c
new file mode 100644
index 00000000000..8c594b78d92
--- /dev/null
+++ b/programs/robocopy/tests/robocopy.c
@@ -0,0 +1,87 @@
+/*
+ * Copyright 2021 Florian Eder
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
+ */
+
+#define WIN32_LEAN_AND_MEAN
+#include <windows.h>
+#include <wchar.h>
+#include <wine/test.h>
+
+static DWORD execute_robocopy(const WCHAR *args)
+{
+ STARTUPINFOW startup_info;
+ PROCESS_INFORMATION process_info;
+ DWORD exit_code;
+ WCHAR *cmd;
+
+ memset(&startup_info, 0, sizeof(STARTUPINFOW));
+ startup_info.dwFlags = STARTF_USESTDHANDLES;
+ startup_info.hStdInput = INVALID_HANDLE_VALUE;
+ startup_info.hStdOutput = INVALID_HANDLE_VALUE;
+ startup_info.hStdError = INVALID_HANDLE_VALUE;
+
+ cmd = calloc(ARRAY_SIZE(L"robocopy.exe ") + wcslen(args), sizeof(WCHAR));
+ wcscpy(cmd, L"robocopy.exe ");
+ wcscat(cmd, args);
+
+ if (!CreateProcessW(NULL, cmd, NULL, NULL, TRUE, 0, NULL, NULL, &startup_info, &process_info))
+ return MAXDWORD;
+
+ if (WaitForSingleObject(process_info.hProcess, 30000) == WAIT_TIMEOUT)
+ return MAXDWORD;
+
+ GetExitCodeProcess(process_info.hProcess, &exit_code);
+
+ CloseHandle(process_info.hThread);
+ CloseHandle(process_info.hProcess);
+
+ return exit_code;
+}
+
+START_TEST(robocopy)
+{
+ DWORD exit_code;
+ WCHAR temp_path[MAX_PATH];
+
+ /* robocopy is only available from Vista onwards, abort test if not available */
+ if (execute_robocopy(L"") == MAXDWORD) return;
+
+ ok(GetTempPathW(ARRAY_SIZE(temp_path), temp_path) != 0, "couldn't get temp folder path");
+
+ ok(SetCurrentDirectoryW(temp_path), "couldn't set CWD to temp folder \"%s\"", debugstr_w(temp_path));
+
+ winetest_push_context("syntax test 1");
+ RemoveDirectoryW(L"invalid_folder");
+ exit_code = execute_robocopy(L"");
+ todo_wine ok(exit_code == 16, "unexpected exit code %d\n", exit_code);
+ winetest_pop_context();
+
+ winetest_push_context("syntax test 2");
+ exit_code = execute_robocopy(L"invalid_folder");
+ todo_wine ok(exit_code == 16, "unexpected exit code %d\n", exit_code);
+ winetest_pop_context();
+
+ winetest_push_context("syntax test 3");
+ exit_code = execute_robocopy(L"-flag invalid_folder");
+ todo_wine ok(exit_code == 16, "unexpected exit code %d\n", exit_code);
+ winetest_pop_context();
+
+ winetest_push_context("syntax test 4");
+ exit_code = execute_robocopy(L"invalid_folder robocopy_destination");
+ todo_wine ok(exit_code == 16, "unexpected exit code %d\n", exit_code);
+ winetest_pop_context();
+}
--
2.32.0
More information about the wine-devel
mailing list