Skip to content
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

Fix errors reported by KWStyle in tests folder #117

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+396 −176
Open

Fix errors reported by KWStyle in tests folder #117

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
29 changes: 29 additions & 0 deletions KWStyle.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
<?xml version="1.0" encoding="utf-8"?>
<Description>
<!--
Copying and distribution of this file, with or without modification,
are permitted in any medium without royalty provided this notice is
preserved. This file is offered as-is, without any warranty.
Names of contributors must not be used to endorse or promote products
derived from this file without specific prior written permission.
-->

<!-- Text -->
<LineLength>169</LineLength> <!-- max -->
<EmptyLines>2</EmptyLines> <!-- max -->
<EndOfFileNewLine>true</EndOfFileNewLine>

<!-- C/C++ -->
<VariablePerLine>3</VariablePerLine> <!-- max -->
<StatementPerLine>1</StatementPerLine> <!-- max -->
<Functions>
<regex>[a-z_]</regex>
<length>439</length>
</Functions> <!-- length = number of lines -->
<Comments>
<begin>/*</begin>
<middle> * </middle>
<end> */</end>
<checkMissingComment>false</checkMissingComment>
</Comments>
</Description>
10 changes: 9 additions & 1 deletion tests/Makefile.am
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,15 @@ noinst_PROGRAMS = \
check_mem_leaks \
ex_output

EXTRA_DIST = test_output.sh test_check_nofork.sh test_check_nofork_teardown.sh test_log_output.sh test_vars.in test_xml_output.sh test_tap_output.sh test_mem_leaks.sh test_output_strings
EXTRA_DIST = \
test_check_nofork.sh test_check_nofork_teardown.sh \
test_log_output.sh \
test_vars.in \
test_mem_leaks.sh \
test_output.sh \
test_output_strings \
test_tap_output.sh \
test_xml_output.sh

if NO_TIMEOUT_TESTS
check_check_CFLAGS = -DTIMEOUT_TESTS_ENABLED=0
Expand Down
1 change: 0 additions & 1 deletion tests/check_check_log.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -312,4 +312,3 @@ Suite *make_log_suite(void)

return s;
}

1 change: 0 additions & 1 deletion tests/check_check_log_internal.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -72,4 +72,3 @@ Suite *make_log_internal_suite(void)

return s;
}

506 changes: 338 additions & 168 deletions tests/check_check_master.c
View file
Open in desktop

Large diffs are not rendered by default.

24 changes: 20 additions & 4 deletions tests/check_check_sub.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2261,8 +2261,16 @@ END_TEST

START_TEST(test_ck_assert_mem_eq_exact)
{
const char *s = "\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02";
const char *t = "\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01";
const char *s =
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02";
const char *t =
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01";
record_test_name(tcase_name());
record_failure_line_num(__LINE__);
ck_assert_mem_eq(t, s, 64);
Expand All @@ -2271,8 +2279,16 @@ END_TEST

START_TEST(test_ck_assert_mem_eq_longer)
{
const char *s = "\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02";
const char *t = "\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01";
const char *s =
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x02";
const char *t =
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00"
"\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x01";
record_test_name(tcase_name());
record_failure_line_num(__LINE__);
ck_assert_mem_eq(t, s, 65);
Expand Down
1 change: 0 additions & 1 deletion tests/check_mem_leaks.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,4 +87,3 @@ int main ()
srunner_free(sr);
return (n == 0) ? EXIT_SUCCESS : EXIT_FAILURE;
}