From 34ff5208d426d61f9911caf237bb5deb884588df Mon Sep 17 00:00:00 2001 From: Tim Burke Date: Sat, 25 Feb 2017 14:33:44 +0000 Subject: [PATCH] Make test format strings consistent with test function names * When we specify a particular backend, we don't need to include it in the display string; we'll already include it in the output. * test_destroy_backend_invalid_args shouldn't be reported as test_create_backend_invalid_args. * Consistently include the leading "test_" in the display string. Change-Id: Ibd72ba4ae609ad77e8808aa1594b0adb62e34ef0 --- test/liberasurecode_test.c | 142 ++++++++++++++++++------------------- 1 file changed, 71 insertions(+), 71 deletions(-) diff --git a/test/liberasurecode_test.c b/test/liberasurecode_test.c index 68c1c13..1e75eb1 100644 --- a/test/liberasurecode_test.c +++ b/test/liberasurecode_test.c @@ -1791,7 +1791,7 @@ struct testcase testcases[] = { test_create_backend_invalid_args, EC_BACKENDS_MAX, CHKSUM_TYPES_MAX, .skip = false}, - {"test_create_backend_invalid_args", + {"test_destroy_backend_invalid_args", test_destroy_backend_invalid_args, EC_BACKENDS_MAX, CHKSUM_TYPES_MAX, .skip = false}, @@ -1840,11 +1840,11 @@ struct testcase testcases[] = { EC_BACKENDS_MAX, 0, .skip = false}, // NULL backend test - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_NULL, CHKSUM_NONE, .skip = false}, - {"simple_encode_null", + {"test_simple_encode_null", test_simple_encode_decode, EC_BACKEND_NULL, CHKSUM_NONE, .skip = false}, @@ -1853,47 +1853,47 @@ struct testcase testcases[] = { EC_BACKEND_NULL, CHKSUM_NONE, .skip = false}, // Flat XOR backend tests - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"simple_encode_flat_xor_hd", + {"test_simple_encode_decode", test_simple_encode_decode, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_data_flat_xor_hd", + {"test_decode_with_missing_data", test_decode_with_missing_data, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_parity_flat_xor_hd", + {"test_decode_with_missing_parity", test_decode_with_missing_parity, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_flat_xor_hd", + {"test_decode_with_missing_multi_data", test_decode_with_missing_multi_data, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_parity_flat_xor_hd", + {"test_decode_with_missing_multi_parity", test_decode_with_missing_multi_parity, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"test_decode_with_missing_multi_data_parity_flat_xor_hd", + {"test_decode_with_missing_multi_data_parity", test_decode_with_missing_multi_data_parity, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"simple_reconstruct_flat_xor_hd", + {"test_simple_reconstruct", test_simple_reconstruct, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"test_fragments_needed_flat_xor_hd", + {"test_fragments_needed", test_fragments_needed, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_flat_xor_hd", + {"test_get_fragment_metadata", test_get_fragment_metadata, EC_BACKEND_FLAT_XOR_HD, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_flat_xor_hd_crc32", + {"test_get_fragment_metadata_crc32", test_get_fragment_metadata, EC_BACKEND_FLAT_XOR_HD, CHKSUM_CRC32, .skip = false}, @@ -1922,43 +1922,43 @@ struct testcase testcases[] = { EC_BACKEND_FLAT_XOR_HD, CHKSUM_CRC32, .skip = false}, // Jerasure RS Vand backend tests - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"simple_encode_jerasure_rs_vand", + {"test_simple_encode_decode", test_simple_encode_decode, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_data_jerasure_rs_vand", + {"test_decode_with_missing_data", test_decode_with_missing_data, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_jerasure_rs_vand", + {"test_decode_with_missing_multi_data", test_decode_with_missing_multi_data, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_parity_jerasure_rs_vand", + {"test_decode_with_missing_multi_parity", test_decode_with_missing_multi_parity, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_decode_with_missing_multi_data_parity_jerasure_rs_vand", + {"test_decode_with_missing_multi_data_parity", test_decode_with_missing_multi_data_parity, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"simple_reconstruct_jerasure_rs_vand", + {"test_simple_reconstruct", test_simple_reconstruct, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_fragments_needed_jerasure_rs_vand", + {"test_fragments_needed", test_fragments_needed, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_jerasure_rs_vand", + {"test_get_fragment_metadata", test_get_fragment_metadata, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_jerasure_rs_vand_crc32", + {"test_get_fragment_metadata_crc32", test_get_fragment_metadata, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_CRC32, .skip = false}, @@ -1991,43 +1991,43 @@ struct testcase testcases[] = { EC_BACKENDS_MAX, 0, // NB: this is hardcoded to use jerasure RS vand .skip = false}, // Jerasure RS Cauchy backend tests - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"simple_encode_jerasure_rs_cauchy", + {"test_simple_encode_decode", test_simple_encode_decode, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_data_jerasure_rs_cauchy", + {"test_decode_with_missing_data", test_decode_with_missing_data, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_jerasure_rs_cauchy", + {"test_decode_with_missing_multi_data", test_decode_with_missing_multi_data, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_parity_jerasure_rs_cauchy", + {"test_decode_with_missing_multi_parity", test_decode_with_missing_multi_parity, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_parity_jerasure_rs_cauchy", + {"test_decode_with_missing_multi_data_parity", test_decode_with_missing_multi_data_parity, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"simple_reconstruct_jerasure_rs_cauchy", + {"test_simple_reconstruct", test_simple_reconstruct, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"test_fragments_needed_jerasure_rs_cauchy", + {"test_fragments_needed", test_fragments_needed, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_jerasure_rs_cauchy", + {"test_get_fragment_metadata", test_get_fragment_metadata, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_jerasure_rs_cauchy_crc32", + {"test_get_fragment_metadata_crc32", test_get_fragment_metadata, EC_BACKEND_JERASURE_RS_CAUCHY, CHKSUM_CRC32, .skip = false}, @@ -2060,39 +2060,39 @@ struct testcase testcases[] = { EC_BACKENDS_MAX, 0, .skip = false}, // ISA-L rs_vand tests - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"simple_encode_isa_l", + {"test_simple_encode_decode", test_simple_encode_decode, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_data_isa_l", + {"test_decode_with_missing_data", test_decode_with_missing_data, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_isa_l", + {"test_decode_with_missing_multi_data", test_decode_with_missing_multi_data, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_parity_isa_l", + {"test_decode_with_missing_multi_parity", test_decode_with_missing_multi_parity, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_decode_with_missing_multi_data_parity_isa_l", + {"test_decode_with_missing_multi_data_parity", test_decode_with_missing_multi_data_parity, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"simple_reconstruct_isa_l", + {"test_simple_reconstruct", test_simple_reconstruct, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_fragments_needed_isa_l", + {"test_fragments_needed", test_fragments_needed, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_isa_l", + {"test_get_fragment_metadata", test_get_fragment_metadata, EC_BACKEND_ISA_L_RS_VAND, CHKSUM_NONE, .skip = false}, @@ -2125,39 +2125,39 @@ struct testcase testcases[] = { EC_BACKENDS_MAX, 0, // note this test is using ISA-L in hard coded .skip = false}, // ISA-L rs cauchy tests - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"simple_encode_isa_l", + {"test_simple_encode_decode", test_simple_encode_decode, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_data_isa_l", + {"test_decode_with_missing_data", test_decode_with_missing_data, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_isa_l", + {"test_decode_with_missing_multi_data", test_decode_with_missing_multi_data, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_parity_isa_l", + {"test_decode_with_missing_multi_parity", test_decode_with_missing_multi_parity, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"test_decode_with_missing_multi_data_parity_isa_l", + {"test_decode_with_missing_multi_data_parity", test_decode_with_missing_multi_data_parity, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"simple_reconstruct_isa_l", + {"test_simple_reconstruct", test_simple_reconstruct, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"test_fragments_needed_isa_l", + {"test_fragments_needed", test_fragments_needed, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_isa_l", + {"test_get_fragment_metadata", test_get_fragment_metadata, EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_NONE, .skip = false}, @@ -2186,39 +2186,39 @@ struct testcase testcases[] = { EC_BACKEND_ISA_L_RS_CAUCHY, CHKSUM_CRC32, .skip = false}, // shss tests - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"simple_encode_shss", + {"test_simple_encode_decode", test_simple_encode_decode, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_data_shss", + {"test_decode_with_missing_data", test_decode_with_missing_data, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_shss", + {"test_decode_with_missing_multi_data", test_decode_with_missing_multi_data, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_parity_shss", + {"test_decode_with_missing_multi_parity", test_decode_with_missing_multi_parity, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"test_decode_with_missing_multi_data_parity_shss", + {"test_decode_with_missing_multi_data_parity", test_decode_with_missing_multi_data_parity, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"simple_reconstruct_shss", + {"test_simple_reconstruct", test_simple_reconstruct, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"test_fragments_needed_shss", + {"test_fragments_needed", test_fragments_needed, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_shss", + {"test_get_fragment_metadata", test_get_fragment_metadata, EC_BACKEND_SHSS, CHKSUM_NONE, .skip = false}, @@ -2247,43 +2247,43 @@ struct testcase testcases[] = { EC_BACKEND_SHSS, CHKSUM_CRC32, .skip = false}, // Internal RS Vand backend tests - {"create_and_destroy_backend", + {"test_create_and_destroy_backend", test_create_and_destroy_backend, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"simple_encode_liberasurecode_rs_vand", + {"test_simple_encode_decode", test_simple_encode_decode, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_data_liberasurecode_rs_vand", + {"test_decode_with_missing_data", test_decode_with_missing_data, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_data_liberasurecode_rs_vand", + {"test_decode_with_missing_multi_data", test_decode_with_missing_multi_data, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"decode_with_missing_multi_parity_liberasurecode_rs_vand", + {"test_decode_with_missing_multi_parity", test_decode_with_missing_multi_parity, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_decode_with_missing_multi_data_parity_liberasurecode_rs_vand", + {"test_decode_with_missing_multi_data_parity", test_decode_with_missing_multi_data_parity, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"simple_reconstruct_liberasurecode_rs_vand", + {"test_simple_reconstruct", test_simple_reconstruct, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_fragments_needed_liberasurecode_rs_vand", + {"test_fragments_needed", test_fragments_needed, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_liberasurecode_rs_vand", + {"test_get_fragment_metadata", test_get_fragment_metadata, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_NONE, .skip = false}, - {"test_get_fragment_metadata_liberasurecode_rs_vand_crc32", + {"test_get_fragment_metadata_crc32", test_get_fragment_metadata, EC_BACKEND_LIBERASURECODE_RS_VAND, CHKSUM_CRC32, .skip = false},