editorconfig: Add test for finding VCS directory

josephfrazier · josephfrazier · commit d9b244781896 · 2017-12-26T00:47:05.000-05:00
It's a little hacky in that the .hg file isn't really a Mercurial repository, but it's enough to illustrate the intent. See here for context: #3559 (comment)
diff --git a/tests_integration/__tests__/__snapshots__/config-resolution.js.snap b/tests_integration/__tests__/__snapshots__/config-resolution.js.snap
@@ -18,6 +18,11 @@ function f() {
                 \\"should have space width 8\\"
         )
 }
+function f() {
+  console.log(
+    \\"should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present\\"
+  )
+}
 console.log(
   \\"jest/__best-tests__/file.js should have semi\\"
 );
@@ -108,6 +113,11 @@ function f() {
 function f() {
         console.log(\\"should have space width 8\\")
 }
+function f() {
+  console.log(
+    \\"should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present\\"
+  )
+}
 console.log(\\"jest/__best-tests__/file.js should have semi\\");
 console.log(\\"jest/Component.js should not have semi\\")
 console.log(\\"jest/Component.test.js should have semi\\");
diff --git a/tests_integration/__tests__/__snapshots__/with-config-precedence.js.snap b/tests_integration/__tests__/__snapshots__/with-config-precedence.js.snap
@@ -95,6 +95,11 @@ function f() {
                 \\"should have space width 8\\"
         )
 }
+function f() {
+  console.log(
+    \\"should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present\\"
+  )
+}
 console.log(
   \\"jest/__best-tests__/file.js should have semi\\"
 );
@@ -167,6 +172,11 @@ function f() {
                 \\"should have space width 8\\"
         )
 }
+function f() {
+  console.log(
+    \\"should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present\\"
+  )
+}
 console.log(
   \\"jest/__best-tests__/file.js should have semi\\"
 );
diff --git a/tests_integration/cli/config/editorconfig/repo-root/.hg b/tests_integration/cli/config/editorconfig/repo-root/.hg
@@ -0,0 +1,2 @@
+This isn't really a Mercurial repo, but we want to pretend it is for testing purposes.
+See https://github.com/prettier/prettier/pull/3559#issuecomment-353857109
diff --git a/tests_integration/cli/config/editorconfig/repo-root/file.js b/tests_integration/cli/config/editorconfig/repo-root/file.js
@@ -0,0 +1,3 @@
+function f() {
+  console.log("should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present");
+}

Original file line number	Diff line number	Diff line change
`@@ -95,6 +95,11 @@ function f() {`
`95`	`95`	`\\"should have space width 8\\"`
`96`	`96`	`)`
`97`	`97`	`}`
	`98`	`+function f() {`
	`99`	`+ console.log(`
	`100`	`+ \\"should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present\\"`
	`101`	`+ )`
	`102`	`+}`
`98`	`103`	`console.log(`
`99`	`104`	`\\"jest/__best-tests__/file.js should have semi\\"`
`100`	`105`	`);`
`@@ -167,6 +172,11 @@ function f() {`
`167`	`172`	`\\"should have space width 8\\"`
`168`	`173`	`)`
`169`	`174`	`}`
	`175`	`+function f() {`
	`176`	`+ console.log(`
	`177`	`+ \\"should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present\\"`
	`178`	`+ )`
	`179`	`+}`
`170`	`180`	`console.log(`
`171`	`181`	`\\"jest/__best-tests__/file.js should have semi\\"`
`172`	`182`	`);`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+This isn't really a Mercurial repo, but we want to pretend it is for testing purposes.`
	`2`	`+See https://github.com/prettier/prettier/pull/3559#issuecomment-353857109`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+function f() {`
	`2`	`+ console.log("should have space width 2 despite ../.editorconfig specifying 8, because ./.hg is present");`
	`3`	`+}`