Skip to content

Commit

Permalink
Enabled code style check for tests (openvinotoolkit#6724)
Browse files Browse the repository at this point in the history
  • Loading branch information
ilya-lavrenov authored and akuporos committed Sep 29, 2021
1 parent 1a92427 commit bfb4425
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/code_style.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
run: |
mkdir build
cd build
cmake -DENABLE_PYTHON=ON -DENABLE_PROFILING_ITT=ON -DSELECTIVE_BUILD=COLLECT ..
cmake -DENABLE_PYTHON=ON -DENABLE_TESTS=ON -DENABLE_PROFILING_ITT=ON -DSELECTIVE_BUILD=COLLECT ..
- name: Check code style
run: cmake --build build --target clang_format_check_all
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -309,7 +309,6 @@ class MOCK_API FrontEndMockPy : public FrontEnd
public:
FrontEndMockPy() {}


std::shared_ptr<ngraph::Function> convert(InputModel::Ptr model) const override
{
m_stat.m_convert_model++;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -504,7 +504,8 @@ class MOCK_API FrontEndMockPy : public FrontEnd
InputModel::Ptr load_impl(const std::vector<std::shared_ptr<Variant>>& params) const override
{
if (params.size() > 0 && is_type<VariantWrapper<std::string>>(params[0]))
m_stat.m_load_paths.push_back(as_type_ptr<VariantWrapper<std::string>>(params[0])->get());
m_stat.m_load_paths.push_back(
as_type_ptr<VariantWrapper<std::string>>(params[0])->get());
return std::make_shared<InputModelMockPy>();
}

Expand Down
3 changes: 1 addition & 2 deletions ngraph/test/frontend/frontend_manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -34,8 +34,7 @@ static int set_test_env(const char* name, const char* value)
TEST(FrontEndManagerTest, testAvailableFrontEnds)
{
FrontEndManager fem;
ASSERT_NO_THROW(fem.register_front_end(
"mock", []() { return std::make_shared<FrontEnd>(); }));
ASSERT_NO_THROW(fem.register_front_end("mock", []() { return std::make_shared<FrontEnd>(); }));
auto frontends = fem.get_available_front_ends();
ASSERT_NE(std::find(frontends.begin(), frontends.end(), "mock"), frontends.end());
FrontEnd::Ptr fe;
Expand Down
2 changes: 1 addition & 1 deletion ngraph/test/frontend/shared/src/cut_specific_model.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3,8 +3,8 @@
//

#include "cut_specific_model.hpp"
#include "utils.hpp"
#include "ngraph/opsets/opset7.hpp"
#include "utils.hpp"

using namespace ngraph;
using namespace ngraph::frontend;
Expand Down
2 changes: 1 addition & 1 deletion ngraph/test/frontend/shared/src/load_from.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ TEST_P(FrontEndLoadFromTest, testLoadFromTwoFiles)
TEST_P(FrontEndLoadFromTest, testLoadFromStream)
{
auto ifs = std::make_shared<std::ifstream>(m_param.m_modelsPath + m_param.m_stream,
std::ios::in | std::ifstream::binary);
std::ios::in | std::ifstream::binary);
auto is = std::dynamic_pointer_cast<std::istream>(ifs);
std::vector<std::string> frontends;
FrontEnd::Ptr fe;
Expand Down

0 comments on commit bfb4425

Please sign in to comment.