Skip to content

Commit fdc58db

Browse files
committed
More fixes
1 parent 673e5b0 commit fdc58db

File tree

5 files changed

+721
-13
lines changed

5 files changed

+721
-13
lines changed

cmake/yup_generate_java_header.cmake

Lines changed: 19 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,9 +36,25 @@ file (READ "${INPUT_FILE}" hex_content HEX)
3636
get_filename_component (INPUT_FILE_NAME "${INPUT_FILE}" NAME)
3737

3838
# Convert hex string to C++ array format
39-
string (REGEX MATCHALL "([A-Fa-f0-9][A-Fa-f0-9])" separated_hex ${hex_content})
40-
list (JOIN separated_hex ", 0x" formatted_hex)
41-
string (PREPEND formatted_hex "0x")
39+
set (count 0)
40+
set (wrapped_hex "")
41+
string (REGEX MATCHALL "([A-Fa-f0-9][A-Fa-f0-9])" separated_hex "${hex_content}")
42+
foreach (byte IN LISTS separated_hex)
43+
if (count EQUAL 0)
44+
set (current_line "0x${byte}")
45+
else()
46+
set (current_line "${current_line},0x${byte}")
47+
endif()
48+
math (EXPR count "${count} + 1")
49+
if (count EQUAL 60)
50+
list (APPEND wrapped_hex "${current_line}")
51+
set (count 0)
52+
endif()
53+
endforeach()
54+
if (count GREATER 0)
55+
list(APPEND wrapped_hex "${current_line}")
56+
endif()
57+
string (JOIN "\n" formatted_hex ${wrapped_hex})
4258

4359
# Generate timestamp
4460
string (TIMESTAMP current_time "%Y-%m-%d %H:%M:%S UTC" UTC)

cmake/yup_python.cmake

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,6 @@ function (yup_prepare_python_stdlib target_name python_tools_path output_variabl
6464
"${Python_EXECUTABLE}" "${python_tools_path}/ArchivePythonStdlib.py"
6565
-r "${python_root_path}" -o "${CMAKE_CURRENT_BINARY_DIR}" -M "${Python_VERSION_MAJOR}" -m "${Python_VERSION_MINOR}"
6666
-x "\"${ignored_library_patterns}\""
67-
COMMAND_ECHO STDOUT
6867
COMMAND_ERROR_IS_FATAL ANY)
6968

7069
set (${output_variable} ${python_standard_library} PARENT_SCOPE)

tests/yup_core/yup_File.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -904,7 +904,7 @@ TEST_F (FileTests, Version)
904904
// Version might be empty for test executables
905905
}
906906

907-
TEST_F (FileTests, StartAsProcess)
907+
TEST_F (FileTests, DISABLED_StartAsProcess)
908908
{
909909
// Limited testing - we don't want to actually launch processes in unit tests
910910
tempFolder.createDirectory();
@@ -913,6 +913,7 @@ TEST_F (FileTests, StartAsProcess)
913913

914914
// Just verify the method exists and doesn't crash
915915
// Actual process launching should be tested manually
916+
textFile.startAsProcess();
916917
}
917918

918919
TEST_F (FileTests, RecursiveReadOnly)
@@ -1107,7 +1108,7 @@ TEST_F (FileTests, FileOutputStreamFlush)
11071108
}
11081109
#endif
11091110

1110-
TEST_F (FileTests, RevealToUser)
1111+
TEST_F (FileTests, DISABLED_RevealToUser)
11111112
{
11121113
// Test File::revealToUser() - this method shows the file in the OS file browser
11131114
tempFolder.createDirectory();

tests/yup_core/yup_Process.cpp

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ class ProcessTests : public ::testing::Test
4848
File testFile;
4949
};
5050

51-
TEST_F (ProcessTests, OpenDocumentWithFileName)
51+
TEST_F (ProcessTests, DISABLED_OpenDocumentWithFileName)
5252
{
5353
// Test Process::openDocument() with a file name
5454
// This attempts to open the file with the default application
@@ -64,7 +64,7 @@ TEST_F (ProcessTests, OpenDocumentWithFileName)
6464
SUCCEED();
6565
}
6666

67-
TEST_F (ProcessTests, OpenDocumentWithUrl)
67+
TEST_F (ProcessTests, DISABLED_OpenDocumentWithUrl)
6868
{
6969
// Test opening a URL (this should be safer than opening a file)
7070
// Most systems have a default browser
@@ -77,15 +77,15 @@ TEST_F (ProcessTests, OpenDocumentWithUrl)
7777
SUCCEED();
7878
}
7979

80-
TEST_F (ProcessTests, OpenDocumentWithParameters)
80+
TEST_F (ProcessTests, DISABLED_OpenDocumentWithParameters)
8181
{
8282
// Test Process::openDocument() with parameters
8383
[[maybe_unused]] bool result = Process::openDocument (testFile.getFullPathName(), "--test-param");
8484

8585
SUCCEED();
8686
}
8787

88-
TEST_F (ProcessTests, OpenDocumentWithEnvironment)
88+
TEST_F (ProcessTests, DISABLED_OpenDocumentWithEnvironment)
8989
{
9090
// Test Process::openDocument() with custom environment variables
9191
StringPairArray environment;
@@ -97,7 +97,7 @@ TEST_F (ProcessTests, OpenDocumentWithEnvironment)
9797
SUCCEED();
9898
}
9999

100-
TEST_F (ProcessTests, OpenDocumentWithEmptyPath)
100+
TEST_F (ProcessTests, DISABLED_OpenDocumentWithEmptyPath)
101101
{
102102
// Test with empty path (should fail gracefully)
103103
[[maybe_unused]] bool result = Process::openDocument ("", "");
@@ -106,7 +106,7 @@ TEST_F (ProcessTests, OpenDocumentWithEmptyPath)
106106
SUCCEED();
107107
}
108108

109-
TEST_F (ProcessTests, OpenDocumentWithNonExistentFile)
109+
TEST_F (ProcessTests, DISABLED_OpenDocumentWithNonExistentFile)
110110
{
111111
// Test with a file that doesn't exist
112112
File nonExistent = File::getSpecialLocation (File::tempDirectory)
@@ -119,7 +119,7 @@ TEST_F (ProcessTests, OpenDocumentWithNonExistentFile)
119119
SUCCEED();
120120
}
121121

122-
TEST_F (ProcessTests, OpenDocumentWithSpecialCharacters)
122+
TEST_F (ProcessTests, DISABLED_OpenDocumentWithSpecialCharacters)
123123
{
124124
// Create a file with special characters in the name
125125
File specialFile = testFile.getParentDirectory().getChildFile ("test file with spaces & special.txt");

0 commit comments

Comments
 (0)