Skip to content

test(provider-baileys,provider-sherpa): enable skipped sendPoll unit …#1231

Open
nicoquintanapy wants to merge 1 commit into
codigoencasa:builderbotfrom
nicoquintanapy:test/enable-sendpoll-unit-tests
Open

test(provider-baileys,provider-sherpa): enable skipped sendPoll unit …#1231
nicoquintanapy wants to merge 1 commit into
codigoencasa:builderbotfrom
nicoquintanapy:test/enable-sendpoll-unit-tests

Conversation

@nicoquintanapy
Copy link
Copy Markdown

Enable previously skipped describe.skip('#sendPoll') test suites in both provider-baileys and provider-sherpa.

For provider-baileys: sendPoll is fully implemented; tests now assert the exact poll payload passed to vendor.sendMessage, verifying selectableCount values for each multiselect variant (false→0, undefined→1, true→1).

For provider-sherpa: sendPoll is deprecated and always returns false; tests are rewritten to verify the 'notice' event is emitted with the correct METHOD NOT AVAILABLE payload.

Que tipo de Pull Request es?

  • Mejoras
  • Bug
  • Docs / tests

Descripción

Enable the two describe.skip('#sendPoll') test suites that were previously skipped.

  • provider-baileys: 4 tests verify the exact poll payload (name, values, selectableCount) sent to vendor.sendMessage for each multiselect variant.
  • provider-sherpa: 3 tests verify that the deprecated sendPoll always returns false and emits a notice event with METHOD NOT AVAILABLE.

All tests pass with pnpm run test.

Forma parte de este proyecto.

…tests

Enable previously skipped describe.skip('#sendPoll') test suites in both
provider-baileys and provider-sherpa.

For provider-baileys: sendPoll is fully implemented; tests now assert the
exact poll payload passed to vendor.sendMessage, verifying selectableCount
values for each multiselect variant (false→0, undefined→1, true→1).

For provider-sherpa: sendPoll is deprecated and always returns false; tests
are rewritten to verify the 'notice' event is emitted with the correct
METHOD NOT AVAILABLE payload.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant