message: Fix requested changes
diff: @@ -1986,7 +1986,7 @@ TEST_F(OpConverterTest, ConvertActivation) {
  }
  // Get nodedef for activation layer.
  - auto get_act_nodedef = [](std::string op_name) -> NodeDef {
  + auto get_act_nodedef = [](string op_name) -> NodeDef {
  Scope s = Scope::NewRootScope();
  auto input = ops::Placeholder(s.WithOpName("input"), DT_FLOAT);
  if (op_name == "Relu") {
  @@ -1999,11 +1999,11 @@ TEST_F(OpConverterTest, ConvertActivation) {
  auto act = ops::Tanh(s.WithOpName("my_act"), input);
  return act.operation.node()->def();
  }
  - EXPECT_TRUE(false);
  - return MakeNodeDef("my_act", "Relu", {});
  + ASSERT_TRUE(false);
  + return NodeDef();
  };
  // Get expected output for activation layer.
  - auto get_act_output = [](std::string op_name, float input) -> float {
  + auto get_act_output = [](string op_name, float input) -> float {
  if (op_name == "Relu") {
  return (input > 0.0f) ? input : 0.0f;
  } else if (op_name == "Sigmoid") {
  @@ -2011,13 +2011,12 @@ TEST_F(OpConverterTest, ConvertActivation) {
  } else if (op_name == "Tanh") {
  return std::tanh(input);
  }
  - EXPECT_TRUE(false);
  - return input;
  + ASSERT_TRUE(false);
  + return 0;
  };
  - {
  // Ok.
  - for (std::string op_name : {"Relu", "Sigmoid", "Tanh"}) {
  + for (string op_name : {"Relu", "Sigmoid", "Tanh"}) {
  Reset();
  NodeDef node_def = get_act_nodedef(op_name);
  AddTestTensor("input", {1, 2, 3});
  @@ -2025,18 +2024,14 @@ TEST_F(OpConverterTest, ConvertActivation) {
  TRT_TensorOrWeights output;
  TF_EXPECT_OK(GetTensorOrWeights("my_act", &output));
  EXPECT_TRUE(output.is_tensor());
  - EXPECT_TRUE(TrtDimsEqualsArray({1, 2, 3}, output.tensor()->getDimensions()))
  - << output.DebugString();
  + ExpectTrtDimsEqualsArray({1, 2, 3}, output.tensor()->getDimensions());
  const std::vector<float> input_data = {-100, -2, -1, 0, 1, 100};
  std::vector<float> output_data(6);
  BuildAndRun("input", input_data, "my_act", &output_data);
  for (int i = 0; i < input_data.size(); i++) {
  const float expected_output = get_act_output(op_name, input_data[i]);
  - EXPECT_FLOAT_EQ(output_data[i], expected_output)
  - << op_name << "(" << input_data[i] << ") should be equal to "
  - << expected_output;
  - }
  + EXPECT_FLOAT_EQ(output_data[i], expected_output);
  }
  }
  }
  