Skip to content

Event logger

Afonso Mukai requested to merge event-logger into master

Created by: michele-brambilla

Description of work

Notify relevant events in the status topic that NICOS can interpret and report to the user. The log codes must match what NICOS expect.

Acceptance Criteria

The log events must be when they are expected correctly and with the correct code. The logEvent function must be accessible from any component of the file writer.

Unit Tests

Add 'EventLogger' test to check the fields in the log message and the log codes.

Code Review (To be filled in by the reviewer only)

  • Is the code of an acceptable quality?
  • Do the changes function as described and is it robust?

Nominate for Group Code Review (Anyone can nominate it)

Indicate if you think the code should be reviewed in a Thursday code review session.

  • Recommend for group code review

Also, nominate it on the code_review Slack channel (does someone want to automate this?).

Merge request reports