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

UpdateTasksController/Logging #73

Closed
magwas opened this issue Mar 5, 2020 · 3 comments
Closed

UpdateTasksController/Logging #73

magwas opened this issue Mar 5, 2020 · 3 comments

Comments

@magwas
Copy link
Contributor

magwas commented Mar 5, 2020

Behaviour: UpdateTasksController/Logging

Annotations for the test cases:

@TestedBehaviour("Logging")
@TestedService("UpdateTasksController")

The production code is at com.kodekonveyor.market.tasks.UpdateTasksController.java
The testcase base should be at com.kodekonveyor.market.tasks.UpdateTasksControllerTestBase.java
The testcase should probably be at com.kodekonveyor.market.tasks.UpdateTasksControllerLoggingTest.java

  • All log entries should contain the identifier of the user and the (attempted) operation.
  • All activities should be logged at start of the operation.
  • All failed operations should be logged with the reason of the failure
  • All successful operations should be logged with reference to the accessed data

The service:

UpdateTasksController
UpdateTasksController

DTOs and Entities:

TaskDTO, TaskListDTO, TaskEntity, TaskStatusEnum
TaskDTO,  TaskListDTO, TaskEntity, TaskStatusEnum

If you have questions, see the FAQ, or ask on the development slack channel or ask your mentor.

@pthakkar02
Copy link
Contributor

Grab

@szirbucz
Copy link
Collaborator

@pthakkar02 We are planning to refactor the logging in this project. Please choose if you finish this task in 2-3 days or do it after the refactor. Have you already started the work on it?

pthakkar02 added a commit that referenced this issue Mar 19, 2020
#73 UpdateTasksController/Logging
@magwas
Copy link
Contributor Author

magwas commented Apr 7, 2020

this task is no longer necessary. Please discontinue developing it, and issue a pull request for anything you have done so far. we pay for any testcases you have already created.

Please comment when ready.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants