Skip to content
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

DELETE /api/v1/lesoes/:id_lesoes(\d+) isn't implemented #177

Open
rgaiacs opened this issue Aug 19, 2020 · 2 comments
Open

DELETE /api/v1/lesoes/:id_lesoes(\d+) isn't implemented #177

rgaiacs opened this issue Aug 19, 2020 · 2 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@rgaiacs
Copy link
Contributor

rgaiacs commented Aug 19, 2020

  ● DELETE /api/v1/lesoes/:id_lesoes(\d+) › anonymous
    expect(received).toBe(expected) // Object.is equality
    Expected: 401
    Received: 501
      762 |                     .then(
      763 |                         response => {
    > 764 |                             expect(response.statusCode).toBe(HttpStatus.UNAUTHORIZED);
          |                                                         ^
      765 |                         }
      766 |                     );
      767 |             }
      at src/__tests__/injury.js:764:57
          at runMicrotasks (<anonymous>)
  ● DELETE /api/v1/lesoes/:id_lesoes(\d+) › charles
    expect(received).toBe(expected) // Object.is equality
    Expected: 403
    Received: 501
      779 |                     .then(
      780 |                         response => {
    > 781 |                             expect(response.statusCode).toBe(HttpStatus.FORBIDDEN);
          |                                                         ^
      782 |                         }
      783 |                     );
      784 |             }
      at src/__tests__/injury.js:781:57
          at runMicrotasks (<anonymous>)
  ● DELETE /api/v1/lesoes/:id_lesoes(\d+) › admin
    expect(received).toBe(expected) // Object.is equality
    Expected: 204
    Received: 501
      796 |                     .then(
      797 |                         response => {
    > 798 |                             expect(response.statusCode).toBe(HttpStatus.NO_CONTENT);
          |                                                         ^
      799 |                         }
      800 |                     );
      801 |             }
      at src/__tests__/injury.js:798:57
          at runMicrotasks (<anonymous>)
@rgaiacs rgaiacs added the enhancement New feature or request label Aug 19, 2020
@rgaiacs rgaiacs added this to the v2.6.0 milestone Aug 19, 2020
@rgaiacs rgaiacs added the good first issue Good for newcomers label Aug 19, 2020
@rgaiacs
Copy link
Contributor Author

rgaiacs commented Aug 24, 2020

@agcbianchi what should be done with image and cells that are associate with the deleted injury?

@andrea-g-campos
Copy link
Contributor

Keep them in a directory with a date for 1 month.
So after 1 mont, someone may delete them if they were not used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants