apidash icon indicating copy to clipboard operation
apidash copied to clipboard

Add UI/riverpod tests cases

Open ashitaprasad opened this issue 1 year ago • 14 comments

Tell us about the task you want to perform and are unable to do so because the feature is not available Currently, API Dash has Unit tests & Widget tests, but needs support for adding UI tests which involves injecting riverpod states and testing the behaviour of screens. This will help improve the code coverage.

ashitaprasad avatar Feb 22 '24 00:02 ashitaprasad

I'll like to work on this.

I went through the contribution guidelines and the second step is to comment on the issue. I want to fork the repo and run it locally now.

sixtusagbo avatar Mar 03 '24 12:03 sixtusagbo

I've run the app and the tests and I'm getting started on the UI tests now

sixtusagbo avatar Mar 05 '24 04:03 sixtusagbo

Awesome @sixtusagbo, you can send across a PR once you are done writing tests.

ashitaprasad avatar Mar 05 '24 05:03 ashitaprasad

So, I had school exams that I just concluded yesterday. I'm fully available from now. I'm working on the tests and I'll send across a PR when I'm done.

sixtusagbo avatar Mar 09 '24 15:03 sixtusagbo

Awesome @sixtusagbo 👍

animator avatar Mar 10 '24 09:03 animator

I've written tests for some of the providers. We'll be on break during the GSoC period so I have all the time to work on this project.

sixtusagbo avatar Mar 20 '24 07:03 sixtusagbo

@sixtusagbo Please send across a PR as soon as possible so that we can review the approach and provide feedback.

ashitaprasad avatar Mar 20 '24 12:03 ashitaprasad

I've done that, while I wait. I want to round up the Enter Key creates a new entry for "Add Param" and "Add Header"

sixtusagbo avatar Mar 20 '24 23:03 sixtusagbo

@sixtusagbo that issue has already been resolved and is in the final stages of review. Kindly work on adding more UI tests.

animator avatar Mar 20 '24 23:03 animator

Okay

sixtusagbo avatar Mar 20 '24 23:03 sixtusagbo

@ashitaprasad @animator This issue can be closed now I believe.

Tanish2002 avatar Mar 24 '24 15:03 Tanish2002

I don't think so. There are other test cases I'm working on. I just haven't updated remote yet. And I have an approach for the mock. I had to modify the request() helper method in http_services.dart

sixtusagbo avatar Mar 24 '24 16:03 sixtusagbo

Ah my bad, the linked PR was closed so I thought it was fine.

Tanish2002 avatar Mar 24 '24 16:03 Tanish2002

I'll open a PR on my progress before the end of the week. I added more tests. My goal is to finish saveDataStateProvider, clearDataStateProvider, hasUnsavedChangesProvider, nameTextFieldFocusNodeProvider and searchQueryProvider before the actual GSoC coding week begins.

sixtusagbo avatar Apr 08 '24 10:04 sixtusagbo