test: this is not experimental feature

Author: zackad

src/index.js

@@ -103,12 +103,6 @@ const options = {
         category: "Global",
         default: false,
         description: "Output the Twig block name in the 'endblock' tag"
-    },
-    experimentalMethodChainIndentation: {
-        type: "boolean",
-        category: "Global",
-        default: false,
-        description: "Experimental: Fix indentation for chained method call"
     }
 };
 

src/print/CallExpression.js

@@ -8,7 +8,7 @@ import {
 
 const { group, softline, line, indent, join } = doc.builders;
 
-const p = (node, path, print, options) => {
+const p = (node, path, print) => {
     node[EXPRESSION_NEEDED] = false;
     node[STRING_NEEDS_QUOTES] = true;
     const mappedArguments = path.map(print, "arguments");
@@ -22,8 +22,6 @@ const p = (node, path, print, options) => {
         // Optimization: No line break between "(" and "{" if
         // there is exactly one object parameter
         parts.push(mappedArguments[0], ")");
-    } else if (options.experimentalMethodChainIndentation) {
-        parts.push(indent([join([", "], mappedArguments)]), ")");
     } else {
         parts.push(
             indent([softline, join([",", line], mappedArguments)]),

src/print/MemberExpression.js

@@ -5,19 +5,13 @@ import {
     wrapExpressionIfNeeded
 } from "../util/index.js";
 
-const { group, softline, indent } = doc.builders;
+const { group } = doc.builders;
 
-const p = (node, path, print, options) => {
+const p = (node, path, print) => {
     node[EXPRESSION_NEEDED] = false;
     node[STRING_NEEDS_QUOTES] = true;
     const parts = [path.call(print, "object")];
-    if (node.computed) {
-        parts.push("[");
-    } else if (options.experimentalMethodChainIndentation) {
-        parts.push(indent([softline, "."]));
-    } else {
-        parts.push(".");
-    }
+    parts.push(node.computed ? "[" : ".");
     parts.push(path.call(print, "property"));
     if (node.computed) {
         parts.push("]");

tests/Experimental/experimental.spec.js

@@ -1,11 +1,25 @@
-<a class="task-edit"
+<a
+    class="task-edit"
     href="{{
         ea_url()
             .setDashboard('App\\Controller\\Dashboard\\DashboardController')
             .setController('App\\Controller\\Dashboard\\CRUD\\TacheCrudController')
             .setAction('edit')
             .setEntityId(tache.id)
             .set('sort', null)
-    }}">
+    }}"
+>
     <i class="fa-solid fa-edit"></i>
 </a>
+
+{{
+    craft
+        .entries
+        .section('news')
+        .section('news')
+        .orderBy('postDate DESC')
+        .limit(10)
+        .all(a.b)
+}}
+
+{{ craft.entries.all() }}

tests/Experimental/__snapshots__/chain_indentation.snap.twig renamed to tests/Expressions/__snapshots__/chain_indentation.snap.twig

@@ -91,4 +91,14 @@ describe("Expressions", () => {
         });
         await expect(actual).toMatchFileSnapshot(snapshotFile);
     });
+
+    it("Properly indent chain method", async () => {
+        const { actual, snapshotFile } = await run_spec(import.meta.url, {
+            source: "chain_indentation.twig",
+            formatOptions: {
+                experimentalMethodChainIndentation: true
+            }
+        });
+        await expect(actual).toMatchFileSnapshot(snapshotFile);
+    });
 });