Skip to content

Commit

Permalink
chore(test): renamed tap.teardown() to tap.tearDown() for consistency…
Browse files Browse the repository at this point in the history
… (while both works, only tap.tearDown() is documented)
  • Loading branch information
lamweili committed Jan 19, 2022
1 parent a0baec2 commit 8cba85f
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 8 deletions.
2 changes: 1 addition & 1 deletion test/tap/configuration-validation-test.js
Expand Up @@ -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(() =>
Expand Down
8 changes: 4 additions & 4 deletions test/tap/dateFileAppender-test.js
Expand Up @@ -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");
});

Expand Down Expand Up @@ -72,7 +72,7 @@ test("../../lib/appenders/dateFile", batch => {
);
});

t.teardown(() => {
t.tearDown(() => {
removeFile("date-file-test.log");
});
});
Expand Down Expand Up @@ -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);
});

Expand Down Expand Up @@ -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");
});

Expand Down
2 changes: 1 addition & 1 deletion test/tap/file-descriptor-leak-test.js
Expand Up @@ -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);
Expand Down
4 changes: 2 additions & 2 deletions test/tap/file-sighup-test.js
Expand Up @@ -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);
Expand Down Expand Up @@ -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);
Expand Down

0 comments on commit 8cba85f

Please sign in to comment.