From 11244e449ffcfcc6fdbd37bc4320a94225161fef Mon Sep 17 00:00:00 2001 From: Tim Burke Date: Sat, 25 Feb 2017 20:07:46 +0000 Subject: [PATCH] Stop running test_simple_encode_decode_over32 repeatedly It uses hard-coded backend and arguments, so it should use EC_BACKEND_MAX and have a name that reflect how it's only targeted to Jerasure. This is similar to how we handled the targeted test case for ISA-L. Change-Id: I0a397930b2f49c0775290970c820bc70310a475e --- test/liberasurecode_test.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/test/liberasurecode_test.c b/test/liberasurecode_test.c index a80e324..4a1e7ba 100644 --- a/test/liberasurecode_test.c +++ b/test/liberasurecode_test.c @@ -1857,10 +1857,6 @@ struct testcase testcases[] = { test_verify_stripe_metadata_frag_idx_invalid, EC_BACKEND_FLAT_XOR_HD, CHKSUM_CRC32, .skip = false}, - {"test_simple_encode_decode_over32", - test_simple_encode_decode_over32, - EC_BACKEND_JERASURE_RS_VAND, CHKSUM_CRC32, - .skip = false}, // Jerasure RS Vand backend tests {"create_and_destroy_backend", test_create_and_destroy_backend, @@ -1926,6 +1922,10 @@ struct testcase testcases[] = { test_verify_stripe_metadata_frag_idx_invalid, EC_BACKEND_JERASURE_RS_VAND, CHKSUM_CRC32, .skip = false}, + {"test_simple_encode_decode_over32_jerasure_rs_vand", + test_simple_encode_decode_over32, + 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, @@ -2262,7 +2262,7 @@ int main(int argc, char **argv) continue; } if (testcases[ii].be_id == EC_BACKENDS_MAX) { - /* EC_BACKEND_MAX basically designed for invalid args tests + /* EC_BACKENDS_MAX basically designed for invalid args tests * and not takes the args so call the function w/o args here */ testcases[ii].function(); fprintf(stdout, "ok %d - %s: %s (idx=%d)\n", num_cases,