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

[MB] Create Unit Tests for Portfolio and Markets #553

Closed
4 tasks done
furkansenkal opened this issue Dec 16, 2024 · 2 comments · Fixed by #548
Closed
4 tasks done

[MB] Create Unit Tests for Portfolio and Markets #553

furkansenkal opened this issue Dec 16, 2024 · 2 comments · Fixed by #548
Assignees
Labels
mobile Related to the mobile priority: medium Urgent issue workload: 2 issues that are requiring medium effort

Comments

@furkansenkal
Copy link
Contributor

furkansenkal commented Dec 16, 2024

Description

Unit tests should be created for portfolio and markets pages.

Required Activities

  • Portfolio page test created.
  • Markets page test created.
  • Stock details page test created.
  • Portfolio details page test created.

Estimated Time

3

Deadline

16 Dec 2024

Reviewer

@m-erkam

@furkansenkal furkansenkal added priority: medium Urgent issue workload: 2 issues that are requiring medium effort issue template This issue is generated using templates mobile Related to the mobile labels Dec 16, 2024
@furkansenkal furkansenkal added this to the 451 - Milestone 3 milestone Dec 16, 2024
@furkansenkal furkansenkal self-assigned this Dec 16, 2024
@furkansenkal furkansenkal linked a pull request Dec 16, 2024 that will close this issue
@furkansenkal
Copy link
Contributor Author

Unit tests for mentioned pages are created. Issue is open to be reviewed by @m-erkam.

@furkansenkal furkansenkal removed the issue template This issue is generated using templates label Dec 16, 2024
@m-erkam
Copy link
Contributor

m-erkam commented Dec 20, 2024

Tests are nicely implemented and works well. Nice!

@m-erkam m-erkam closed this as completed Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mobile Related to the mobile priority: medium Urgent issue workload: 2 issues that are requiring medium effort
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants