test: use data-testid instead of relying on classes

Author: MatteoGabriele

app/components/PackageVersions.vue

@@ -318,6 +318,7 @@ function getTagVersions(tag: string): VersionDisplay[] {
             <span
               v-if="loadingTags.has(row.tag)"
               class="i-carbon-rotate-180 w-3 h-3 motion-safe:animate-spin"
+              data-testid="loading-spinner"
               aria-hidden="true"
             />
             <span
@@ -458,6 +459,7 @@ function getTagVersions(tag: string): VersionDisplay[] {
             <span
               v-if="otherVersionsLoading"
               class="i-carbon-rotate-180 w-3 h-3 motion-safe:animate-spin"
+              data-testid="loading-spinner"
               aria-hidden="true"
             />
             <span

test/nuxt/components/PackageVersions.spec.ts

@@ -782,7 +782,7 @@ describe('PackageVersions', () => {
 
       // Should show loading spinner (animate-spin class)
       await vi.waitFor(() => {
-        expect(component.find('.animate-spin').exists()).toBe(true)
+        expect(component.find('[data-testid="loading-spinner"]').exists()).toBe(true)
       })
 
       // Resolve the promise to clean up
@@ -816,7 +816,7 @@ describe('PackageVersions', () => {
 
       // Should show loading spinner
       await vi.waitFor(() => {
-        expect(component.find('.animate-spin').exists()).toBe(true)
+        expect(component.find('[data-testid="loading-spinner"]').exists()).toBe(true)
       })
 
       // Resolve the promise to clean up