Skip to content

Commit 9715039

Browse files
committed
Fixed double logger error (microsoft#25079)
## Description When merging this PR: [Here](https://github.com/microsoft/FluidFramework/pull/24880/files#diff-ed55acf5d71c17b285dadd6ba41c1438b6206ab9475ea32ac2b3e2b34d58c386) an error was merged that caused the pipeline to create two session loggers, this change will fix that error. ## Reviewer Guidance Let me know if there's something I should change or be aware of. Fixes: [AB#44795](https://dev.azure.com/fluidframework/235294da-091d-4c29-84fc-cdfc3d90890b/_workitems/edit/44795)
1 parent b0dfc2c commit 9715039

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

packages/test/mocha-test-setup/mocharc-common.cjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ function getFluidTestMochaConfig(packageDir, additionalRequiredModules, testRepo
4242
});
4343

4444
if (process.env.FLUID_TEST_LOGGER_PKG_SPECIFIER) {
45-
const modulePath = require.resolve(process.env.FLUID_TEST_LOGGER_PKG_SPECIFIER);
45+
const modulePath = path.join(moduleDir, process.env.FLUID_TEST_LOGGER_PKG_SPECIFIER);
4646
// Inject implementation of createTestLogger, put it first before mocha-test-setup
4747
if (existsSync(modulePath)) {
4848
requiredModulePaths.unshift(modulePath);

0 commit comments

Comments
 (0)