Move setAttribute before connect

ni · May 20, 2024 · 4a9a0b1 · 4a9a0b1
1 parent 36a01b8
commit 4a9a0b1
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/packages/nimble-components/src/anchor/tests/anchor.spec.ts b/packages/nimble-components/src/anchor/tests/anchor.spec.ts
@@ -92,9 +92,9 @@ describe('Anchor', () => {
     });
 
     it('should clear tabindex attribute from the internal control when cleared from the host', async () => {
+        element.setAttribute('tabindex', '-1');
         await connect();
 
-        element.setAttribute('tabindex', '-1');
         element.removeAttribute('tabindex');
         await waitForUpdatesAsync();
 

diff --git a/packages/nimble-components/src/menu-button/tests/menu-button.spec.ts b/packages/nimble-components/src/menu-button/tests/menu-button.spec.ts
@@ -136,8 +136,8 @@ describe('MenuButton', () => {
 
         it('should clear tabindex from the toggle button when cleared from the host', async () => {
             element.setAttribute('tabindex', '-1');
-            element.removeAttribute('tabindex');
             await connect();
+            element.removeAttribute('tabindex');
             expect(element.toggleButton!.getAttribute('tabindex')).toBeNull();
         });
 

diff --git a/packages/nimble-components/src/toggle-button/tests/toggle-button.spec.ts b/packages/nimble-components/src/toggle-button/tests/toggle-button.spec.ts
@@ -118,9 +118,9 @@ describe('ToggleButton', () => {
     });
 
     it('should set the `tabindex` attribute to 0 on the internal button when removed from host', async () => {
+        element.setAttribute('tabindex', '-1');
         await connect();
 
-        element.setAttribute('tabindex', '-1');
         element.removeAttribute('tabindex');
         await waitForUpdatesAsync();