[Enhancement] Swap these 2 arguments so they are in the correct order.[ClientConfigurationTest] #4101
Closed
Description
Search before asking
- I had searched in the issues and found no similar issues.
Enhancement Request
located at:
eventmesh-storage-plugin/eventmesh-storage-knative/src/test/java/org/apache/eventmesh/storage/knative/config/ClientConfigurationTest.java line 48-49
analysis and explanation:
The standard assertions library methods such as org.junit.Assert.assertEquals, and org.junit.Assert.assertSame expect the first argument to be the expected value and the second argument to be the actual value. For AssertJ, it’s the other way around, the argument of org.assertj.core.api.Assertions.assertThat is the actual value, and the subsequent calls contain the expected values. Swap them, and your test will still have the same outcome (succeed/fail when it should) but the error messages will be confusing.
Describe the solution you'd like
swap these two parameters.
Are you willing to submit PR?
- Yes I am willing to submit a PR!