Add tests for "toggle block style" with no selection

rolandcrosby-columntax · rolandcrosby-columntax · commit 7de974578604 · 2025-11-10T11:30:00.000-05:00
diff --git a/test/expectations/code_action_resolve/aref_call_aref_assign_without_selection.exp.json b/test/expectations/code_action_resolve/aref_call_aref_assign_without_selection.exp.json
@@ -0,0 +1,47 @@
+{
+    "params": {
+        "kind": "refactor.rewrite",
+        "title": "Refactor: Toggle block style",
+        "data": {
+            "range": {
+                "start": {
+                    "line": 0,
+                    "character": 29
+                },
+                "end": {
+                    "line": 0,
+                    "character": 29
+                }
+            },
+            "uri": "file:///fake"
+        }
+    },
+    "result": {
+        "title": "Refactor: Toggle block style",
+        "edit": {
+            "documentChanges": [
+                {
+                    "textDocument": {
+                        "uri": "file:///fake",
+                        "version": null
+                    },
+                    "edits": [
+                        {
+                            "range": {
+                                "start": {
+                                    "line": 0,
+                                    "character": 26
+                                },
+                                "end": {
+                                    "line": 0,
+                                    "character": 58
+                                }
+                            },
+                            "newText": "do |a|\n  a[\"field\"] == \"expected\"\nend"
+                        }
+                    ]
+                }
+            ]
+        }
+    }
+}
diff --git a/test/expectations/code_action_resolve/nested_block_calls_without_selection.exp.json b/test/expectations/code_action_resolve/nested_block_calls_without_selection.exp.json
@@ -0,0 +1,47 @@
+{
+    "params": {
+        "kind": "refactor.rewrite",
+        "title": "Refactor: Toggle block style",
+        "data": {
+            "range": {
+                "start": {
+                    "line": 1,
+                    "character": 30
+                },
+                "end": {
+                    "line": 1,
+                    "character": 30
+                }
+            },
+            "uri": "file:///fake"
+        }
+    },
+    "result": {
+        "title": "Refactor: Toggle block style",
+        "edit": {
+            "documentChanges": [
+                {
+                    "textDocument": {
+                        "uri": "file:///fake",
+                        "version": null
+                    },
+                    "edits": [
+                        {
+                            "range": {
+                                "start": {
+                                    "line": 0,
+                                    "character": 29
+                                },
+                                "end": {
+                                    "line": 3,
+                                    "character": 3
+                                }
+                            },
+                            "newText": "{ |a| nested_call(fourth_call).each { |b| } }"
+                        }
+                    ]
+                }
+            ]
+        }
+    }
+}
diff --git a/test/expectations/code_action_resolve/nested_block_calls_without_selection_inner.exp.json b/test/expectations/code_action_resolve/nested_block_calls_without_selection_inner.exp.json
@@ -0,0 +1,47 @@
+{
+    "params": {
+        "kind": "refactor.rewrite",
+        "title": "Refactor: Toggle block style",
+        "data": {
+            "range": {
+                "start": {
+                    "line": 1,
+                    "character": 37
+                },
+                "end": {
+                    "line": 1,
+                    "character": 37
+                }
+            },
+            "uri": "file:///fake"
+        }
+    },
+    "result": {
+        "title": "Refactor: Toggle block style",
+        "edit": {
+            "documentChanges": [
+                {
+                    "textDocument": {
+                        "uri": "file:///fake",
+                        "version": null
+                    },
+                    "edits": [
+                        {
+                            "range": {
+                                "start": {
+                                    "line": 1,
+                                    "character": 32
+                                },
+                                "end": {
+                                    "line": 2,
+                                    "character": 5
+                                }
+                            },
+                            "newText": "{ |b| }"
+                        }
+                    ]
+                }
+            ]
+        }
+    }
+}
diff --git a/test/expectations/code_action_resolve/nested_oneline_blocks_without_selection.exp.json b/test/expectations/code_action_resolve/nested_oneline_blocks_without_selection.exp.json
@@ -0,0 +1,47 @@
+{
+    "params": {
+        "kind": "refactor.rewrite",
+        "title": "Refactor: Toggle block style",
+        "data": {
+            "range": {
+                "start": {
+                    "line": 0,
+                    "character": 31
+                },
+                "end": {
+                    "line": 0,
+                    "character": 31
+                }
+            },
+            "uri": "file:///fake"
+        }
+    },
+    "result": {
+        "title": "Refactor: Toggle block style",
+        "edit": {
+            "documentChanges": [
+                {
+                    "textDocument": {
+                        "uri": "file:///fake",
+                        "version": null
+                    },
+                    "edits": [
+                        {
+                            "range": {
+                                "start": {
+                                    "line": 0,
+                                    "character": 29
+                                },
+                                "end": {
+                                    "line": 0,
+                                    "character": 74
+                                }
+                            },
+                            "newText": "do |a|\n  nested_call(fourth_call).each do |b|\n    \n  end\nend"
+                        }
+                    ]
+                }
+            ]
+        }
+    }
+}
diff --git a/test/expectations/code_action_resolve/nested_oneline_blocks_without_selection_inner.exp.json b/test/expectations/code_action_resolve/nested_oneline_blocks_without_selection_inner.exp.json
@@ -0,0 +1,47 @@
+{
+    "params": {
+        "kind": "refactor.rewrite",
+        "title": "Refactor: Toggle block style",
+        "data": {
+            "range": {
+                "start": {
+                    "line": 0,
+                    "character": 68
+                },
+                "end": {
+                    "line": 0,
+                    "character": 68
+                }
+            },
+            "uri": "file:///fake"
+        }
+    },
+    "result": {
+        "title": "Refactor: Toggle block style",
+        "edit": {
+            "documentChanges": [
+                {
+                    "textDocument": {
+                        "uri": "file:///fake",
+                        "version": null
+                    },
+                    "edits": [
+                        {
+                            "range": {
+                                "start": {
+                                    "line": 0,
+                                    "character": 65
+                                },
+                                "end": {
+                                    "line": 0,
+                                    "character": 72
+                                }
+                            },
+                            "newText": "do |b|\n                                     \n                                   end"
+                        }
+                    ]
+                }
+            ]
+        }
+    }
+}
diff --git a/test/expectations/code_actions/toggle_block_outside_block.exp.json b/test/expectations/code_actions/toggle_block_outside_block.exp.json
@@ -0,0 +1,22 @@
+{
+  "params": {
+    "range": {
+      "start": {
+        "line": 0,
+        "character": 0
+      },
+      "end": {
+        "line": 0,
+        "character": 0
+      }
+    },
+    "textDocument": {
+      "uri": "file:///fake",
+      "version": null
+    },
+    "context": {
+      "diagnostics": []
+    }
+  },
+  "result": []
+}
diff --git a/test/expectations/code_actions/toggle_block_without_selection.exp.json b/test/expectations/code_actions/toggle_block_without_selection.exp.json
@@ -0,0 +1,40 @@
+{
+  "params": {
+    "range": {
+      "start": {
+        "line": 0,
+        "character": 22
+      },
+      "end": {
+        "line": 0,
+        "character": 22
+      }
+    },
+    "textDocument": {
+      "uri": "file:///fake",
+      "version": null
+    },
+    "context": {
+      "diagnostics": []
+    }
+  },
+  "result": [
+    {
+      "title": "Refactor: Toggle block style",
+      "kind": "refactor.rewrite",
+      "data": {
+        "range": {
+          "start": {
+            "line": 0,
+            "character": 22
+          },
+          "end": {
+            "line": 0,
+            "character": 22
+          }
+        },
+        "uri": "file:///fake"
+      }
+    }
+  ]
+}
diff --git a/test/fixtures/aref_call_aref_assign_without_selection.rb b/test/fixtures/aref_call_aref_assign_without_selection.rb
@@ -0,0 +1 @@
+object["attributes"].find { |a| a["field"] == "expected" }["value"] = "changed"
diff --git a/test/fixtures/nested_block_calls_without_selection.rb b/test/fixtures/nested_block_calls_without_selection.rb
@@ -0,0 +1,4 @@
+method_call(other_call).each do |a|
+  nested_call(fourth_call).each do |b|
+  end
+end
diff --git a/test/fixtures/nested_block_calls_without_selection_inner.rb b/test/fixtures/nested_block_calls_without_selection_inner.rb
@@ -0,0 +1,4 @@
+method_call(other_call).each do |a|
+  nested_call(fourth_call).each do |b|
+  end
+end
diff --git a/test/fixtures/nested_oneline_blocks_without_selection.rb b/test/fixtures/nested_oneline_blocks_without_selection.rb
@@ -0,0 +1 @@
+method_call(other_call).each { |a| nested_call(fourth_call).each { |b| } }
diff --git a/test/fixtures/nested_oneline_blocks_without_selection_inner.rb b/test/fixtures/nested_oneline_blocks_without_selection_inner.rb
@@ -0,0 +1 @@
+method_call(other_call).each { |a| nested_call(fourth_call).each { |b| } }
diff --git a/test/fixtures/toggle_block_outside_block.rb b/test/fixtures/toggle_block_outside_block.rb
@@ -0,0 +1 @@
+some_call { block_body }
diff --git a/test/fixtures/toggle_block_without_selection.rb b/test/fixtures/toggle_block_without_selection.rb
@@ -0,0 +1 @@
+some_call { block_body }

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1 @@`
	`1`	`+object["attributes"].find { \|a\| a["field"] == "expected" }["value"] = "changed"`
-Original file line number
+Diff line change
@@ @@ -0,0 +1,4 @@ @@
 +method_call(other_call).each do |a|
 +  nested_call(fourth_call).each do |b|
 +  end
 +end