archive-gh-me/log4js-node
1
0
Fork 0
mirror of https://github.com/log4js-node/log4js-node.git synced 2025-12-08 19:26:01 +00:00
Code Issues Projects Releases Wiki Activity

refactor(test): replaced tap.tearDown() (deprecated) to tap.teardown() (backward-compatible) in preparation for tap v15

Browse Source 
https://github.com/tapjs/node-tap/pull/649/files
This commit is contained in:
Lam Wei Li 2022-01-25 16:12:40 +08:00
parent 461f2fbe8c
commit 166f2252f7
No known key found for this signature in database
GPG Key ID: 90F6ABECF080D7BF
9 changed files with 32 additions and 32 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
test/tap/configuration-validation-test.js
View File

@ -256,7 +256,7 @@ test("log4js configuration validation", batch => {
batch.test("should not throw error if configure object is freezed", t => {
const testFile = "test/tap/freeze-date-file-test";
t.tearDown(() => {
t.teardown(() => {
removeFiles(testFile);
});
t.doesNotThrow(() =>

8
test/tap/dateFileAppender-test.js
View File

@ -25,7 +25,7 @@ test("../../lib/appenders/dateFile", batch => {
const logger = log4js.getLogger("default-settings");
logger.info("This should be in the file.");
t.tearDown(() => {
t.teardown(() => {
removeFile("date-appender-default.log");
});
@ -72,7 +72,7 @@ test("../../lib/appenders/dateFile", batch => {
);
});
t.tearDown(() => {
t.teardown(() => {
removeFile("date-file-test.log");
});
});
@ -108,7 +108,7 @@ test("../../lib/appenders/dateFile", batch => {
const logger = log4js.getLogger("tests");
logger.warn("this should be written to the file with the appended date");
t.tearDown(() => {
t.teardown(() => {
removeFile(testFile);
});
@ -140,7 +140,7 @@ test("../../lib/appenders/dateFile", batch => {
logger.info("1");
logger.info("2");
logger.info("3");
t.tearDown(() => {
t.teardown(() => {
removeFile("date-appender-flush.log");
});

2
test/tap/file-descriptor-leak-test.js
View File

@ -67,7 +67,7 @@ if (process.platform !== "win32") {
}, 250);
});
batch.tearDown(async () => {
batch.teardown(async () => {
log4js.shutdown();
const filenames = Object.values(config.appenders).map(appender => appender.filename);

4
test/tap/file-sighup-test.js
View File

@ -38,7 +38,7 @@ test("file appender single SIGHUP handler", t => {
const log4js = require("../../lib/log4js");
log4js.configure(config);
t.tearDown(async () => {
t.teardown(async () => {
log4js.shutdown();
const filenames = Object.values(config.appenders).map(appender => appender.filename);
@ -120,7 +120,7 @@ test("file appender SIGHUP handler leak", t => {
},
categories: { default: { appenders: ["file"], level: "info" } }
});
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles("test.log");
});
t.plan(2);

14
test/tap/fileAppender-test.js
View File

@ -24,7 +24,7 @@ test("log4js fileAppender", batch => {
const logger = log4js.getLogger("default-settings");
await removeFile(testFile);
t.tearDown(async () => {
t.teardown(async () => {
await new Promise(resolve => log4js.shutdown(resolve));
await removeFile(testFile);
});
@ -50,7 +50,7 @@ test("log4js fileAppender", batch => {
const testFile = path.join(__dirname, "fa-default-test.log");
await removeFile(testFile);
t.tearDown(async () => {
t.teardown(async () => {
await new Promise(resolve => log4js.shutdown(resolve));
await removeFile(testFile);
});
@ -80,7 +80,7 @@ test("log4js fileAppender", batch => {
const testFile = path.join(__dirname, "fa-maxFileSize-test.log");
const logger = log4js.getLogger("max-file-size");
t.tearDown(async () => {
t.teardown(async () => {
await new Promise(resolve => log4js.shutdown(resolve));
await removeFile(testFile);
});
@ -121,7 +121,7 @@ test("log4js fileAppender", batch => {
const testFile = path.join(__dirname, "fa-maxFileSize-unit-test.log");
const logger = log4js.getLogger("max-file-size-unit");
t.tearDown(async () => {
t.teardown(async () => {
await new Promise(resolve => log4js.shutdown(resolve));
await Promise.all([removeFile(testFile), removeFile(`${testFile}.1`)]);
});
@ -174,7 +174,7 @@ test("log4js fileAppender", batch => {
removeFile(`${testFile}.2`)
]);
t.tearDown(async () => {
t.teardown(async () => {
await new Promise(resolve => log4js.shutdown(resolve));
await Promise.all([
removeFile(testFile),
@ -234,7 +234,7 @@ test("log4js fileAppender", batch => {
removeFile(`${testFile}.2.gz`)
]);
t.tearDown(async () => {
t.teardown(async () => {
await new Promise(resolve => log4js.shutdown(resolve));
await Promise.all([
removeFile(testFile),
@ -350,7 +350,7 @@ test("log4js fileAppender", batch => {
await removeFile(testFilePlain);
await removeFile(testFileAsIs);
t.tearDown(async () => {
t.teardown(async () => {
await new Promise(resolve => log4js.shutdown(resolve));
await removeFile(testFilePlain);
await removeFile(testFileAsIs);

12
test/tap/fileSyncAppender-test.js
View File

@ -18,7 +18,7 @@ test("log4js fileSyncAppender", batch => {
const logger = log4js.getLogger("default-settings");
remove(testFile);
t.tearDown(() => {
t.teardown(() => {
remove(testFile);
});
@ -44,7 +44,7 @@ test("log4js fileSyncAppender", batch => {
const logger = log4js.getLogger("max-file-size");
remove(testFile);
t.tearDown(() => {
t.teardown(() => {
remove(testFile);
});
@ -93,7 +93,7 @@ test("log4js fileSyncAppender", batch => {
remove(testFile);
remove(`${testFile}.1`);
t.tearDown(() => {
t.teardown(() => {
remove(testFile);
remove(`${testFile}.1`);
});
@ -148,7 +148,7 @@ test("log4js fileSyncAppender", batch => {
remove(`${testFile}.1`);
remove(`${testFile}.2`);
t.tearDown(() => {
t.teardown(() => {
remove(testFile);
remove(`${testFile}.1`);
remove(`${testFile}.2`);
@ -218,7 +218,7 @@ test("log4js fileSyncAppender", batch => {
const testFile = "tmp-sync-tests.log";
remove(testFile);
t.tearDown(() => {
t.teardown(() => {
remove(testFile);
});
@ -252,7 +252,7 @@ test("log4js fileSyncAppender", batch => {
const testFile = "tmp-options-tests.log";
remove(testFile);
t.tearDown(() => {
t.teardown(() => {
remove(testFile);
});

2
test/tap/logLevelFilter-test.js
View File

@ -58,7 +58,7 @@ test("log4js logLevelFilter", batch => {
remove(`${__dirname}/logLevelFilter-warnings.log`);
remove(`${__dirname}/logLevelFilter-debugs.log`);
t.tearDown(() => {
t.teardown(() => {
remove(`${__dirname}/logLevelFilter.log`);
remove(`${__dirname}/logLevelFilter-warnings.log`);
remove(`${__dirname}/logLevelFilter-debugs.log`);

14
test/tap/multi-file-appender-test.js
View File

@ -15,7 +15,7 @@ test("multiFile appender", batch => {
batch.test(
"should write to multiple files based on the loggingEvent property",
t => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles(["logs/A.log", "logs/B.log"]);
});
log4js.configure({
@ -44,7 +44,7 @@ test("multiFile appender", batch => {
batch.test(
"should write to multiple files based on loggingEvent.context properties",
t => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles(["logs/C.log", "logs/D.log"]);
});
log4js.configure({
@ -73,7 +73,7 @@ test("multiFile appender", batch => {
);
batch.test("should close file after timeout", t => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles("logs/C.log");
});
/* checking that the file is closed after a timeout is done by looking at the debug logs
@ -120,7 +120,7 @@ test("multiFile appender", batch => {
batch.test(
"should fail silently if loggingEvent property has no value",
t => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles("logs/E.log");
});
log4js.configure({
@ -152,7 +152,7 @@ test("multiFile appender", batch => {
);
batch.test("should pass options to rolling file stream", t => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles(["logs/F.log", "logs/F.log.1", "logs/F.log.2"]);
});
log4js.configure({
@ -186,7 +186,7 @@ test("multiFile appender", batch => {
});
batch.test("should inherit config from category hierarchy", t => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles("logs/test.someTest.log");
});
log4js.configure({
@ -236,7 +236,7 @@ test("multiFile appender", batch => {
});
});
batch.tearDown(async () => {
batch.teardown(async () => {
try {
const files = fs.readdirSync("logs");
await removeFiles(files.map(filename => `logs/${filename}`));

6
test/tap/pause-test.js
View File

@ -12,7 +12,7 @@ const removeFiles = async filenames => {
tap.test("Drain event test", batch => {
batch.test("Should emit pause event and resume when logging in a file with high frequency", t => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles("logs/drain.log");
});
// Generate logger with 5k of highWaterMark config
@ -47,7 +47,7 @@ tap.test("Drain event test", batch => {
batch.test("Should emit pause event and resume when logging in a date file with high frequency", (t) => {
t.tearDown(async () => {
t.teardown(async () => {
await removeFiles("logs/date-file-drain.log");
});
// Generate date file logger with 5kb of highWaterMark config
@ -79,7 +79,7 @@ tap.test("Drain event test", batch => {
t.end();
});
batch.tearDown(async () => {
batch.teardown(async () => {
try {
const files = fs.readdirSync("logs");
await removeFiles(files.map(filename => `logs/${filename}`));