LearnNewsExamplesServices
Frontmatter
titledocs(test-plan): add component-Video test plan (#7489)
authorritikrikm
stateMerged
createdAtOct 14, 2025, 8:04 PM
updatedAtOct 15, 2025, 1:38 PM
closedAtOct 15, 2025, 1:38 PM
mergedAtOct 15, 2025, 1:38 PM
branchesdevfeat/test-plan-component-video-7489
urlhttps://github.com/neomjs/neo/pull/7490
Merged
ritikrikm
ritikrikm commented on Oct 14, 2025, 8:04 PM

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe: Documentation (AI-Native test plan)

Does this PR introduce a breaking change? (check one)

  • Yes
  • [ x ] No

If yes, please describe the impact and migration path for existing applications:

The PR fulfills these requirements:

  • [ x ] It's submitted to the dev branch, not the main branch
  • [ x ] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

  • A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information: This PR adds the AI-Native Cookbook test plan for Neo.component.Video.

  • Epic: Create Component Test Plans (Hacktoberfest Cookbook) #7475
  • Sub-issue: Create Test Plan for component.Video #7489
  • File: .github/ISSUE/ticket-cpm-test-component-video.md

Scope:

  • Inherited behaviours (hidden, cls, style, baseCls)
  • Component-specific behaviours (autoplay, controls, muted/inline, ghost toggle, afterSetUrl, play/pause, afterSetPlaying)
  • Edge cases (url changes, empty url, autoplay without url)

Notes:

  • Excludes non-component demo UI like theme toggle.
tobiu
tobiu commented on Oct 15, 2025, 1:37 PM

Sorry for the delay. i actually wanted to review the PR with using our new github-workflow mcp server for the first time, but then ended up in a massive refactoring effort to get it "right": https://github.com/neomjs/neo/issues/7477

Still work in progress.

Screenshot 2025-10-15 at 11 39 26

I will accept the PR. We could probably enhance the ticket a bit more, and change the url of an existing video to something else at run-time.

Out of the testing scope, but probably good to get an idea what neo can do with this widget: start playing the video, then click the move button: https://neomjs.com/examples/videoMove/index.html