chore(SkipContent): improve failing test

[langz]

This is just a first attempt, where I've moved the expect statements inside of the waitForU(seems to be a recurring pattern in SkipContent.text.tsx), not sure if it's fixing anything yet.

Why do I suggest this change?

I've opened two PRs(#3656 & #3653), both builds(3656 & 3653) failing with:

FAIL src/components/skip-content/__tests__/SkipContent.test.tsx
  ● SkipContent.Return › should show button and set focus on content element

    expect(element).not.toBeInTheDocument()

    expected document not to contain element, found <button class="dnb-button dnb-button--secondary dnb-button--has-text dnb-button--wrap" type="button"><span aria-hidden="true" class="dnb-button__alignment">‌</span><span class="dnb-button__text dnb-skeleton--show-font">Aria</span></button> instead

      218 |       expect(document.activeElement.tagName).toBe('SECTION')
      219 |     })
    > 220 |     expect(element.querySelector('.dnb-button')).not.toBeInTheDocument()
          |                                                      ^
      221 |     expect(document.activeElement.classList).toContain(
      222 |       'dnb-skip-content__focus'
      223 |     )

      at Object.toBeInTheDocument (src/components/skip-content/__tests__/SkipContent.test.tsx:220:54)

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
eufemia	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 5, 2024 9:04am

[joakbjerk]

Seems to have fixed the issue? 😄

[tujoworker]

🎉 This PR is included in version 10.35.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀