2024-03-28 13:27 UTC

View Issue Details Jump to Notes ]
IDProjectCategoryView StatusLast Update
0002736libnslog[All Projects] Generalpublic2020-05-27 08:12
Reporterbavier 
Assigned ToDaniel Silverstone 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformLinuxOSGuix SystemOS Versionguix 9448d0e
Summary0002736: libnslog tests fail with latest check version 0.13
DescriptionRunning tests from libnslog version 0.1.2 that were linked against most recent check version 0.13 produces the following failures:

Running suite(s): Test suite for libnslog
 libnslog: Basic tests
70%: Checks: 24, Failures: 7, Errors: 0
test/basictests.c:86:F:Simple log checks, no filters:test_nslog_trivial_corked_message:0: Captured message wasn't correct function name
test/basictests.c:112:F:Simple log checks, no filters:test_nslog_trivial_uncorked_message:0: Captured message wasn't correct function name
test/basictests.c:153:F:Simple log checks, no filters:test_nslog_two_corked_messages:0: Captured message wasn't correct function name
test/basictests.c:226:F:Simple filter checks:test_nslog_simple_filter_corked_message:0: Captured message wasn't correct function name
test/basictests.c:253:F:Simple filter checks:test_nslog_simple_filter_uncorked_message:0: Captured message wasn't correct function name
test/basictests.c:280:F:Simple filter checks:test_nslog_simple_filter_subcategory_message:0: Captured message wasn't correct function name
test/basictests.c:534:F:Trivial, varied, filter checks:test_nslog_filter_funcname:0: Captured message count was wrong

The failures in question attempt to check the function name logged by libnslog against the expected test case function name. In check 0.13 however, the implementation of the "START_TEST" macro was changed in such a way that the testcase functions have a "_fn" suffix. That is, the actual function name of the test should maybe be treated as an implementation detail.
Steps To Reproduce1. Build libnslog against check version 0.13.
2. Run test_testrunner "basic" tests
Additional InformationWorkaround in GNU Guix package is currently to build with the older check 0.12 package: https://git.savannah.gnu.org/cgit/guix.git/commit/?id=7357b3d7a52eb5db1674012c50d308d792741c48
TagsNo tags attached.
Attached Files

-Relationships
+Relationships

-Notes
Daniel Silverstone

~0002170

Daniel Silverstone (administrator)

Thanks, I've fixed the test suites to cope with both variants of check by using the `__func__` macro appropriately instead.
Vincent Sanders

~0002225

Vincent Sanders (administrator)

Thankyou for reporting this issue.
We believe this is fixed in the 3.10 release.
If this is not the case please feel free to reopen the issue with additional details.
+Notes

-Issue History
Date Modified Username Field Change
2020-01-23 06:49 bavier New Issue
2020-02-21 10:01 Daniel Silverstone Project NetSurf => libnslog
2020-02-21 10:01 Daniel Silverstone Category Development => General
2020-02-21 10:07 Daniel Silverstone Assigned To => Daniel Silverstone
2020-02-21 10:07 Daniel Silverstone Status new => assigned
2020-02-21 10:58 Daniel Silverstone Status assigned => resolved
2020-02-21 10:58 Daniel Silverstone Resolution open => fixed
2020-02-21 10:58 Daniel Silverstone Note Added: 0002170
2020-05-27 08:12 Vincent Sanders Status resolved => closed
2020-05-27 08:12 Vincent Sanders Note Added: 0002225
+Issue History