refactor: Clean up test logging to make it easier to search (#5396)

This PR replaces the word `failed` with `failure` in any test names and renames some test files to fix MSVC warnings, so that it is easier to search through the test output to find tests that failed.
This commit is contained in:
Mayukha Vadari
2025-04-11 05:07:42 -04:00
committed by GitHub
parent 1c99ea23d1
commit cba512068b
9 changed files with 33 additions and 12 deletions

View File

@@ -65,6 +65,7 @@ public:
void
testValid()
{
testcase("Valid");
using namespace jtx;
Account const alice{"alice"};
Account const becky{"becky"};
@@ -162,6 +163,7 @@ public:
void
testErrors()
{
testcase("Errors");
using namespace jtx;
Account const alice{"alice"};
Account const becky{"becky"};
@@ -333,6 +335,8 @@ public:
void
testCredentials()
{
testcase("Credentials");
using namespace jtx;
const char credType[] = "abcde";
@@ -363,7 +367,7 @@ public:
{
testcase(
"deposit_authorized with credentials failed: empty array.");
"deposit_authorized with credentials failure: empty array.");
auto args = depositAuthArgs(alice, becky, "validated");
args[jss::credentials] = Json::arrayValue;
@@ -376,7 +380,7 @@ public:
{
testcase(
"deposit_authorized with credentials failed: not a string "
"deposit_authorized with credentials failure: not a string "
"credentials");
auto args = depositAuthArgs(alice, becky, "validated");
@@ -392,7 +396,7 @@ public:
{
testcase(
"deposit_authorized with credentials failed: not a hex string "
"deposit_authorized with credentials failure: not a hex string "
"credentials");
auto args = depositAuthArgs(alice, becky, "validated");
@@ -412,7 +416,7 @@ public:
{
testcase(
"deposit_authorized with credentials failed: not a credential "
"deposit_authorized with credentials failure: not a credential "
"index");
auto args = depositAuthArgs(