[freebsd/kyua] f7ac5e: Add more information to failed results

0 views
Skip to first unread message

Enji Cooper

unread,
Dec 1, 2024, 12:27:08 PM12/1/24
to kyua...@googlegroups.com
Branch: refs/heads/master
Home: https://github.com/freebsd/kyua
Commit: f7ac5e0f3fa806cb417277f63e738a9e862c64b0
https://github.com/freebsd/kyua/commit/f7ac5e0f3fa806cb417277f63e738a9e862c64b0
Author: Enji Cooper <ng...@FreeBSD.org>
Date: 2024-12-01 (Sun, 01 Dec 2024)

Changed paths:
M engine/atf_result.cpp
M engine/atf_result_test.cpp
M engine/atf_test.cpp
M integration/cmd_test_test.sh
M store/write_transaction_test.cpp

Log Message:
-----------
Add more information to failed results

This change adds more information to "premature failures" or failures
where no output was provided on the command-line. This is being done to
help make the reasons for why a failure occurred clearer to the
end-user.

This also fixes the fact that several ATF-related tests were failing with
"empty output" (lacking exit code data), or not failing with the
appropriate (expected) "timeout" messages.

While here, address the fact that newer versions of sqlite3 fail when
executing statements with `sqlite::api_error` instead of `store::error`.

This addresses part of #237 and makes the kyua tests pass cleanly
again on FreeBSD.

Signed-off-by: Enji Cooper <ng...@FreeBSD.org>


Commit: 688ac533587501f18ad6acabb0fe2b337239609b
https://github.com/freebsd/kyua/commit/688ac533587501f18ad6acabb0fe2b337239609b
Author: Enji Cooper <ng...@FreeBSD.org>
Date: 2024-12-01 (Sun, 01 Dec 2024)

Changed paths:
M engine/atf_result.cpp
M engine/atf_result_test.cpp
M engine/atf_test.cpp
M integration/cmd_test_test.sh
M store/write_transaction_test.cpp

Log Message:
-----------
Merge pull request #238 from ngie-eign/improve-failure-diagnostics

Add more information to failed results


Compare: https://github.com/freebsd/kyua/compare/ff8e9703c58d...688ac5335875

To unsubscribe from these emails, change your notification settings at https://github.com/freebsd/kyua/settings/notifications
Reply all
Reply to author
Forward
0 new messages