This repository has been archived by the owner on Feb 18, 2023. It is now read-only.
aubuf: refactor aubuf_auframe test #70
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sreimers
changed the title
|
should we merge this one to main ? |
|
Yes I will resolve the conflicts and merge together with baresip/rem#55 |
sreimers
force-pushed
the
aubuf_check
branch
from
7aa0fec to
c7e9698
Compare
cspiel1
added a commit
that referenced
this pull request
Oct 1, 2022
* async: add new tests (#107) * cmake: add path to windows build * mock: fix warning * dns/client: add async getaddrinfo usage (#112) * dns: refactor IP checks and add system test * use dnsc_getaddrinfo * fmt: workaround for windows compiler * sys: fix unlink and WIN32 * test: cast values to fix WIN32 warnings * trace: fix unlink and WIN32 * main: only run regular tests if GETOPT is missing * test: various fixes for WIN32 * async: fix multithreading tests (#115) * main: if GETOPT is missing run regular tests * cmake: add LINKLIBS variable * test cmake on Windows (#114) * test cmake on Windows * You have an error in your yaml syntax on line 14 * fix yaml syntax error * try to find re * try to find rem * mock: fix win32 warning * fix unlink warnings * fix warnings * fix write * fix size_t * write needs unsigned int * link to win32 libs * add qwave * execute retest.exe in build/debug * set verbose * set verbose * disable all * update flags * ci: fix compiler list and choco * LINKLIBS and flags * test: fix c11 err handling * cmake: use re config (#118) * base64: Encoding/Decoding with URL and Filename Safe Alphabet (#113) * cmake: bump minimum to 3.10 (#119) * Don't fail on compilation when there no OpenSSL on target platform (#120) * trice: fix win32 with explicit error tests (#121) * http: test http/https requests with large body * http: validate body of http requests for get method * http: add test for http conn requests * dns: test dnsc_getaddrinfo_enabled * rtp: Update tests for the changed rtp_send signature. This updates RTP tests for compatibility with this PR: baresip/re#418 * crc32: add re wrapper * sipreg: use TEST_ERR and remove static port test Tests should run without interfere with each other. Looks like on Github Actions runners share the same port range sometimes? So for tests we have to use dynamic ports. * cmake: no need to link to libz * sa: init err * cmake: add optional zlib linking * cmake/FindREM: add rem-static target name * sys: remove old sys div test (#130) * aubuf: refactor aubuf_auframe test (#70) * rtcp: use udp_send() (#132) * rtcp: use udp_send() * rtcp: add workaround for OOM test Co-authored-by: Sebastian Reimers <[email protected]> Co-authored-by: Alfred E. Heggestad <[email protected]> Co-authored-by: Dmitry Ilyin <[email protected]> Co-authored-by: Franz Auernigg <[email protected]> Co-authored-by: Andrey Semashev <[email protected]>
alfredh
added a commit
that referenced
this pull request
Nov 1, 2022
* async: add new tests (#107) * cmake: add path to windows build * mock: fix warning * dns/client: add async getaddrinfo usage (#112) * dns: refactor IP checks and add system test * use dnsc_getaddrinfo * fmt: workaround for windows compiler * sys: fix unlink and WIN32 * test: cast values to fix WIN32 warnings * trace: fix unlink and WIN32 * main: only run regular tests if GETOPT is missing * test: various fixes for WIN32 * async: fix multithreading tests (#115) * main: if GETOPT is missing run regular tests * cmake: add LINKLIBS variable * test cmake on Windows (#114) * test cmake on Windows * You have an error in your yaml syntax on line 14 * fix yaml syntax error * try to find re * try to find rem * mock: fix win32 warning * fix unlink warnings * fix warnings * fix write * fix size_t * write needs unsigned int * link to win32 libs * add qwave * execute retest.exe in build/debug * set verbose * set verbose * disable all * update flags * ci: fix compiler list and choco * LINKLIBS and flags * test: fix c11 err handling * cmake: use re config (#118) * base64: Encoding/Decoding with URL and Filename Safe Alphabet (#113) * cmake: bump minimum to 3.10 (#119) * Don't fail on compilation when there no OpenSSL on target platform (#120) * trice: fix win32 with explicit error tests (#121) * http: test http/https requests with large body * http: validate body of http requests for get method * http: add test for http conn requests * dns: test dnsc_getaddrinfo_enabled * rtp: Update tests for the changed rtp_send signature. This updates RTP tests for compatibility with this PR: baresip/re#418 * crc32: add re wrapper * sipreg: use TEST_ERR and remove static port test Tests should run without interfere with each other. Looks like on Github Actions runners share the same port range sometimes? So for tests we have to use dynamic ports. * cmake: no need to link to libz * sa: init err * cmake: add optional zlib linking * cmake/FindREM: add rem-static target name * sys: remove old sys div test (#130) * aubuf: refactor aubuf_auframe test (#70) * rtcp: use udp_send() (#132) * rtcp: use udp_send() * rtcp: add workaround for OOM test * leb128: remove debug * http: fix http content length format (#136) * Support run specific integration test and add http with local dnssrv (#135) * support run specific integration test * http: add integration test for http using local dns server * odict: check return value of compare() * odict: use correct union field (boolean) (#137) Fixes a bug on Alpine/S390x which is 64-bit Big Endian architecture * test: use fs_stdio_restore() (#138) * oom: align testcase names * align testcase names * tmr: increase upper bound * remove leb128 printf * sys: add todo * net: add ipv6_supported() * sys: unique filename in test_sys_fs_fopen (#142) * net: remove logging * Enable valgrind for Linux build (#140) * integration: make IPv6 tests optional * bfcp: check for handler errors after re_main * fmt: add test_fmt_str_itoa (#143) * git: ignore cmake generated files * workflows: add run-on-arch action (#134) * test_multithread: increase timeout to 15 seconds * tls: remove some warnings (#144) * ci/run-on-arch: add prepare (needed for dependency) (#145) * ci: use actions/checkout@v3 (#148) * list: add test_list_flush (#151) * list: fix flushl global (#152) Co-authored-by: Sebastian Reimers <[email protected]> Co-authored-by: Dmitry Ilyin <[email protected]> Co-authored-by: Franz Auernigg <[email protected]> Co-authored-by: Andrey Semashev <[email protected]> Co-authored-by: Franz <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
While working on aubuf enhancements (get rid of malloc calls), I found a bug that is not catched by aubuf tests.