Skip to content

Restyle test: add tests for the ToxAV multithreading modes #1858

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 11 commits into from
Closed

Restyle test: add tests for the ToxAV multithreading modes #1858

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
1b73447
test: add test for multithreaded ToxAV
sudden6 Jan 4, 2022
84945fe
add C++ tests to bazel
iphydf Jan 4, 2022
a3a1db7
Fix Windows build failing due to missing std::mutex
nurupo Jan 5, 2022
d77c6ce
Fix build error due to compiler being old
nurupo Jan 5, 2022
c3b8f38
Fix Dockerfile having no workdir set
nurupo Jan 8, 2022
ac5a87f
fix: prevent unlocked access to av->calls
sudden6 Jan 8, 2022
4cf603c
fix: missing lock
sudden6 Jan 8, 2022
b33bfe1
squash, cleanup CMake changes
sudden6 Jan 9, 2022
999ae39
squash, avoid copy initialisation
sudden6 Jan 9, 2022
fdef6db
Restyled by astyle
restyled-commits Jan 9, 2022
38aaaa3
Restyled by clang-format
restyled-commits Jan 9, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .travis/cmake-windows.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ travis_script() {
add_flag -gdwarf-2

# Fix invalid register for .seh_savexmm error
add_flag -fno-asynchronous-unwind-tables
add_c_flag -fno-asynchronous-unwind-tables

docker run \
-e ALLOW_TEST_FAILURE=true \
Expand Down
13 changes: 11 additions & 2 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -393,10 +393,10 @@ endif()

option(AUTOTEST "Enable autotests (mainly for CI)" OFF)

function(auto_test target)
function(auto_test_common target extension)
if(AUTOTEST AND NOT (MSVC AND ARGV1 STREQUAL "MSVC_DONT_BUILD"))
add_executable(auto_${target}_test ${CPUFEATURES}
auto_tests/${target}_test.c)
auto_tests/${target}_test.${extension})
target_link_modules(auto_${target}_test toxcore misc_tools)
if(NOT ARGV1 STREQUAL "DONT_RUN")
add_test(NAME ${target} COMMAND ${CROSSCOMPILING_EMULATOR} auto_${target}_test)
Expand All @@ -406,6 +406,14 @@ function(auto_test target)
endif()
endfunction()

function(auto_test target)
auto_test_common(${target} "c" ${ARGN})
endfunction()

function(auto_test_cc target)
auto_test_common(${target} "cc" ${ARGN})
endfunction()

auto_test(TCP)
auto_test(conference)
auto_test(conference_double_invite)
Expand Down Expand Up @@ -454,6 +462,7 @@ if(BUILD_TOXAV)
auto_test(conference_av)
auto_test(toxav_basic)
auto_test(toxav_many)
auto_test_cc(toxav_mt)
endif()

################################################################################
Expand Down
4 changes: 2 additions & 2 deletions auto_tests/BUILD.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ flaky_tests = {
}

[cc_test(
name = src[:-2],
name = src[:src.rindex(".")-len(src)],
size = "small",
srcs = [src],
args = ["$(location %s)" % src],
Expand All @@ -38,4 +38,4 @@ flaky_tests = {
"//c-toxcore/toxcore:DHT_srcs",
"//c-toxcore/toxencryptsave",
],
) for src in glob(["*_test.c"])]
) for src in glob(["*_test.c*"])]
Loading