Skip to content

Conversation

@Mastersam07
Copy link
Collaborator

  • Test todo change notifier
  • Test todo state notifier

@Mastersam07 Mastersam07 changed the title Notifiers test Draft - Notifiers test Apr 13, 2023
@Mastersam07 Mastersam07 marked this pull request as draft April 13, 2023 20:17
@Mastersam07 Mastersam07 marked this pull request as ready for review April 13, 2023 20:30
@Mastersam07 Mastersam07 changed the title Draft - Notifiers test Notifiers test Apr 13, 2023
Copy link
Contributor

@developerjamiu developerjamiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I really want to approve this but I however would like it to be a beginning session in Unit Testing.

We would still have an elaborate session, however I think this little tests would help beginners understand the how in testing.

What do you think?

@Mastersam07
Copy link
Collaborator Author

I agree @developerjamiu. However, we could leave the PR open till then? I will add a separate branch for widget tests so that each branch serves a different session.

@developerjamiu
Copy link
Contributor

Exactly, we will leave it open.

I’ll handle the widget tests session and you can do unit. Or we can have a twin session if you want.

@Mastersam07
Copy link
Collaborator Author

oh, interesting. I just pushed a widget test branch 😅 😅 😅

@developerjamiu
Copy link
Contributor

Eiiiii, Wahala Wahala Wahala 😂.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants