improve consistency in case of volid_test failure

Maximilien Siavelis

2018-01-14 56a796974a46b1a6dcf2dbc8d483227eb68fe372

 test/functional/volid_test.cpp

@@ -50,12 +50,12 @@
   for (int i = 0; i < num_strategies; i++) {
      FUNCTION_RETURN generate_ok = generate_user_pc_signature(identifier_out,
            strategies[i]);
      if (generate_ok != FUNC_RET_OK){
         idfile.close();
         remove(idfileLocation.c_str());
         BOOST_ERROR("Generating identifier for strategy " << strategies[i] << "failed with: " << generate_ok);
      }
      BOOST_ASSERT(generate_ok == FUNC_RET_OK);
      if (generate_ok != FUNC_RET_OK){
         BOOST_ERROR("Generating identifier for strategy " << strategies[i] << " failed with: " << generate_ok);
            idfile << "0000-0000-0000-0000" << endl;
      }
        else
      idfile << identifier_out << endl;
   }
   idfile.close();
@@ -73,46 +73,44 @@
   if (!test_idfile_exist.good()) {
      generate_reference_file(idfileLocation, strategies, num_strategies);
   }
   //try to locate a "good" reference file.
   int tries = 2;
   std::vector<string> reference_signatures;
   do{
      std::ifstream is(idfileLocation);
      std::istream_iterator<string> start(is), end;
      reference_signatures = vector<string>(start, end);
      if (reference_signatures.size() != num_strategies){
         is.close();
         remove(idfileLocation.c_str());
    else{
        std::istream_iterator<string> start(test_idfile_exist), end;
        std::vector<string> reference_signatures(start, end);
        test_idfile_exist.close();
        if (reference_signatures.size() != num_strategies ||
            std::find(reference_signatures.begin(), reference_signatures.end(), "0000-0000-0000-0000") != reference_signatures.end())
         generate_reference_file(idfileLocation, strategies, num_strategies);
      }
   } while (reference_signatures.size() != num_strategies && tries-- > 0);
   BOOST_ASSERT(tries > 0);

   PcSignature generated_identifier;
    std::ifstream is(idfileLocation);
    std::istream_iterator<string> start(is), end;
    std::vector<string> reference_signatures(start, end);
   BOOST_TEST_CHECKPOINT("Generating current signatures and comparing with past");
   for (int i = 0; i < num_strategies; i++) {
        PcSignature generated_identifier;
      FUNCTION_RETURN generate_ok = generate_user_pc_signature(
            generated_identifier, strategies[i]);
      BOOST_ASSERT(generate_ok == FUNCTION_RETURN::FUNC_RET_OK);
        if (generate_ok != FUNC_RET_OK){
         BOOST_ERROR("Generating identifier for strategy " << strategies[i] << " failed with: " << generate_ok);
            continue;
        }
      if (reference_signatures[i] != generated_identifier) {
         string message = string("pc signature compare fail: strategy:")
               + to_string((long double) strategies[i]) + " generated: ["
               + to_string(static_cast<long long>(strategies[i])) + " generated: ["
               + generated_identifier + "] reference: ["
               + reference_signatures[i] + "]";
         BOOST_FAIL(message);
         BOOST_ERROR(message);
      }
   }

   BOOST_TEST_CHECKPOINT("Verifying signatures");
   for (int j = 0; j < 100; j++) {
      for (unsigned int i = 0; i < reference_signatures.size(); i++) {
            if (reference_signatures[i] == "0000-0000-0000-0000")
                continue;
         PcSignature pcsig;
         strncpy(pcsig, reference_signatures[i].c_str(),
               sizeof(PcSignature));
         EVENT_TYPE val_result = validate_pc_signature(pcsig);
         string message = string("pc signature verification strategy:")
               + to_string(static_cast<long long>(i)) + " generated: [" + generated_identifier
               + "] reference: [" + reference_signatures[i] + "]";
         BOOST_TEST_CHECKPOINT("Verifying signature: ");
         BOOST_CHECK_EQUAL(val_result, LICENSE_OK);
      }