fix(oracle): For Raw queries avoid converting the input parameters passed

[sudarshan12s]

Pull Request Checklist

• Have you added new tests to prevent regressions?
• If a documentation update is necessary, have you opened a PR to the documentation repository?
• Did you update the typescript typings accordingly (if applicable)?
• Does the description below contain a link to an existing issue (Closes #[issue]) or a description of the issue you are solving?
• Does the name of your PR follow our conventions?

Description Of Change

For Sequelize Raw queries such as below , the input bind value parameter, { one: 1, two: 2 } is converted to array [1,2] which is not required. Hence fixed code to avoid the same.

const result = await this.sequelize.query(select $one${typeCast} as foo, $two${typeCast} as bar${Support.addDualInSelect()}, { raw: true, bind: { one: 1, two: 2 }, logging(s) { logSql = s; } });

Todos

[WikiRik]

Can you add a test for this?

[sudarshan12s]

Can you add a test for this?

Can you add a test for this?

Yes. changed the existing test case. The issue happened when bind values were sent as array and they are treated like replacements (?) . I modified the existing test case to change the order of bind names like ' select :one as foo, :one as bar, :two as baz FROM DUAL' instead of 'select :one as foo, :two as bar, :one as baz FROM DUAL'. The test fails with Oracle dialect with error 'ORA-01008: not all variables bound' . With the fix to retain the bind values as it is { one: 1, two: 2 } , the test case passed.

[github-actions]

🎉 This PR is included in version 6.32.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀