Sort cached fragment report in test output

Reviewed By: monicatang Differential Revision: D69603365 fbshipit-source-id: 6949e0363360ca07615acc9298e72ed554676276
facebook · Feb 14, 2025 · 4daa9de · 4daa9de
1 parent 4b80697
commit 4daa9de
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/compiler/crates/relay-transforms/tests/validate_unused_variables.rs b/compiler/crates/relay-transforms/tests/validate_unused_variables.rs
@@ -33,7 +33,7 @@ pub async fn transform_fixture(fixture: &Fixture<'_>) -> Result<String, String>
 
     // If we didn't error, print out the state of the fragment cache so we can
     // validate which entries get populated.
-    let lines = validator
+    let mut lines = validator
         .visitor
         .visited_fragments
         .iter()
@@ -52,6 +52,8 @@ pub async fn transform_fixture(fixture: &Fixture<'_>) -> Result<String, String>
         })
         .collect::<Vec<_>>();
 
+    lines.sort();
+
     if lines.is_empty() {
         Ok("OK.\n\nNo Cached Fragments.".to_string())
     } else {

diff --git a/.../tests/validate_unused_variables/fixtures/cycles-read-from-different-entrypoints.expected b/.../tests/validate_unused_variables/fixtures/cycles-read-from-different-entrypoints.expected
@@ -29,5 +29,5 @@ query QueryB($varX: String, $varY: String) {
 OK.
 
 Cached Fragments:
-FragmentD -> POPULATED ("varX, varY")
 FragmentC -> POPULATED ("varX, varY")
+FragmentD -> POPULATED ("varX, varY")
diff --git a/...xtures/query-with-variables-shadowed-by-local-variable-and-used-as-root-variable.expected b/...xtures/query-with-variables-shadowed-by-local-variable-and-used-as-root-variable.expected
@@ -27,6 +27,6 @@ fragment AnotherUser_data on User {
 OK.
 
 Cached Fragments:
+AnotherUser_data -> POPULATED ("foo")
 User_data -> POPULATED ("foo")
 User_data_with_args -> POPULATED ("foo")
-AnotherUser_data -> POPULATED ("foo")