Skip to content

Commit bfe97f1

Browse files
committed
chore(logger): adhere to tests standard
1 parent 99561a0 commit bfe97f1

File tree

1 file changed

+18
-12
lines changed

1 file changed

+18
-12
lines changed
Lines changed: 18 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,38 +1,44 @@
11
import { Logger } from '../../src/Logger.js';
2-
import { describe, it, expect, vi } from 'vitest';
2+
import { describe, it, expect, vi, afterEach } from 'vitest';
3+
4+
describe('Logger PowertoolsLogData environment variables', () => {
5+
afterEach(() => {
6+
vi.unstubAllEnvs();
7+
});
38

4-
describe('Logger PowertoolsLogData AWS_REGION', () => {
59
it.each([
610
[undefined, ''],
711
['us-west-2', 'us-west-2'],
8-
])('should set awsRegion to %s when AWS_REGION env var is %s', (envValue, expected) => {
12+
])('sets awsRegion to %s when AWS_REGION env var is %s', (envValue, expected) => {
13+
// Prepare
914
vi.stubEnv('AWS_REGION', envValue);
15+
// Act
1016
const logger = new Logger();
17+
// Assess
1118
expect(logger['powertoolsLogData'].awsRegion).toBe(expected);
12-
vi.unstubAllEnvs();
1319
});
14-
});
1520

16-
describe('Logger PowertoolsLogData ENVIRONMENT', () => {
1721
it.each([
1822
[undefined, ''],
1923
['prd', 'prd'],
20-
])('should set environment to %s when ENVIRONMENT env var is %s', (envValue, expected) => {
24+
])('sets environment to %s when ENVIRONMENT env var is %s', (envValue, expected) => {
25+
// Prepare
2126
vi.stubEnv('ENVIRONMENT', envValue);
27+
// Act
2228
const logger = new Logger();
29+
// Assess
2330
expect(logger['powertoolsLogData'].environment).toBe(expected);
24-
vi.unstubAllEnvs();
2531
});
26-
});
2732

28-
describe('Logger PowertoolsLogData POWERTOOLS_SERVICE_NAME', () => {
2933
it.each([
3034
[undefined, 'service_undefined'],
3135
['my-service', 'my-service'],
32-
])('should set serviceName to %s when POWERTOOLS_SERVICE_NAME env var is %s', (envValue, expected) => {
36+
])('sets serviceName to %s when POWERTOOLS_SERVICE_NAME env var is %s', (envValue, expected) => {
37+
// Prepare
3338
vi.stubEnv('POWERTOOLS_SERVICE_NAME', envValue);
39+
// Act
3440
const logger = new Logger();
41+
// Assess
3542
expect(logger['powertoolsLogData'].serviceName).toBe(expected);
36-
vi.unstubAllEnvs();
3743
});
3844
});

0 commit comments

Comments
 (0)