Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 18, 2025

Describe the PR

Adds comprehensive event emission tests for BTable and BTableLite. Existing tests covered structure, sorting, filtering, and selection but lacked verification of emitted events.

Small replication

N/A - Test-only changes

Changes

BTableLite (+9 tests)

  • Row interaction events: row-clicked, row-dblclicked, row-contextmenu, row-hovered, row-unhovered, row-middle-clicked
  • Header interaction: head-clicked
  • Event filtering for interactive elements (buttons/links)
  • Multiple simultaneous event emissions

BTable (+9 tests)

  • Table-specific events: sorted, filtered, row-selected, row-unselected, change
  • Inherited BTableLite events: row-clicked, row-dblclicked, head-clicked
  • Event sequencing validation

PR checklist

What kind of change does this PR introduce? (check at least one)

  • Bugfix 🐛 - fix(...)
  • Feature - feat(...)
  • ARIA accessibility - fix(...)
  • Documentation update - docs(...)
  • Other (please describe): Test coverage improvement

The PR fulfills these requirements:

Original prompt

This section details on the original issue you should resolve

<issue_title>write unit tests for btable & btablelite</issue_title>
<issue_description>Write unit tests for btable and btable lite</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@coderabbitai
Copy link

coderabbitai bot commented Nov 18, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Copilot AI changed the title [WIP] Add unit tests for btable and btable lite test: add event emission tests for BTable and BTableLite Nov 18, 2025
@VividLemon VividLemon marked this pull request as ready for review November 18, 2025 18:20
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 18, 2025

bsvn-vite-ts

npm i https://pkg.pr.new/bootstrap-vue-next/bootstrap-vue-next@2915
npm i https://pkg.pr.new/bootstrap-vue-next/bootstrap-vue-next/@bootstrap-vue-next/nuxt@2915

commit: a46d327

@VividLemon VividLemon merged commit 1fa1f12 into main Nov 18, 2025
8 checks passed
xvaara added a commit to xvaara/bootstrap-vue-next that referenced this pull request Nov 30, 2025
* upstream/main: (28 commits)
  fix(BToggle)! Remove redundant attribute cleanup & update docs for accessibility attributes on show/hide components (bootstrap-vue-next#2918)
  chore: release main (bootstrap-vue-next#2912)
  fix: allow custom component props in orchestrator create methods with type safety (bootstrap-vue-next#2922)
  fix(BModal): prevent focus trap error when no tabbable elements exist (bootstrap-vue-next#2864)
  Update package description for clarity (bootstrap-vue-next#2917)
  Update project description for clarity and detail
  test: add event emission tests for BTable and BTableLite (bootstrap-vue-next#2915)
  fix(BTableLite): Use primary key to persist row-details state when items change (bootstrap-vue-next#2871)
  chore: release main (bootstrap-vue-next#2896)
  feat(BTable): add configurable debouncing
  ci: Add publint to the build process (bootstrap-vue-next#2909)
  docs(Offcanvas): Parity pass (bootstrap-vue-next#2900)
  fix(directives): Robustness fixes for directives (bootstrap-vue-next#2906)
  docs(BFormInput): document @wheel.prevent for disabling mousewheel events (bootstrap-vue-next#2903)
  fix(typings): Fix paths to `*.d.mts` files (bootstrap-vue-next#2907)
  feat: add name and form props to BFormRating for form submission (bootstrap-vue-next#2895)
  docs: refactor docs to avoid duplication and boilerplate code (bootstrap-vue-next#2891)
  docs(BToast): Parity (bootstrap-vue-next#2887)
  docs(BModal): fix attribute to hide footer (bootstrap-vue-next#2888)
  docs(BPlaceholder): Parity pass (bootstrap-vue-next#2886)
  ...
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.

write unit tests for btable & btablelite

2 participants