Skip to content

Commit

Permalink
[ GPU/Unit test ] Disable setPropertiesInvalid_n_gpu test
Browse files Browse the repository at this point in the history
- This commit is related to #2774.
- The test `setPropertiesInvalid_n_gpu` was failed with
LayerPropertySemantics, which tests negative case for some layers not
implemented for GPU.
- This test should be enabled later.

Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Eunju Yang <[email protected]>
  • Loading branch information
EunjuYang committed Nov 1, 2024
1 parent 8d301f6 commit 71f3a4a
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion test/unittest/layers/layers_dependent_common_tests.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -131,11 +131,18 @@ TEST_P(LayerSemanticsGpu, createFromClContext_pn) {
expected_type);
}

TEST_P(LayerPropertySemantics, setPropertiesInvalid_n_gpu) {
/**
* TODO: Fix the LayerPropertySemantics test errors for GPU
* In current, layers tested with LayerPropertySemantics are not supported in GPU.
* This should be enabled with new test name with `LayerPropertySemanticsGPU`
* for negative test case
TEST_P(LayerPropertySemanticsGpu, setPropertiesInvalid_n_gpu) {
auto lnode =
nntrainer::createLayerNode(expected_type, {}, ComputeEngine::GPU);
EXPECT_THROW(layer->setProperty({valid_properties}), std::invalid_argument);
}
*/

TEST_P(LayerSemanticsGpu, setPropertiesInvalid_n) {
auto lnode =
Expand Down

0 comments on commit 71f3a4a

Please sign in to comment.