feat(NODE-4814): implement remaining severity logging methods

[W-A-James]

Description

What is changing?

Adds in the remainder of the severity loggers whose base designs were implemented in NODE-5169

Is there new documentation needed for these changes?

No

Double check the following

• Ran npm run check:lint script
• Self-review completed using the steps outlined here
• PR title follows the correct format: type(NODE-xxxx)[!]: description
  • Example: feat(NODE-1234)!: rewriting everything in coffeescript
• Changes are covered by tests
• New TODOs have a related JIRA ticket

[W-A-James]

Question: Do we want to remove the entries in the SeverityLevel enum that correspond to the severities that we are not using?

These would be:

• emergency
• alert
• critical
• notice

[W-A-James]

Question: Do we want to have the helper that corresponds with the warn severity be named warn or warning? Doing the former allows for easy accessing on the logger object by using the severity name directly as a property, so I skew towards that approach.

[baileympearson]

Question: Do we want to have the helper that corresponds with the warn severity be named warn or warning? Doing the former allows for easy accessing on the logger object by using the severity name directly as a property, so I skew towards that approach.

Agreed. I also think that these methods make the most sense in the verb form (i.e., "warn" instead of "warning")

Question: Do we want to remove the entries in the SeverityLevel enum that correspond to the severities that we are not using?
These would be:

• emergency
• alert
• critical
• notice

I lean towards leaving all severity levels, for consistency with the spec. If we TS support to help enforce the lack of helpers for emergency, alert, critical, notice, we could consider defining severity level something like this? This would constrain severity level to only allow supported severity levels. Not sure if it's worthwhile though.

export type SeverityLevel = (typeof SeverityLevel)[keyof typeof SeverityLevel] &	('log' | 'debug' .... ); 

[baileympearson]

Tiny change

[baileympearson]

if we type severities as SeverityLevel[], we should be able to remove the cast in line 683

[baileympearson]

CI failures are unrelated.