Cyress e2e plugin generates _eslintrc.js instead of .eslintrc.js

[dobromir-hristov]

Version

3.0.1

Node and OS info

node v9.3.0 / npm 6.1.0 / Windows 10

Steps to reproduce

Start a new app with cli.
Select Cypress as e2e testing framework

What is expected?

installation finishes without errors

What is actually happening?

Lint rules are failing
https://gyazo.com/04e0ba22ffa2617e0d0e816623419ded

---

The _eslintrc.js file inside the templates folder of the cypress plugin is not being transformed to .eslintrc.js as per docs, so it just gets copied with the underscore in the name.

I can make a PR and rename it, but I am not sure that's the proper thing to do, as per the docs at least.

[Akryum]

Can't reproduce on Node 10.8.0/linux

[dobromir-hristov]

I will do more tests later today. Going to make a custom local plugin to check if it's OS dependent or something.

[Akryum]

Did you try with an more recent version? BTW, node 9.x shouldn't really be used, it's a development branch.

[dobromir-hristov]

OK so I tried again, updated node, same results. However, when creating custom plugin, with a file with underscore as first char, I dont see the problem.

Versions:
Windows 10
node 10.9.0
npm 6.1.0
vue-cli 3.0.1

[dobromir-hristov]

I played around with creating custom plugins, cloning the cli locally. What I gathered is this:

cli-plugin-whatever/generator/templates/_file.js gets rendered as .file.js top level in the project.
cli-plugin-whatever/generator/templates/tests/e2e/_file.js gets rendered as _file.js in tests/e2e inside the project.

This is both with custom plugins and augmenting the shipped cypress one.

Should I delve deeper into this? I will try tomorrow with a fresh Windows 10 install just in case.

[dobromir-hristov]

I think I found the culprit. Globby returns paths with forward slash on Windows, i.e. tests/e2e/_eslintrc.js, but we are splitting the target path by the OS separator defined in path.sep, thus by \.

vue-cli/packages/@vue/cli/lib/GeneratorAPI.js

Lines 170 to 172 in 5efbd1b

	const _files = await globby(['**/*'], { cwd: source })
	for (const rawPath of _files) {
	const targetPath = rawPath.split(path.sep).map(filename => {

Top level files work fine, because they start with underscore, _super_duper_cypress.js, thus matching the first char works.

vue-cli/packages/@vue/cli/lib/GeneratorAPI.js

Line 175 in 5efbd1b

if (filename.charAt(0) === '_' && filename.charAt(1) !== '_') {

Should I test other globbing libs, to see if they return paths based on OS? Or we can safely assume that we can split by forward slash / because of globby's implementation and call it a day?

[LinusBorg]

Thanks for diggign into this!

Or we can safely assume that we can split by forward slash / because of globby's implementation and call it a day?

I think we should do that. Can you send a PR?

[dobromir-hristov]

Sure :)