Skip to content

Supporting method in SimpleBrokerMessageHandlerTests not asserting anything [SPR-12615] #17216

@spring-projects-issues

Description

@spring-projects-issues

Ralph Schaer opened SPR-12615 and commented

The return value of the method org.springframework.messaging.simp.broker.SimpleBrokerMessageHandlerTests#assertCapturedMessage is never checked.
So it never asserts the captured messages

There are two wrong expected assert messages (assertEquals("Expected .....) in this unit test
org.springframework.messaging.simp.broker.DefaultSubscriptionRegistryTests

Line 269: Should be "two" instead of "three"
Line 313: Should be "one" instead of "three"


Affects: 4.1.4

Reference URL: https://github.com/spring-projects/spring-framework/blob/master/spring-messaging/src/test/java/org/springframework/messaging/simp/broker/SimpleBrokerMessageHandlerTests.java

Referenced from: commits 497944f, 84807c5

Metadata

Metadata

Assignees

Labels

in: messagingIssues in messaging modules (jms, messaging)type: taskA general task

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions