[PATCH v7 0/1] MR10518: wscript: Implement error messages, usage output, and //nologo banner.
Report user-visible "Input Error" messages to stdout matching Windows behavior for missing script files and nonexistent files. Display usage information with Wine version when invoked with no arguments. Script runtime errors (OnScriptError) remain on stderr. Implement //nologo and //logo flags to control the banner displayed before script execution. Treat unrecognized options as filenames to support Unix-style paths on Wine. Add FIXME stubs for other recognized but unimplemented options (//X, //S, //E:, //H:, //Job:, //T:). -- v7: wscript: Implement error messages, usage output, and //nologo banner. https://gitlab.winehq.org/wine/wine/-/merge_requests/10518
From: Francis De Brabandere <francisdb@gmail.com> Report user-visible "Input Error" messages to stdout matching Windows behavior for missing script files and nonexistent files. Display usage information with Wine version when invoked with no arguments. Script runtime errors (OnScriptError) remain on stderr. Implement //nologo and //logo flags to control the banner displayed before script execution. Treat unrecognized options as filenames to support Unix-style paths on Wine. Add FIXME stubs for other recognized but unimplemented options (//X, //S, //E:, //H:, //Job:, //T:). --- programs/wscript/main.c | 140 ++++++++++++--- programs/wscript/tests/run.c | 329 ++++++++++++++++++++++++++++++++++- 2 files changed, 438 insertions(+), 31 deletions(-) diff --git a/programs/wscript/main.c b/programs/wscript/main.c index 2f1e517d419..897decb8e55 100644 --- a/programs/wscript/main.c +++ b/programs/wscript/main.c @@ -52,6 +52,8 @@ WCHAR scriptFullName[MAX_PATH]; ITypeInfo *host_ti; ITypeInfo *arguments_ti; +static BOOL nologo; + static HRESULT query_interface(REFIID,void**); static HRESULT WINAPI ActiveScriptSite_QueryInterface(IActiveScriptSite *iface, @@ -120,20 +122,15 @@ static HRESULT WINAPI ActiveScriptSite_OnStateChange(IActiveScriptSite *iface, return S_OK; } -static void print_error(const WCHAR *string) +static void write_to_handle(HANDLE handle, const WCHAR *string) { DWORD count, ret, len, lena; char *buf; - if(wshInteractive) { - MessageBoxW(NULL, string, L"Windows Script Host", MB_OK); - return; - } - len = lstrlenW(string); - ret = WriteConsoleW(GetStdHandle(STD_ERROR_HANDLE), string, len, &count, NULL); + ret = WriteConsoleW(handle, string, len, &count, NULL); if(ret) { - WriteConsoleW(GetStdHandle(STD_ERROR_HANDLE), L"\r\n", 2, &count, NULL); + WriteConsoleW(handle, L"\r\n", 2, &count, NULL); return; } @@ -143,9 +140,38 @@ static void print_error(const WCHAR *string) return; WideCharToMultiByte(GetOEMCP(), 0, string, len, buf, lena, NULL, NULL); - WriteFile(GetStdHandle(STD_ERROR_HANDLE), buf, lena, &count, FALSE); + WriteFile(handle, buf, lena, &count, FALSE); free(buf); - WriteFile(GetStdHandle(STD_ERROR_HANDLE), "\r\n", 2, &count, FALSE); + WriteFile(handle, "\r\n", 2, &count, FALSE); +} + +static void print_error(const WCHAR *string) +{ + if(wshInteractive) { + MessageBoxW(NULL, string, L"Windows Script Host", MB_OK); + return; + } + + write_to_handle(GetStdHandle(STD_ERROR_HANDLE), string); +} + +static void print_string(const WCHAR *string) +{ + write_to_handle(GetStdHandle(STD_OUTPUT_HANDLE), string); +} + +static void print_banner(void) +{ + const char * (CDECL *wine_get_version)(void); + WCHAR header[64]; + + wine_get_version = (void *)GetProcAddress(GetModuleHandleW(L"ntdll.dll"), "wine_get_version"); + if(wine_get_version) + swprintf(header, ARRAY_SIZE(header), L"Wine %S Script Host", wine_get_version()); + else + swprintf(header, ARRAY_SIZE(header), L"Wine Script Host"); + print_string(header); + print_string(L""); } static HRESULT WINAPI ActiveScriptSite_OnScriptError(IActiveScriptSite *iface, @@ -446,16 +472,27 @@ static BOOL set_host_properties(const WCHAR *prop) else if(wcsicmp(prop, L"b") == 0) wshInteractive = VARIANT_FALSE; else if(wcsicmp(prop, L"nologo") == 0) - WINE_FIXME("ignored %s switch\n", debugstr_w(L"nologo")); + nologo = TRUE; + else if(wcsicmp(prop, L"logo") == 0) + nologo = FALSE; else if(wcsicmp(prop, L"d") == 0) - WINE_FIXME("ignoring /d\n"); + WINE_FIXME("ignoring //d\n"); + else if(wcsicmp(prop, L"x") == 0) + WINE_FIXME("ignoring //x\n"); else if(wcsicmp(prop, L"u") == 0) - WINE_FIXME("ignoring /u\n"); + WINE_FIXME("ignoring //u\n"); + else if(wcsicmp(prop, L"s") == 0) + WINE_FIXME("ignoring //s\n"); + else if(wcsnicmp(prop, L"e:", 2) == 0) + WINE_FIXME("ignoring //e:\n"); + else if(wcsnicmp(prop, L"h:", 2) == 0) + WINE_FIXME("ignoring //h:\n"); + else if(wcsnicmp(prop, L"job:", 4) == 0) + WINE_FIXME("ignoring //job:\n"); + else if(wcsnicmp(prop, L"t:", 2) == 0) + WINE_FIXME("ignoring //t:\n"); else - { - WINE_FIXME("unsupported switch %s\n", debugstr_w(prop)); return FALSE; - } return TRUE; } @@ -475,26 +512,79 @@ int WINAPI wWinMain(HINSTANCE hInst, HINSTANCE hPrevInst, LPWSTR cmdline, int cm if(!argv) return 1; + /* Pass 1: consume // host options from all positions. */ + for(i=1; i<argc; i++) { + if(argv[i][0] == '/' && argv[i][1] == '/' && set_host_properties(argv[i])) + argv[i] = NULL; + } + + /* Pass 2: find filename, consuming single-/ and - options before it. */ for(i=1; i<argc; i++) { - if(*argv[i] == '/' || *argv[i] == '-') { - if(!set_host_properties(argv[i])) - return 1; - }else { - filename = argv[i]; - argums = argv+i+1; - numOfArgs = argc-i-1; - break; + if(!argv[i]) + continue; + if(argv[i][0] == '-' && argv[i][1] == '-' && !argv[i][2]) + continue; /* -- separator */ + if((*argv[i] == '/' || *argv[i] == '-') && set_host_properties(argv[i])) + continue; + filename = argv[i]; + i++; + break; + } + + /* Pass 3: compact script args, skipping consumed // options. */ + if(filename) { + int j = 0, first_arg = i; + for(; i<argc; i++) { + if(!argv[i]) + continue; + argv[first_arg + j] = argv[i]; + j++; } + argums = argv + first_arg; + numOfArgs = j; } + if(!filename && argc == 1) { + if(!nologo) + print_banner(); + print_string(L"Usage: CScript scriptname.extension [option...] [arguments...]"); + print_string(L""); + print_string(L"Options:"); + print_string(L" //B Batch mode: Suppresses script errors and prompts from displaying"); + print_string(L" //D Enable Active Debugging"); + print_string(L" //E:engine Use engine for executing script"); + print_string(L" //H:CScript Changes the default script host to CScript.exe"); + print_string(L" //H:WScript Changes the default script host to WScript.exe (default)"); + print_string(L" //I Interactive mode (default, opposite of //B)"); + print_string(L" //Job:xxxx Execute a WSF job"); + print_string(L" //Logo Display logo (default)"); + print_string(L" //Nologo Prevent logo display: No banner will be shown at execution time"); + print_string(L" //S Save current command line options for this user"); + print_string(L" //T:nn Time out in seconds: Maximum time a script is permitted to run"); + print_string(L" //X Execute script in debugger"); + print_string(L" //U Use Unicode for redirected I/O from the console"); + return 0; + } + + if(!nologo) + print_banner(); + if(!filename) { - WINE_FIXME("No file name specified\n"); + print_string(L"Input Error: There is no script file specified."); return 1; } + res = GetFullPathNameW(filename, ARRAY_SIZE(scriptFullName), scriptFullName, &filepart); if(!res || res > ARRAY_SIZE(scriptFullName)) return 1; + if(GetFileAttributesW(scriptFullName) == INVALID_FILE_ATTRIBUTES) { + WCHAR buf[MAX_PATH + 64]; + swprintf(buf, ARRAY_SIZE(buf), L"Input Error: Can not find script file \"%s\".", scriptFullName); + print_string(buf); + return 1; + } + ext = wcsrchr(filepart, '.'); if(!ext || !get_engine_clsid(ext, &clsid)) { WINE_FIXME("Could not find engine for %s\n", wine_dbgstr_w(ext)); diff --git a/programs/wscript/tests/run.c b/programs/wscript/tests/run.c index 9aae91ddc7a..abb01774117 100644 --- a/programs/wscript/tests/run.c +++ b/programs/wscript/tests/run.c @@ -324,20 +324,49 @@ static IClassFactory testobj_cf = { &ClassFactoryVtbl }; static void run_script_file(const char *file_name, DWORD expected_exit_code) { char command[MAX_PATH]; + SECURITY_ATTRIBUTES sa = {sizeof(sa), NULL, TRUE}; STARTUPINFOA si = {sizeof(si)}; PROCESS_INFORMATION pi; - DWORD exit_code; + HANDLE stdout_read, stdout_write; + HANDLE stderr_read, stderr_write; + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code, size; BOOL bres; script_name = file_name; sprintf(command, "wscript.exe %s arg1 2 ar3", file_name); + bres = CreatePipe(&stdout_read, &stdout_write, &sa, 0); + ok(bres, "CreatePipe failed: %lu\n", GetLastError()); + if(!bres) + return; + + bres = CreatePipe(&stderr_read, &stderr_write, &sa, 0); + ok(bres, "CreatePipe failed: %lu\n", GetLastError()); + if(!bres) { + CloseHandle(stdout_read); + CloseHandle(stdout_write); + return; + } + + SetHandleInformation(stdout_read, HANDLE_FLAG_INHERIT, 0); + SetHandleInformation(stderr_read, HANDLE_FLAG_INHERIT, 0); + + si.dwFlags = STARTF_USESTDHANDLES; + si.hStdInput = GetStdHandle(STD_INPUT_HANDLE); + si.hStdOutput = stdout_write; + si.hStdError = stderr_write; + SET_EXPECT(reportSuccess); bres = CreateProcessA(NULL, command, NULL, NULL, TRUE, 0, NULL, NULL, &si, &pi); + CloseHandle(stdout_write); + CloseHandle(stderr_write); if(!bres) { win_skip("script.exe is not available\n"); CLEAR_CALLED(reportSuccess); + CloseHandle(stdout_read); + CloseHandle(stderr_read); return; } @@ -348,8 +377,18 @@ static void run_script_file(const char *file_name, DWORD expected_exit_code) ok(bres, "GetExitCodeProcess failed: %lu\n", GetLastError()); ok(exit_code == expected_exit_code, "exit_code = %lu, expected %lu\n", exit_code, expected_exit_code); + memset(stderr_buf, 0, sizeof(stderr_buf)); + ReadFile(stderr_read, stderr_buf, sizeof(stderr_buf) - 1, &size, NULL); + stderr_buf[size] = 0; + ok(stderr_buf[0] == 0, "expected no stderr output, got: %s\n", stderr_buf); + + /* Drain stdout to prevent child from blocking on a full pipe */ + ReadFile(stdout_read, stdout_buf, sizeof(stdout_buf) - 1, &size, NULL); + CloseHandle(pi.hThread); CloseHandle(pi.hProcess); + CloseHandle(stdout_read); + CloseHandle(stderr_read); CHECK_CALLED(reportSuccess); } @@ -463,8 +502,9 @@ static void run_cscript_error_test(void) SECURITY_ATTRIBUTES sa = {sizeof(sa), NULL, TRUE}; STARTUPINFOA si = {sizeof(si)}; PROCESS_INFORMATION pi; + HANDLE stdout_read, stdout_write; HANDLE stderr_read, stderr_write; - char stderr_buf[4096]; + char stdout_buf[4096], stderr_buf[4096]; DWORD exit_code, size; HANDLE file; BOOL bres; @@ -481,23 +521,34 @@ static void run_cscript_error_test(void) if(!bres) goto cleanup; - bres = CreatePipe(&stderr_read, &stderr_write, &sa, 0); + bres = CreatePipe(&stdout_read, &stdout_write, &sa, 0); ok(bres, "CreatePipe failed: %lu\n", GetLastError()); if(!bres) goto cleanup; + bres = CreatePipe(&stderr_read, &stderr_write, &sa, 0); + ok(bres, "CreatePipe failed: %lu\n", GetLastError()); + if(!bres) { + CloseHandle(stdout_read); + CloseHandle(stdout_write); + goto cleanup; + } + + SetHandleInformation(stdout_read, HANDLE_FLAG_INHERIT, 0); SetHandleInformation(stderr_read, HANDLE_FLAG_INHERIT, 0); si.dwFlags = STARTF_USESTDHANDLES; si.hStdInput = GetStdHandle(STD_INPUT_HANDLE); - si.hStdOutput = GetStdHandle(STD_OUTPUT_HANDLE); + si.hStdOutput = stdout_write; si.hStdError = stderr_write; sprintf(command, "cscript.exe //nologo %s", file_name); bres = CreateProcessA(NULL, command, NULL, NULL, TRUE, 0, NULL, NULL, &si, &pi); + CloseHandle(stdout_write); CloseHandle(stderr_write); if(!bres) { win_skip("cscript.exe is not available\n"); + CloseHandle(stdout_read); CloseHandle(stderr_read); goto cleanup; } @@ -508,6 +559,11 @@ static void run_cscript_error_test(void) ok(bres, "GetExitCodeProcess failed: %lu\n", GetLastError()); ok(exit_code == 0, "exit_code = %lu\n", exit_code); + memset(stdout_buf, 0, sizeof(stdout_buf)); + ReadFile(stdout_read, stdout_buf, sizeof(stdout_buf) - 1, &size, NULL); + stdout_buf[size] = 0; + ok(stdout_buf[0] == 0, "expected no stdout with //nologo, got: %s\n", stdout_buf); + memset(stderr_buf, 0, sizeof(stderr_buf)); ReadFile(stderr_read, stderr_buf, sizeof(stderr_buf) - 1, &size, NULL); stderr_buf[size] = 0; @@ -515,17 +571,270 @@ static void run_cscript_error_test(void) ok(size > 0, "expected error output on stderr, got nothing\n"); ok(strstr(stderr_buf, "test_err.vbs(1,") != NULL, "expected file and line reference in error, got: %s\n", stderr_buf); - ok(strstr(stderr_buf, "Type mismatch") != NULL, - "expected 'Type mismatch' in output, got: %s\n", stderr_buf); CloseHandle(pi.hThread); CloseHandle(pi.hProcess); + CloseHandle(stdout_read); + CloseHandle(stderr_read); + +cleanup: + DeleteFileA(file_name); +} + +static DWORD run_cscript(const char *args, char *stdout_buf, size_t stdout_buf_size, + char *stderr_buf, size_t stderr_buf_size) +{ + char command[MAX_PATH]; + SECURITY_ATTRIBUTES sa = {sizeof(sa), NULL, TRUE}; + STARTUPINFOA si = {sizeof(si)}; + PROCESS_INFORMATION pi; + HANDLE stdout_read, stdout_write; + HANDLE stderr_read, stderr_write; + DWORD exit_code, size; + BOOL bres; + + bres = CreatePipe(&stderr_read, &stderr_write, &sa, 0); + ok(bres, "CreatePipe failed: %lu\n", GetLastError()); + if(!bres) + return ~0u; + + bres = CreatePipe(&stdout_read, &stdout_write, &sa, 0); + ok(bres, "CreatePipe failed: %lu\n", GetLastError()); + if(!bres) { + CloseHandle(stderr_read); + CloseHandle(stderr_write); + return ~0u; + } + + SetHandleInformation(stderr_read, HANDLE_FLAG_INHERIT, 0); + SetHandleInformation(stdout_read, HANDLE_FLAG_INHERIT, 0); + + si.dwFlags = STARTF_USESTDHANDLES; + si.hStdInput = GetStdHandle(STD_INPUT_HANDLE); + si.hStdOutput = stdout_write; + si.hStdError = stderr_write; + + sprintf(command, "cscript.exe %s", args); + bres = CreateProcessA(NULL, command, NULL, NULL, TRUE, 0, NULL, NULL, &si, &pi); + CloseHandle(stderr_write); + CloseHandle(stdout_write); + if(!bres) { + win_skip("cscript.exe is not available\n"); + CloseHandle(stderr_read); + CloseHandle(stdout_read); + return ~0u; + } + + WaitForSingleObject(pi.hProcess, INFINITE); + + bres = GetExitCodeProcess(pi.hProcess, &exit_code); + ok(bres, "GetExitCodeProcess failed: %lu\n", GetLastError()); + + memset(stdout_buf, 0, stdout_buf_size); + ReadFile(stdout_read, stdout_buf, stdout_buf_size - 1, &size, NULL); + stdout_buf[size] = 0; + + memset(stderr_buf, 0, stderr_buf_size); + ReadFile(stderr_read, stderr_buf, stderr_buf_size - 1, &size, NULL); + stderr_buf[size] = 0; + + CloseHandle(pi.hThread); + CloseHandle(pi.hProcess); + CloseHandle(stdout_read); CloseHandle(stderr_read); + return exit_code; +} + +static void run_cscript_unknown_option_test(void) +{ + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + + exit_code = run_cscript("//nologo /unknownoption", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + return; + ok(exit_code == 1, "exit_code = %lu, expected 1\n", exit_code); + /* On Windows, cscript reports a localized error on stdout. On Wine, + * unrecognized single-/ options are treated as filenames to allow Unix-style + * paths (/tmp/foo.vbs), so the error will be about the file instead. */ + ok(stdout_buf[0] != 0, "expected error message on stdout\n"); + ok(stderr_buf[0] == 0, "expected no stderr output, got: %s\n", stderr_buf); +} + +static void run_cscript_no_file_test(void) +{ + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + + exit_code = run_cscript("//nologo", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + return; + ok(exit_code == 1, "exit_code = %lu, expected 1\n", exit_code); + ok(stdout_buf[0] != 0, "expected error message on stdout\n"); +} + +static void run_cscript_usage_test(void) +{ + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + + exit_code = run_cscript("", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + return; + ok(exit_code == 0, "exit_code = %lu, expected 0\n", exit_code); + ok(stdout_buf[0] != 0, "expected usage output on stdout\n"); +} + +static BOOL create_temp_vbs(const char *file_name, const char *data) +{ + HANDLE file; + DWORD size; + BOOL bres; + + file = CreateFileA(file_name, GENERIC_WRITE, 0, NULL, CREATE_ALWAYS, + FILE_ATTRIBUTE_NORMAL, NULL); + ok(file != INVALID_HANDLE_VALUE, "CreateFile failed: %lu\n", GetLastError()); + if(file == INVALID_HANDLE_VALUE) + return FALSE; + + bres = WriteFile(file, data, strlen(data), &size, NULL); + CloseHandle(file); + ok(bres, "Could not write to file: %lu\n", GetLastError()); + return bres; +} + +static void run_cscript_logo_test(void) +{ + static const char empty_script[] = "\n"; + char file_name[] = "test_logo.vbs"; + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + size_t len; + + if(!create_temp_vbs(file_name, empty_script)) + goto cleanup; + + /* Without //nologo: banner should be shown on stdout */ + exit_code = run_cscript(file_name, stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + goto cleanup; + ok(exit_code == 0, "exit_code = %lu, expected 0\n", exit_code); + ok(stdout_buf[0] != 0, "expected banner in stdout\n"); + + /* Banner should be followed by a blank line */ + ok(strstr(stdout_buf, "\r\n\r\n") != NULL, + "expected blank line after banner, got: %s\n", stdout_buf); + + /* Output should end with a blank line (\r\n\r\n at the end) */ + len = strlen(stdout_buf); + ok(len >= 4 && !memcmp(stdout_buf + len - 4, "\r\n\r\n", 4), + "expected trailing blank line, got: %s\n", stdout_buf); + +cleanup: + DeleteFileA(file_name); +} + +static void run_cscript_nologo_test(void) +{ + static const char empty_script[] = "\n"; + char file_name[] = "test_nologo.vbs"; + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + + if(!create_temp_vbs(file_name, empty_script)) + goto cleanup; + + /* With //nologo: no banner, no output for empty script */ + exit_code = run_cscript("//nologo test_nologo.vbs", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + goto cleanup; + ok(exit_code == 0, "exit_code = %lu, expected 0\n", exit_code); + ok(stdout_buf[0] == 0, "expected no stdout output with //nologo, got: %s\n", stdout_buf); + +cleanup: + DeleteFileA(file_name); +} + +static void run_cscript_host_option_after_filename_test(void) +{ + /* Script that echoes its argument count */ + static const char script_data[] = "WScript.Echo WScript.Arguments.Count\n"; + char file_name[] = "test_args.vbs"; + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + + if(!create_temp_vbs(file_name, script_data)) + goto cleanup; + + /* //nologo after filename should be consumed as host option, not passed as arg */ + exit_code = run_cscript("test_args.vbs //nologo", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + goto cleanup; + ok(exit_code == 0, "exit_code = %lu, expected 0\n", exit_code); + /* No banner (//nologo consumed) and arg count = 0 */ + ok(stdout_buf[0] == '0', + "expected '0' as first output (no banner, //nologo consumed as host option), got: %s\n", stdout_buf); + + /* //nologo between script args should be consumed */ + exit_code = run_cscript("//nologo test_args.vbs arg1 //b arg2", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + goto cleanup; + ok(exit_code == 0, "exit_code = %lu, expected 0\n", exit_code); + /* //b suppresses output AND is consumed: arg count should be 2 (arg1 + arg2) */ + ok(stdout_buf[0] == 0 || strstr(stdout_buf, "2") != NULL, + "//b between args should be consumed, got: %s\n", stdout_buf); cleanup: DeleteFileA(file_name); } +static void run_cscript_nonexistent_file_test(void) +{ + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + + /* Nonexistent file with //nologo: error on stdout */ + exit_code = run_cscript("//nologo nonexistent_file.vbs", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + return; + ok(exit_code == 1, "exit_code = %lu, expected 1\n", exit_code); + ok(strstr(stdout_buf, "nonexistent_file") != NULL, + "expected filename in error on stdout, got: %s\n", stdout_buf); + + /* Nonexistent file without //nologo: banner + error on stdout */ + exit_code = run_cscript("nonexistent_file.vbs", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + return; + ok(exit_code == 1, "exit_code = %lu, expected 1\n", exit_code); + ok(strstr(stdout_buf, "nonexistent_file") != NULL, + "expected filename in error on stdout, got: %s\n", stdout_buf); +} + +static void run_cscript_error_on_stdout_test(void) +{ + char stdout_buf[4096], stderr_buf[4096]; + DWORD exit_code; + + /* Unknown option without //nologo: banner + error on stdout */ + exit_code = run_cscript("/unknownoption", stdout_buf, sizeof(stdout_buf), + stderr_buf, sizeof(stderr_buf)); + if(exit_code == ~0u) + return; + ok(exit_code == 1, "exit_code = %lu, expected 1\n", exit_code); + ok(stdout_buf[0] != 0, "expected banner and error on stdout\n"); + /* Error message should be on stdout, not stderr */ + ok(stderr_buf[0] == 0, "expected no stderr output, got: %s\n", stderr_buf); +} + START_TEST(run) { char **argv; @@ -551,6 +860,14 @@ START_TEST(run) "winetest.ok(false, 'not quit?');\n", 3); run_cscript_error_test(); + run_cscript_unknown_option_test(); + run_cscript_no_file_test(); + run_cscript_usage_test(); + run_cscript_logo_test(); + run_cscript_nologo_test(); + run_cscript_host_option_after_filename_test(); + run_cscript_nonexistent_file_test(); + run_cscript_error_on_stdout_test(); } init_registry(FALSE); -- GitLab https://gitlab.winehq.org/wine/wine/-/merge_requests/10518
On Tue Mar 31 18:44:00 2026 +0000, Francis De Brabandere wrote:
changed this line in [version 7 of the diff](/wine/wine/-/merge_requests/10518/diffs?diff_id=256161&start_sha=28831cbb314236c257b0f17c8ed9b4255a92ec41#560b24cabeb503b6b04e7d6d388bc785ee637414_664_661) good catch, now just checking if there is output or non-locale related parts (I think)
-- https://gitlab.winehq.org/wine/wine/-/merge_requests/10518#note_134559
participants (2)
-
Francis De Brabandere -
Francis De Brabandere (＠francisdb)