Fix browser-based test

inrupt · Dec 13, 2024 · 0ad70a6 · 0ad70a6
1 parent ad066ce
commit 0ad70a6
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/e2e/browser/test-app/components/accessGrants/index.tsx b/e2e/browser/test-app/components/accessGrants/index.tsx
@@ -74,7 +74,7 @@ function AccessCredential({
       </p>
       <p>
         Custom fields:{" "}
-        <span data-testid="crendential-custom">
+        <span data-testid="credential-custom">
           {JSON.stringify(getCustomFields(vc))}
         </span>
       </p>
@@ -189,6 +189,7 @@ export default function AccessController({
       fetch: session.fetch,
     });
     setAccessRequest(undefined);
+    setAccessRequestUrl(undefined);
   };
 
   const handleCallAuthedGrant = async () => {
@@ -201,7 +202,7 @@ export default function AccessController({
   };
 
   const handleAccessRequest = async () => {
-    if (accessRequestUrl === undefined || URL.canParse(accessRequestUrl)) {
+    if (accessRequestUrl === undefined || !URL.canParse(accessRequestUrl)) {
       console.error(
         "Please issue an Access Request and provide its URL before being redirected.",
       );
@@ -320,6 +321,7 @@ export default function AccessController({
           onChange={(e) => {
             setAccessRequestUrl(e.currentTarget.value);
           }}
+          value={accessRequestUrl}
         />
       </p>
       <div>

diff --git a/e2e/browser/test/e2e.playwright.ts b/e2e/browser/test/e2e.playwright.ts
@@ -98,7 +98,7 @@ test("Redirect to AMC to accept Access Request", async ({
   ]);
 });
 
-test("Issue an access request, then revoking the access request", async ({
+test.only("Issue an access request, then revoking the access request", async ({
   page,
   auth,
 }) => {
@@ -117,7 +117,7 @@ test("Issue an access request, then revoking the access request", async ({
 
   // Issue an access request to the resource.
   await page.getByTestId("issue-access").click();
-  await expect(page.getByTestId("access-request")).resolves.not.toBeEmpty();
+  await expect(page.getByTestId("access-request")).not.toBeEmpty();
   const customFields = await page
     .getByTestId("credential-custom")
     .textContent();
@@ -127,7 +127,7 @@ test("Issue an access request, then revoking the access request", async ({
 
   // Revoke the access request.
   await page.getByTestId("revoke-access").click();
-  await expect(page.getByTestId("access-request")).toBeEmpty();
+  await expect(page.getByTestId("access-request")).toBeHidden();
 
   // Cleanup the resource
   await page.getByTestId("delete-resource").click();