test: order configuration files

ematipico · ematipico · commit 0f651422834b · 2026-06-29T10:06:58.000+01:00
diff --git a/crates/biome_cli/tests/snap_test.rs b/crates/biome_cli/tests/snap_test.rs
@@ -42,7 +42,7 @@ pub(crate) struct CliSnapshot {
     /// the configuration, if set
     /// First string is the content
     /// Second string is the name
-    pub configuration_list: Vec<(String, String)>,
+    pub configuration_list: BTreeMap<String, String>,
     /// file name -> content
     pub files: BTreeMap<String, String>,
     /// messages written in console
@@ -55,7 +55,7 @@ impl CliSnapshot {
     pub fn from_result(result: Result<(), CliDiagnostic>) -> Self {
         Self {
             in_messages: InMessages::default(),
-            configuration_list: vec![],
+            configuration_list: BTreeMap::default(),
             files: BTreeMap::default(),
             messages: Vec::new(),
             termination: result.err().map(Error::from),
@@ -459,7 +459,7 @@ impl From<SnapshotPayload<'_>> for CliSnapshot {
             {
                 cli_snapshot
                     .configuration_list
-                    .push((content.to_string(), file.to_string()));
+                    .insert(content.to_string(), file.to_string());
             } else {
                 cli_snapshot
                     .files
diff --git a/crates/biome_cli/tests/snapshots/main_cases_config_extends/extends_config_ok_from_npm_package.snap b/crates/biome_cli/tests/snapshots/main_cases_config_extends/extends_config_ok_from_npm_package.snap
@@ -2,22 +2,22 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `node_modules/@shared/format/biome.json`
+## `biome.json`
 
 ```json
-{ "javascript": { "formatter": { "quoteStyle": "single" } } }
+{ "extends": ["@shared/format/biome", "@shared/linter/biome"] }
 ```
 
-## `node_modules/@shared/linter/biome.jsonc`
+## `node_modules/@shared/format/biome.json`
 
 ```json
-{ "linter": { "enabled": false } }
+{ "javascript": { "formatter": { "quoteStyle": "single" } } }
 ```
 
-## `biome.json`
+## `node_modules/@shared/linter/biome.jsonc`
 
 ```json
-{ "extends": ["@shared/format/biome", "@shared/linter/biome"] }
+{ "linter": { "enabled": false } }
 ```
 
 ## `node_modules/@shared/format/package.json`
diff --git a/crates/biome_cli/tests/snapshots/main_cases_config_extends/extends_config_ok_from_npm_package_with_author_field.snap b/crates/biome_cli/tests/snapshots/main_cases_config_extends/extends_config_ok_from_npm_package_with_author_field.snap
@@ -2,22 +2,22 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `node_modules/@shared/format/biome.json`
+## `biome.json`
 
 ```json
-{ "javascript": { "formatter": { "quoteStyle": "single" } } }
+{ "extends": ["@shared/format/biome", "@shared/linter/biome"] }
 ```
 
-## `node_modules/@shared/linter/biome.jsonc`
+## `node_modules/@shared/format/biome.json`
 
 ```json
-{ "linter": { "enabled": false } }
+{ "javascript": { "formatter": { "quoteStyle": "single" } } }
 ```
 
-## `biome.json`
+## `node_modules/@shared/linter/biome.jsonc`
 
 ```json
-{ "extends": ["@shared/format/biome", "@shared/linter/biome"] }
+{ "linter": { "enabled": false } }
 ```
 
 ## `node_modules/@shared/format/package.json`
diff --git a/crates/biome_cli/tests/snapshots/main_cases_config_extends/extends_config_ok_from_npm_package_with_condition_names.snap b/crates/biome_cli/tests/snapshots/main_cases_config_extends/extends_config_ok_from_npm_package_with_condition_names.snap
@@ -2,22 +2,22 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `node_modules/@shared/format/biome.json`
+## `biome.json`
 
 ```json
-{ "javascript": { "formatter": { "quoteStyle": "single" } } }
+{ "extends": ["@shared/format", "@shared/linter/biome"] }
 ```
 
-## `node_modules/@shared/linter/biome.jsonc`
+## `node_modules/@shared/format/biome.json`
 
 ```json
-{ "linter": { "enabled": false } }
+{ "javascript": { "formatter": { "quoteStyle": "single" } } }
 ```
 
-## `biome.json`
+## `node_modules/@shared/linter/biome.jsonc`
 
 ```json
-{ "extends": ["@shared/format", "@shared/linter/biome"] }
+{ "linter": { "enabled": false } }
 ```
 
 ## `node_modules/@shared/format/package.json`
diff --git a/crates/biome_cli/tests/snapshots/main_cases_configuration/uses_project_config_before_user_config.snap b/crates/biome_cli/tests/snapshots/main_cases_configuration/uses_project_config_before_user_config.snap
@@ -2,25 +2,25 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `.biome.json`
+## `<CONFIG_DIR>/biome.json`
 
 ```json
 {
   "javascript": {
     "formatter": {
-      "quoteStyle": "single"
+      "enabled": false
     }
   }
 }
 ```
 
-## `<CONFIG_DIR>/biome.json`
+## `.biome.json`
 
 ```json
 {
   "javascript": {
     "formatter": {
-      "enabled": false
+      "quoteStyle": "single"
     }
   }
 }
diff --git a/crates/biome_cli/tests/snapshots/main_cases_included_files/can_force_ignore_biome_json.snap b/crates/biome_cli/tests/snapshots/main_cases_included_files/can_force_ignore_biome_json.snap
@@ -2,16 +2,16 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `nested/biome.json`
+## `biome.json`
 
 ```json
-{ "formatter": { "enabled": false } }
+{ "files": { "includes": ["**/*.js", "!!nested/biome.json"] } }
 ```
 
-## `biome.json`
+## `nested/biome.json`
 
 ```json
-{ "files": { "includes": ["**/*.js", "!!nested/biome.json"] } }
+{ "formatter": { "enabled": false } }
 ```
 
 ## `a.js`
diff --git a/crates/biome_cli/tests/snapshots/main_cases_included_files/errors_on_ignored_nested_biome_json.snap b/crates/biome_cli/tests/snapshots/main_cases_included_files/errors_on_ignored_nested_biome_json.snap
@@ -2,16 +2,16 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `nested/biome.json`
+## `biome.json`
 
 ```json
-{ "formatter": { "enabled": false } }
+{ "files": { "includes": ["**/*.js", "!nested/biome.json"] } }
 ```
 
-## `biome.json`
+## `nested/biome.json`
 
 ```json
-{ "files": { "includes": ["**/*.js", "!nested/biome.json"] } }
+{ "formatter": { "enabled": false } }
 ```
 
 ## `a.js`
diff --git a/crates/biome_cli/tests/snapshots/main_cases_migrate_v2/should_migrate_nested_config.snap b/crates/biome_cli/tests/snapshots/main_cases_migrate_v2/should_migrate_nested_config.snap
@@ -2,18 +2,6 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `bar/biome.json`
-
-```json
-{ "linter": { "rules": { "recommended": true } } }
-```
-
-## `biome.json`
-
-```json
-{ "linter": { "rules": { "recommended": true } } }
-```
-
 ## `foo/biome.json`
 
 ```json
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_fail_for_nested_roots.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_fail_for_nested_roots.snap
@@ -2,18 +2,6 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `biome.json`
-
-```json
-{
-  "javascript": {
-    "formatter": {
-      "quoteStyle": "double"
-    }
-  }
-}
-```
-
 ## `packages/lib/biome.json`
 
 ```json
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_run_from_nested_dir.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_run_from_nested_dir.snap
@@ -2,27 +2,27 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `biome.jsonc`
+## `packages/lib/biome.jsonc`
 
 ```json
 {
+  "extends": "//",
   "linter": {
     "rules": {
-      "correctness": { "noUnusedVariables": "off" },
-      "suspicious": { "noDebugger": "off" }
+      "correctness": { "noUnusedVariables": "error" }
     }
   }
 }
 ```
 
-## `packages/lib/biome.jsonc`
+## `biome.jsonc`
 
 ```json
 {
-  "extends": "//",
   "linter": {
     "rules": {
-      "correctness": { "noUnusedVariables": "error" }
+      "correctness": { "noUnusedVariables": "off" },
+      "suspicious": { "noDebugger": "off" }
     }
   }
 }
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_targeting_file_in_nested_dir.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_targeting_file_in_nested_dir.snap
@@ -2,28 +2,28 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `biome.jsonc`
+## `packages/lib/biome.jsonc`
 
 ```json
 {
+  "extends": "//",
   "linter": {
-    "includes": ["**/*.js"],
     "rules": {
-      "correctness": { "noUnusedVariables": "off" },
-      "suspicious": { "noDebugger": "off" }
+      "correctness": { "noUnusedVariables": "error" }
     }
   }
 }
 ```
 
-## `packages/lib/biome.jsonc`
+## `biome.jsonc`
 
 ```json
 {
-  "extends": "//",
   "linter": {
+    "includes": ["**/*.js"],
     "rules": {
-      "correctness": { "noUnusedVariables": "error" }
+      "correctness": { "noUnusedVariables": "off" },
+      "suspicious": { "noDebugger": "off" }
     }
   }
 }
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_targeting_nested_dir.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_targeting_nested_dir.snap
@@ -2,28 +2,28 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `biome.jsonc`
+## `packages/lib/biome.jsonc`
 
 ```json
 {
+  "extends": "//",
   "linter": {
-    "includes": ["**/*.js"],
     "rules": {
-      "correctness": { "noUnusedVariables": "off" },
-      "suspicious": { "noDebugger": "off" }
+      "correctness": { "noUnusedVariables": "error" }
     }
   }
 }
 ```
 
-## `packages/lib/biome.jsonc`
+## `biome.jsonc`
 
 ```json
 {
-  "extends": "//",
   "linter": {
+    "includes": ["**/*.js"],
     "rules": {
-      "correctness": { "noUnusedVariables": "error" }
+      "correctness": { "noUnusedVariables": "off" },
+      "suspicious": { "noDebugger": "off" }
     }
   }
 }
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_targeting_parent_of_nested_dir.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_find_settings_when_targeting_parent_of_nested_dir.snap
@@ -2,28 +2,28 @@
 source: crates/biome_cli/tests/snap_test.rs
 expression: redactor(content)
 ---
-## `biome.jsonc`
+## `packages/lib/biome.jsonc`
 
 ```json
 {
+  "extends": "//",
   "linter": {
-    "includes": ["**/*.js"],
     "rules": {
-      "correctness": { "noUnusedVariables": "off" },
-      "suspicious": { "noDebugger": "off" }
+      "correctness": { "noUnusedVariables": "error" }
     }
   }
 }
 ```
 
-## `packages/lib/biome.jsonc`
+## `biome.jsonc`
 
 ```json
 {
-  "extends": "//",
   "linter": {
+    "includes": ["**/*.js"],
     "rules": {
-      "correctness": { "noUnusedVariables": "error" }
+      "correctness": { "noUnusedVariables": "off" },
+      "suspicious": { "noDebugger": "off" }
     }
   }
 }
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_format_nested_files_differently.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_format_nested_files_differently.snap
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_ignore_files_in_nested_projects.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_ignore_files_in_nested_projects.snap
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_ignore_nested_configuration_in_ignored_directory.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/should_ignore_nested_configuration_in_ignored_directory.snap
diff --git a/crates/biome_cli/tests/snapshots/main_cases_monorepo/when_running_from_a_subdirectory_should_extend_from_the_root_config.snap b/crates/biome_cli/tests/snapshots/main_cases_monorepo/when_running_from_a_subdirectory_should_extend_from_the_root_config.snap
diff --git a/crates/biome_cli/tests/snapshots/main_commands_migrate/should_migrate_nested_files.snap b/crates/biome_cli/tests/snapshots/main_commands_migrate/should_migrate_nested_files.snap

Original file line number	Diff line number	Diff line change
`@@ -2,25 +2,25 @@`
`2`	`2`	`source: crates/biome_cli/tests/snap_test.rs`
`3`	`3`	`expression: redactor(content)`
`4`	`4`	`---`
`5`		-## `.biome.json`
	`5`	+## `<CONFIG_DIR>/biome.json`
`6`	`6`
`7`	`7`	```json
`8`	`8`	`{`
`9`	`9`	`"javascript": {`
`10`	`10`	`"formatter": {`
`11`		`- "quoteStyle": "single"`
	`11`	`+ "enabled": false`
`12`	`12`	`}`
`13`	`13`	`}`
`14`	`14`	`}`
`15`	`15`	```
`16`	`16`
`17`		-## `<CONFIG_DIR>/biome.json`
	`17`	+## `.biome.json`
`18`	`18`
`19`	`19`	```json
`20`	`20`	`{`
`21`	`21`	`"javascript": {`
`22`	`22`	`"formatter": {`
`23`		`- "enabled": false`
	`23`	`+ "quoteStyle": "single"`
`24`	`24`	`}`
`25`	`25`	`}`
`26`	`26`	`}`
Original file line number	Diff line number	Diff line change
`@@ -2,27 +2,27 @@`
`2`	`2`	`source: crates/biome_cli/tests/snap_test.rs`
`3`	`3`	`expression: redactor(content)`
`4`	`4`	`---`
`5`		-## `biome.jsonc`
	`5`	+## `packages/lib/biome.jsonc`
`6`	`6`
`7`	`7`	```json
`8`	`8`	`{`
	`9`	`+ "extends": "//",`
`9`	`10`	`"linter": {`
`10`	`11`	`"rules": {`
`11`		`- "correctness": { "noUnusedVariables": "off" },`
`12`		`- "suspicious": { "noDebugger": "off" }`
	`12`	`+ "correctness": { "noUnusedVariables": "error" }`
`13`	`13`	`}`
`14`	`14`	`}`
`15`	`15`	`}`
`16`	`16`	```
`17`	`17`
`18`		-## `packages/lib/biome.jsonc`
	`18`	+## `biome.jsonc`
`19`	`19`
`20`	`20`	```json
`21`	`21`	`{`
`22`		`- "extends": "//",`
`23`	`22`	`"linter": {`
`24`	`23`	`"rules": {`
`25`		`- "correctness": { "noUnusedVariables": "error" }`
	`24`	`+ "correctness": { "noUnusedVariables": "off" },`
	`25`	`+ "suspicious": { "noDebugger": "off" }`
`26`	`26`	`}`
`27`	`27`	`}`
`28`	`28`	`}`