[Feature] Add testFilePathName to snapshotPathTemplate (without extension)

[Xtravaganz]

export default defineConfig({
	
	// test files
	testMatch: '/__tests__/*.specs.js',

}).extend(

        // for each test file a specific SnapshotPathTemplate
	overrideSnapshotPathTemplate({
		'/__tests__/testUser1.specs.js': {
			snapshotPathTemplate: './__tests__/testUser1/{snapshotSuffix}/{arg}{ext}',
		},
		'/__tests__/testUser2.specs.js': {
			snapshotPathTemplate: './__tests__/testUser2/{snapshotSuffix}/{arg}{ext}',
		},
	})
);

Such a configuration would be desirable.
Of course it may be possible in other ways?
Unfortunately I havn't found anything in the documentation so far.

[mxschmitt]

Why does using './__tests__/{testFilePath}/{snapshotSuffix}/{arg}{ext}' not work for you as a project specific snapshot path template?

This would then result in two folders for testUser1 and testUser2.

[Xtravaganz]

Is there a way to remove the file extension from the testFilePath?

Since we run tests on different systems, Windows would run into an error that this file already exists, the possibility as in Linux is unfortunately missing here (dir/file same name).

Of course, that would be a solution I could live with. 👍

[Xtravaganz]

test.beforeEach(async({ browser }, testInfo) => {
}

Is there a possibility via the testInfo object?

[mxschmitt]

You can manually override the snapshotSuffix inside beforeEach. See e.g. this:

import { test, expect } from '@playwright/test';

test('basic test', async ({ page }) => {
  test.info().snapshotSuffix = "abc";
});

[Xtravaganz]

You can manually override the snapshotSuffix inside beforeEach. See e.g. this:

import { test, expect } from '@playwright/test';

test('basic test', async ({ page }) => {
  test.info().snapshotSuffix = "abc";
});

The suffix doesn't matter to me, I need to get the .js extension removed from the testFilePath so that everything works cleanly under Linux/Mac/Win

windows problem:

window test error:
Error: ENOENT: no such file or directory, open 'C:\repo\project\__tests__\admin.specs.js\win32\mainpage.png'

The problematic directory name cannot be created in Windows -> C:\repo\project____tests__\admin.specs.js\.
File C:\repo\project__tests__\admin.specs.js exists!

You have a solution for it?

[mxschmitt]

This is not possible as of today. We can make a feature request out of it, usually we recommend putting screenshots into e.g. a __screenshots__ directory see here.

[Xtravaganz]

This is not possible as of today. We can make a feature request out of it, usually we recommend putting screenshots into e.g. a __screenshots__ directory see here.

Then I would clearly be for the feature request! 😃

[Xtravaganz]

Is there an approximate schedule for such requests and their processing?

[mxschmitt]

Would './__tests__/{testFilePath}-snapshots/{snapshotSuffix}/{arg}{ext}' work for you in the meantime?

We usually prioritise based on how many customers want it.

[Xtravaganz]

Would './__tests__/{testFilePath}-snapshots/{snapshotSuffix}/{arg}{ext}' work for you in the meantime?

We usually prioritise based on how many customers want it.

As an intermediate step it works like this for now, but of course this makes the folder name unnecessarily long.