diff --git a/src/bindings/js/node/src/core_wrap.cpp b/src/bindings/js/node/src/core_wrap.cpp index 8264bff0c1a103..33350056fc443c 100644 --- a/src/bindings/js/node/src/core_wrap.cpp +++ b/src/bindings/js/node/src/core_wrap.cpp @@ -86,9 +86,7 @@ Napi::Value CoreWrap::read_model_sync(const Napi::CallbackInfo& info) { } else if (ov::js::validate(info, allowed_signatures)) { model = _core.read_model(info[0].ToString()); } else { - const auto error_message = ov::js::get_parameters_error_msg(info, allowed_signatures); - - OPENVINO_THROW(error_message); + OPENVINO_THROW("'readModelSync'", ov::js::get_parameters_error_msg(info, allowed_signatures)); } return ModelWrap::wrap(info.Env(), model); @@ -168,10 +166,8 @@ Napi::Value CoreWrap::compile_model_sync_dispatch(const Napi::CallbackInfo& info return compile_model_sync(info, info[0].ToObject(), info[1].ToString(), config); } + OPENVINO_THROW("'compileModelSync'", ov::js::get_parameters_error_msg(info, allowed_signatures)); - const auto error_message = ov::js::get_parameters_error_msg(info, allowed_signatures); - - OPENVINO_THROW(error_message); } catch (std::exception& e) { reportError(info.Env(), e.what()); diff --git a/src/bindings/js/node/src/type_validation.cpp b/src/bindings/js/node/src/type_validation.cpp index 737aba21c14f27..4852e0e4051c2a 100644 --- a/src/bindings/js/node/src/type_validation.cpp +++ b/src/bindings/js/node/src/type_validation.cpp @@ -147,8 +147,8 @@ bool validate_value(const Napi::Env& env, const Napi::Value& value) } std::string get_parameters_error_msg(const Napi::CallbackInfo& info, std::vector& allowed_signatures) { - return "Method 'compileModelSync' called with incorrect parameters.\nProvided signature: " + - js::get_current_signature(info) + " \nAllowed signatures:\n- " + ov::util::join(allowed_signatures, "\n- "); + return " method called with incorrect parameters.\nProvided signature: " + js::get_current_signature(info) + + " \nAllowed signatures:\n- " + ov::util::join(allowed_signatures, "\n- "); } } // namespace js } // namespace ov diff --git a/src/bindings/js/node/tests/basic.test.js b/src/bindings/js/node/tests/basic.test.js index e95f76a6470609..ba3e585e40ce7d 100644 --- a/src/bindings/js/node/tests/basic.test.js +++ b/src/bindings/js/node/tests/basic.test.js @@ -75,7 +75,7 @@ describe('Core.compileModelSync()', () => { }); it('compileModelSync(model, device, config) throws when config is a string', () => { - const expectedMsg = ("Method 'compileModelSync' called with incorrect parameters.\n" + + const expectedMsg = ("'compileModelSync' method called with incorrect parameters.\n" + 'Provided signature: (object, string, string) \n' + 'Allowed signatures:\n' + '- (string, string)\n' + @@ -97,7 +97,7 @@ describe('Core.compileModelSync()', () => { }); it('compileModelSync(model) throws if the number of arguments is invalid', () => { - const expectedMsg = ("Method 'compileModelSync' called with incorrect parameters.\n" + + const expectedMsg = ("'compileModelSync' method called with incorrect parameters.\n" + 'Provided signature: (object) \n' + 'Allowed signatures:\n' + '- (string, string)\n' + diff --git a/src/bindings/js/node/tests/read_model.test.js b/src/bindings/js/node/tests/read_model.test.js index 407627c5ae9cb0..731001fbb861b7 100644 --- a/src/bindings/js/node/tests/read_model.test.js +++ b/src/bindings/js/node/tests/read_model.test.js @@ -27,6 +27,13 @@ describe('Core.readModeSync', () => { assert.equal(model.inputs.length, 1); }); + it('readModeSync throws', () => { + assert.throws( + () => core.readModelSync(core), + /'readModelSync' method called with incorrect parameters./, + ) + }); + it('readModeSync(modelUint8ArrayBuffer, weightsUint8ArrayBuffer) ', () => { const model = core.readModelSync( new Uint8Array(modelFile.buffer),