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
Update message producer to support PSB v4 #1
Conversation
codeliner
commented
Aug 30, 2015
- Package + classes are renamed to *message producer
- Classes are adjusted to use new prooph/common message handling
- Coveralls added
- Dependencies updated
- php-cs-fixer added
ping @prolic |
- Package + classes are renamed to *message producer - Classes are adjusted to use new prooph/common message handling - Coveralls added - Dependencies updated - php-cs-fixer added
c745ae8
to
88ae868
Compare
@@ -54,12 +54,14 @@ public function map(Envelope $envelope) | |||
{ | |||
$message = $envelope->getMessage(); | |||
|
|||
if (! $message instanceof BernardMessage) throw new \InvalidArgumentException(sprintf( | |||
if (! $message instanceof BernardMessage) { | |||
throw new \InvalidArgumentException(sprintf( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's use custom exceptions (Prooph\ServiceBus\Message\Bernard\Exception\InvalidArgumentException).
@prolic the producer lib now throws |
*/ | ||
public function useQueue($name) | ||
{ | ||
if (! is_string($name)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assert minLength, or does bernard allow empty strings as queue name?
@prolic Assertion for non empty queue name is added + 100% test coverage 😄 |
Update message producer to support PSB v4