status-mobile
status-mobile copied to clipboard
status-im
Extend clojure integration test to cover starting a one-to-one chat
We should increase the coverage of the newly created clojure integration tests.
One of the most common use cases is to create/start a one to one chat.
The test steps that should be covered in the tests should be:
- Create a new account (already covered by steps) & login (we can start from these steps https://github.com/status-im/status-react/blob/9f9b896eb7fcfac60d55b33a3744480b13d2ba8d/src/status_im/integration_test.cljs#L46 )
- Simulate inputting a public key
- Create a chat
We want to achieve the closest coverage to what the UI is actually doing.
To simulate in the UI:
- Create a new account
- Click on the + button on the home screen
- Click on Start new chat
- Input a public key
- Click on the username
If you have any question feel free to ask me on discord.
Thoughts while working on this one.
The first thing I tried is to get a working REPL to do code on integration tests.
Currently, the test runner is injected by shadow-cljs and doesn't have an option for not quitting the node process.
I suggest we fix it with a custom test runner, or, even better, implement it in shadow-cljs.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
This issue now has a funding of 200.0 DAI attached to it.
- If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
- Want to chip in? Add your own contribution here.
- Questions? Checkout Gitcoin Help or the Gitcoin's Discord
- $1,003,666,529.22 more funded OSS Work available on the Gitcoin Issue Explorer
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
Work has been started.
These users each claimed they can complete the work by 3 weeks, 3 days from now. Please review their action plans below:
1) littlezigy has applied to start work (Funders only: approve worker | reject worker).
Hello. I have taken the steps in "To simulate in the UI" on the mobile app and I understand the task.
I will update the integration tests (integration_test.cljs) and add steps to cover
- Login
- Inputting (and searching for) a public key
- Creating a chat 2) zelenac-buf has applied to start work (Funders only: approve worker | reject worker).
Hello, I would like to write the integration tests. I fully understand the task and look forward to contribute. 3) spac3c4det has been approved to start work.
Was assigned the issue in Github and finished a PR: https://github.com/status-im/status-mobile/pull/13797
Learn more on the Gitcoin Issue Details page.
Could someone please go and merge: https://github.com/status-im/status-mobile/issues/13579
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
Work for 200.0 DAI (200.00 USD @ $1.0/DAI) has been submitted by:
@statussceptre please take a look at the submitted work:
- PR by @spac3c4det
- Learn more on the Gitcoin Issue Details page
- Want to chip in? Add your own contribution here.
- Questions? Checkout Gitcoin Help or the Gitcoin's Discord
- $1,003,620,038.11 more funded OSS Work available on the Gitcoin Issue Explorer