Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add integration tests #119

Closed
ashitaprasad opened this issue Feb 22, 2024 · 7 comments
Closed

Add integration tests #119

ashitaprasad opened this issue Feb 22, 2024 · 7 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@ashitaprasad
Copy link
Member

Tell us about the task you want to perform and are unable to do so because the feature is not available
Add few end-to-end integration tests to automate some user testing workflows.

@ashitaprasad ashitaprasad added enhancement New feature or request good first issue Good for newcomers labels Feb 22, 2024
@Drakennnnn
Copy link

Drakennnnn commented Mar 4, 2024

Hi i would like to work on this issue. It will take a little time to integrate the test files and include related dependencies. Sometimes the debug message of the API calling can be misleading. Currently, are we using manual testing for the app? And do the existing tests cover all aspects. Please let me know if theres something else to be included so i can start on automating user testing. Thank you.

@animator
Copy link
Member

animator commented Mar 4, 2024

@Drakennnnn The issue is not about integrating the test files. We already have automated testing in place.
Integration testing is a totally different concept. You can read more about it here.

@Drakennnnn
Copy link

Drakennnnn commented Mar 5, 2024

@animator I have gone through the article you have shared. As API DASH is a desktop application, we cannot use firebase test tool and have to write code covering every aspect of the app. Its an easy task but lot of code to type. I will be done with it within a week. I am having a little trouble with the code want to know if theres a easier way to implement the code for testing the codegens or should I leave it from the code as of now and it can be tested manually in the meanwhile. Thank You.
Screenshot 2024-03-05 235857

@ashitaprasad
Copy link
Member Author

@Drakennnnn I will suggest you to start writing some widget tests for some screens and once you are well versed you can move onto integration testing.

@Drakennnnn
Copy link

@animator @ashitaprasad I am done with writing around 60-70% of the tests. These tests are UX based and covers the app as whole. Unfortunately I will have to pause until the 22nd as my exams will be starting. Hope that's okay. I can share what I have done until now if required. Thank you.

@animator
Copy link
Member

@Drakennnnn you can send across a PR so that we can review the existing work.

@ashitaprasad
Copy link
Member Author

Integration tests have been added by @DenserMeerkat

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants