chore: cleanup auth code section logic and tests

ory · Sep 15, 2023 · dd1bbcc · dd1bbcc
1 parent 7c32444
commit dd1bbcc
Show file tree

Hide file tree

Showing 2 changed files with 78 additions and 3 deletions.
diff --git a/src/react-components/ory/sections/auth-code-section.spec.tsx b/src/react-components/ory/sections/auth-code-section.spec.tsx
@@ -104,3 +104,77 @@ test("shouldn't render anything with no code group", async ({ mount }) => {
 
   await expect(container.locator("input[name=identifier]")).not.toBeVisible()
 })
+
+test("should render traits on default group", async ({ mount }) => {
+  const container = await mount(
+    <AuthCodeSection
+      nodes={[
+        {
+          group: "default",
+          attributes: {
+            name: "traits.email",
+            node_type: "input",
+            type: "text",
+            required: true,
+            disabled: false,
+          },
+          messages: [],
+          type: "input",
+          meta: {
+            label: {
+              id: 1070002,
+              text: "E-Mail",
+              type: "info",
+              context: {
+                title: "E-Mail",
+              },
+            },
+          },
+        },
+        {
+          group: "default",
+          attributes: {
+            name: "traits.name",
+            node_type: "input",
+            type: "text",
+            required: true,
+            disabled: false,
+          },
+          messages: [],
+          type: "input",
+          meta: {
+            label: {
+              id: 1070001,
+              text: "Name",
+              type: "info",
+            },
+          },
+        },
+        {
+          group: "code",
+          attributes: {
+            name: "method",
+            node_type: "input",
+            type: "submit",
+            required: true,
+            disabled: false,
+            value: "code",
+          },
+          messages: [],
+          type: "input",
+          meta: {
+            label: {
+              id: 1040006,
+              text: "Sign up with code",
+              type: "info",
+            },
+          },
+        },
+      ]}
+    />,
+  )
+
+  await expect(container.locator("input[name='traits.email']")).toBeVisible()
+  await expect(container.locator("input[name='traits.name']")).toBeVisible()
+  await expect(container.locator("button[name=method]")).toBeVisible()
+})
diff --git a/src/react-components/ory/sections/auth-code-section.tsx b/src/react-components/ory/sections/auth-code-section.tsx
@@ -18,7 +18,8 @@ export const AuthCodeSection = ({
           filter={{
             nodes: nodes,
             groups: "code",
-            excludeAttributes: ["submit", "hidden"],
+            withoutDefaultAttributes: true,
+            excludeAttributes: ["hidden", "button", "submit"], // the form will take care of hidden fields
           }}
         />
       </div>
@@ -28,8 +29,8 @@ export const AuthCodeSection = ({
         filter={{
           nodes: nodes,
           groups: "code",
-          withoutDefaultGroup: true,
-          attributes: ["submit", "hidden"],
+          withoutDefaultAttributes: true,
+          attributes: ["button", "submit"],
         }}
       />
     </div>