Frontend integration / e2e tests
Open
Nobody has claimed this yet.
feature: frontend 🖌️
overview
- Dominant language
- PHP
- Stars
- 2.2k
- Forks
- 587
- Avg merge
- 17h 55m
- Merged PRs (30d)
- 350
Description
We could add some integration or e2e tests, where:
- complete App / ChatView / MessagesList is rendered inside of parent (as in production);
- the list (or rule) is defined for NcModals which should be always appended to the body (not inside the other component or modal), and checking it during test
- the messages list is populated, so scrolling position could be check
- ...
- extend the list
As a requirement, I'd expect to migrate at least services to Typescript (so we could mock responses correctly in tests)
Originally posted by @Antreesy in https://github.com/nextcloud/spreed/issues/11945#issuecomment-2022786997
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the App, ChatView, MessagesList, NcModals, and services mentioned in the issue, along with how the production view is mounted inside body. Done means adding integration or e2e coverage for body-level modal placement, populated-list scrolling, and related behavior, with services migrated to TypeScript for mocking.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript, typescript
- Domain
- frontend, testing
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100