[moco-tf] Clear ConcatV2 import test (#6522)
author박세희/On-Device Lab(SR)/Principal Engineer/삼성전자 <saehie.park@samsung.com>
Tue, 13 Aug 2019 01:47:12 +0000 (10:47 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Tue, 13 Aug 2019 01:47:12 +0000 (10:47 +0900)
This will erase ConcatV2 TF dialect import test to reduce complexity of diff for migration to using VariadicArityNode

Signed-off-by: SaeHie Park <saehie.park@samsung.com>
compiler/moco-tf/src/Op/Concat.test.cpp

index 339a8ca..53daa63 100644 (file)
@@ -193,28 +193,7 @@ TEST(TensorFlowImport, concat_01)
   }
 
   // Test "ConcatV2GraphBuilderImpl<ImportTarget::Tensorflow>"
-  {
-    // what to test:
-    // - there should exist TensorConcat
-    // - lhs() should not be nullptr
-    // - rhs() should not be nullptr
-    // - axis() should match
-
-    using ConcatV2GraphBuilder = ConcatV2GraphBuilderImpl<ImportTarget::TensorFlow>;
-
-    moco::tf::GraphBuilderRegistry r{&moco::tf::GraphBuilderRegistry::get()};
-    r.add("ConcatV2", stdex::make_unique<ConcatV2GraphBuilder>());
-    moco::tf::Importer importer{&r};
-
-    std::unique_ptr<loco::Graph> graph = importer.import(signature, graph_def);
-
-    auto concat_node = moco::tf::test::find_first_node_bytype<moco::tf::TFConcatV2>(graph.get());
-
-    ASSERT_NE(concat_node, nullptr);
-    ASSERT_NE(concat_node->lhs(), nullptr);
-    ASSERT_NE(concat_node->rhs(), nullptr);
-    ASSERT_EQ(concat_node->axis(), 0);
-  }
+  // TODO implement test
 }
 
 namespace
@@ -422,39 +401,7 @@ TEST(TensorFlowImport, concat_02)
   }
 
   // Test "ConcatV2GraphBuilderImpl<ImportTarget::TensorFlow>"
-  {
-    // what to test: Concat has 3 inputs --> Importer creates 2 TFConcatV2
-    // - there should exist two TFConcatV2
-    // - lhs() of #1 should not be nullptr
-    // - rhs() of #1 should not be nullptr
-    // - lhs() of #2 should be #1
-    // - rhs() of #2 should not be nullptr
-    // - axis() should match
-
-    using ConcatV2GraphBuilder = ConcatV2GraphBuilderImpl<ImportTarget::TensorFlow>;
-
-    moco::tf::GraphBuilderRegistry r{&moco::tf::GraphBuilderRegistry::get()};
-    r.add("ConcatV2", stdex::make_unique<ConcatV2GraphBuilder>());
-    moco::tf::Importer importer{&r};
-
-    std::unique_ptr<loco::Graph> graph = importer.import(signature, graph_def);
-
-    std::vector<moco::tf::TFConcatV2 *> concat_nodes =
-        moco::tf::test::find_nodes_bytype<moco::tf::TFConcatV2>(graph.get());
-    ASSERT_EQ(concat_nodes.size(), 2);
-    moco::tf::TFConcatV2 *concat_node0 = concat_nodes.at(0);
-    moco::tf::TFConcatV2 *concat_node1 = concat_nodes.at(1);
-
-    ASSERT_NE(concat_node0, nullptr);
-    ASSERT_NE(concat_node1, nullptr);
-    ASSERT_NE(concat_node0->lhs(), nullptr);
-    ASSERT_NE(concat_node0->rhs(), nullptr);
-    ASSERT_NE(concat_node1->lhs(), nullptr);
-    ASSERT_NE(concat_node1->rhs(), nullptr);
-    ASSERT_TRUE(concat_node0->lhs() == concat_node1 || concat_node1->lhs() == concat_node0);
-    ASSERT_EQ(concat_node0->axis(), 0);
-    ASSERT_EQ(concat_node1->axis(), 0);
-  }
+  // TODO implement test
 }
 
 namespace
@@ -601,26 +548,5 @@ TEST(TensorFlowImport, concat_03)
   }
 
   // Test "ConcatV2GraphBuilderImpl<ImportTarget::TensorFlow>"
-  {
-    // what to test: minus axis value validation
-    // - there should exist a TFConcatV2
-    // - lhs() should not be nullptr
-    // - rhs() should not be nullptr
-    // - axis() should match 2 + (-1), where 2 came from rank of input(s)
-
-    using ConcatV2GraphBuilder = ConcatV2GraphBuilderImpl<ImportTarget::TensorFlow>;
-
-    moco::tf::GraphBuilderRegistry r{&moco::tf::GraphBuilderRegistry::get()};
-    r.add("ConcatV2", stdex::make_unique<ConcatV2GraphBuilder>());
-    moco::tf::Importer importer{&r};
-
-    std::unique_ptr<loco::Graph> graph = importer.import(signature, graph_def);
-
-    auto concat_node = moco::tf::test::find_first_node_bytype<moco::tf::TFConcatV2>(graph.get());
-
-    ASSERT_NE(concat_node, nullptr);
-    ASSERT_NE(concat_node->lhs(), nullptr);
-    ASSERT_NE(concat_node->rhs(), nullptr);
-    ASSERT_EQ(concat_node->axis(), (2 + (-1)));
-  }
+  // TODO implement test
 }