-
Notifications
You must be signed in to change notification settings - Fork 3
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
Add Shuttering Implementation #22
Conversation
d6f4025
to
eae2837
Compare
@@ -16,6 +16,7 @@ | |||
import static uk.gov.justice.services.example.cakeshop.it.params.CakeShopUris.RECIPES_RESOURCE_QUERY_URI; | |||
import static uk.gov.justice.services.example.cakeshop.it.params.CakeShopUris.RECIPES_RESOURCE_URI; | |||
|
|||
import org.junit.*; |
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.
the .* import needs changing, maybe the intellij code style is not set
@@ -9,17 +9,13 @@ | |||
import static org.hamcrest.core.IsNull.nullValue; | |||
import static uk.gov.justice.services.example.cakeshop.it.params.CakeShopUris.ORDERS_RESOURCE_URI; | |||
|
|||
import org.junit.*; |
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.
the .* import needs changing
@@ -10,6 +10,7 @@ | |||
import static uk.gov.justice.services.example.cakeshop.it.params.CakeShopMediaTypes.CONTEXT_NAME; | |||
import static uk.gov.justice.services.example.cakeshop.it.params.CakeShopUris.RECIPES_RESOURCE_URI; | |||
|
|||
import org.junit.*; |
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.
the .* import needs changing
@@ -7,6 +7,7 @@ | |||
import static org.junit.Assert.assertNull; | |||
import static uk.gov.justice.services.example.cakeshop.it.params.CakeShopUris.RECIPES_RESOURCE_URI; | |||
|
|||
import org.junit.*; |
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.
the .* import needs changing
@@ -17,6 +17,7 @@ | |||
import static uk.gov.justice.services.example.cakeshop.it.params.CakeShopUris.ORDERS_RESOURCE_URI; | |||
import static uk.gov.justice.services.test.utils.core.matchers.HttpStatusCodeMatcher.isStatus; | |||
|
|||
import org.junit.*; |
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.
the .* import needs changing
@@ -6,6 +6,7 @@ | |||
import static org.hamcrest.core.IsCollectionContaining.hasItem; | |||
import static uk.gov.justice.services.test.utils.core.reflection.ReflectionUtil.setField; | |||
|
|||
import org.junit.*; |
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.
the .* import needs changing
.assertThat("$.recipes[?(@.id=='" + recipeId + "')].name", hasItem(recipeName)) | ||
.assertThat("$.recipes[?(@.id=='" + recipeId2 + "')].name", hasItem(recipeName2)); | ||
} | ||
else{ |
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.
"else" looks better if on the same line as the } above it
if(isShutteringRequired){ | ||
mBeanHelper.getMbeanProxy(connection, objectName, ShutteringMBean.class).doShutteringRequested(); | ||
} | ||
else{ |
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.
"else" looks better if on the same line as the } above it
c43d616
to
39e01c9
Compare
39e01c9
to
2cbe958
Compare
No description provided.