Skip to content

Commit

Permalink
PR feedback
Browse files Browse the repository at this point in the history
  • Loading branch information
asanchezr committed Oct 4, 2023
1 parent ab1acf9 commit 4270c53
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 17 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -42,7 +42,7 @@ public PropertyManagementController(IPropertyService propertyService, IMapper ma
/// </summary>
/// <returns></returns>
[HttpGet("{propertyId}/management")]
[HasPermission(Permissions.PropertyView)]
[HasPermission(Permissions.ManagementView)]
[Produces("application/json")]
[ProducesResponseType(typeof(PropertyManagementModel), 200)]
[SwaggerOperation(Tags = new[] { "property" })]
Expand All @@ -57,7 +57,7 @@ public IActionResult GetPropertyManagement(long propertyId)
/// </summary>
/// <returns></returns>
[HttpPut("{propertyId}/management")]
[HasPermission(Permissions.PropertyEdit)]
[HasPermission(Permissions.ManagementEdit)]
[Produces("application/json")]
[ProducesResponseType(typeof(PropertyManagementModel), 200)]
[SwaggerOperation(Tags = new[] { "property" })]
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -79,11 +79,6 @@ describe('PropertyManagementUpdateContainer component', () => {
jest.clearAllMocks();
});

it('renders as expected', () => {
const { asFragment } = setup();
expect(asFragment()).toMatchSnapshot();
});

it('fetches property management info from the api', () => {
mockGetApi.execute.mockResolvedValue(getMockApiPropertyManagement(1));
setup({ props: { propertyId: 1 } });
Expand Down

This file was deleted.

0 comments on commit 4270c53

Please sign in to comment.