LearnNewsExamplesServices
Frontmatter
id7287
titleConvert state/FeedbackLoop.mjs Test from Siesta to Playwright
stateClosed
labels
enhancementhelp wantedgood first issuehacktoberfest
assigneesnabeel001
createdAtSep 27, 2025, 3:59 PM
updatedAtOct 10, 2025, 1:07 PM
githubUrlhttps://github.com/neomjs/neo/issues/7287
authortobiu
commentsCount7
parentIssue7262
subIssues[]
subIssuesCompleted0
subIssuesTotal0
blockedBy[]
blocking[]
closedAtOct 10, 2025, 1:07 PM

Convert state/FeedbackLoop.mjs Test from Siesta to Playwright

Closed v11.0.0 enhancementhelp wantedgood first issuehacktoberfest
tobiu
tobiu commented on Sep 27, 2025, 3:59 PM

This task is to migrate the unit test for state/FeedbackLoop.mjs from the Siesta test harness to the Playwright test runner.

Acceptance Criteria

  1. Create a new test file at test/playwright/unit/state/FeedbackLoop.spec.mjs.
  2. Translate all assertions from the original file (test/siesta/tests/state/FeedbackLoop.mjs) to the Playwright/Jest expect syntax.
  3. Ensure the new test runs successfully via npm test.
  4. The new test must cover all the functionality of the original Siesta test.
tobiu added the enhancement label on Sep 27, 2025, 3:59 PM
tobiu added parent issue #7262 on Sep 27, 2025, 3:59 PM
tobiu added the help wanted label on Oct 2, 2025, 9:23 PM
tobiu added the good first issue label on Oct 2, 2025, 9:23 PM
tobiu added the hacktoberfest label on Oct 2, 2025, 9:23 PM
shubham220420
shubham220420 Oct 3, 2025, 12:23 PM

hey @tobiu, I'd like to work on this, please assign this to me.

tobiu
tobiu Oct 3, 2025, 6:15 PM

Hi and thanks for your interest. For the testing tickets, I strongly recommend trying out the “ai native” workflow.

I would read the hacktoberfest intro first: https://github.com/neomjs/neo/issues/7296

Then the following 2 guides: https://github.com/neomjs/neo/blob/dev/.github/WORKING_WITH_AGENTS.md https://github.com/neomjs/neo/blob/dev/.github/AI_QUICK_START.md

E.g. gemini cli should be capable to complete it on its own.

tobiu assigned to @shubham220420 on Oct 3, 2025, 6:15 PM
nabeel001
nabeel001 Oct 9, 2025, 6:33 AM

Hi @tobiu In case @shubham220420 is unable to resolve this issue, Kindly assign this to me.

shubham220420
shubham220420 Oct 9, 2025, 10:31 AM

Hey @nabeel001 Its not that I am not able to solve it but currently I am a bit busy in my exams, so you can take over this problem :)

nabeel001
nabeel001 Oct 9, 2025, 11:02 AM

Sure @shubham220420 No worries, All the best with your exams. I'll take care of this issue.

@tobiu kindly assign this to me

tobiu
tobiu Oct 9, 2025, 12:52 PM

Hi guys, sure, I can re-assign it. This was interesting on a meta-level: I just updated our contributing md file, to add:

To keep the project moving, if a ticket is assigned and we do not receive any feedback from the assignee (e.g. via a comment) within 7 days, the ticket may get re-assigned to make it available for other contributors.

https://github.com/neomjs/neo/issues/7426 (also updated it, since it was extremely outdated).

Good luck with your exams from me too!

And no worries: even when this epic is finished, there are a lot more files which could use unit testing (Gemini can identify them, and create new tickets). Plus, there is also component based testing, which would be a full new epic.

tobiu unassigned from @shubham220420 on Oct 9, 2025, 12:53 PM
tobiu assigned to @nabeel001 on Oct 9, 2025, 12:53 PM
nabeel001 cross-referenced by PR #7433 on Oct 10, 2025, 4:19 AM
nabeel001
nabeel001 Oct 10, 2025, 4:20 AM

Hi @tobiu Kindly review and merge the PR.

tobiu closed this issue on Oct 10, 2025, 1:07 PM