Skip to content

Conversation

@GoncaloCanteiro
Copy link
Contributor

Relates to:

Proposed Changes

  • Add tests to all communication producers
  • Add mocks to prevent duplicate code

This pull request closes #1275

@GoncaloCanteiro GoncaloCanteiro added this to the Full Version milestone Mar 21, 2023
@GoncaloCanteiro GoncaloCanteiro self-assigned this Mar 21, 2023
@GoncaloCanteiro GoncaloCanteiro linked an issue Mar 21, 2023 that may be closed by this pull request
@GoncaloCanteiro GoncaloCanteiro marked this pull request as ready for review March 21, 2023 11:43
Copy link
Contributor

@nunocaseiro nunocaseiro left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@GoncaloCanteiro GoncaloCanteiro merged commit 331877a into main Mar 21, 2023
@GoncaloCanteiro GoncaloCanteiro deleted the test/communication-producers branch March 21, 2023 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[TEST]: communication producers

3 participants