test: fix failures from link changes

Author: joshblack

packages/react/src/TopicTag/TopicTag.test.tsx

@@ -5,15 +5,9 @@ import {TopicTag} from '../TopicTag'
 
 describe('TopicTag', () => {
   test('defaults to <a> semantics', async () => {
-    const onClick = vi.fn()
-    render(
-      <TopicTag href="/test" onClick={onClick}>
-        test
-      </TopicTag>,
-    )
+    render(<TopicTag href="#">test</TopicTag>)
 
-    await userEvent.click(screen.getByRole('link', {name: 'test'}))
-    expect(onClick).toHaveBeenCalled()
+    expect(screen.getByRole('link', {name: 'test'})).toBeInTheDocument()
   })
 
   test('support <button> semantics through `as` prop', async () => {
@@ -24,7 +18,7 @@ describe('TopicTag', () => {
       </TopicTag>,
     )
 
-    await userEvent.click(screen.getByRole('link', {name: 'test'}))
+    await userEvent.click(screen.getByRole('button', {name: 'test'}))
     expect(onClick).toHaveBeenCalled()
   })
 

packages/react/src/TopicTag/TopicTagGroup.tsx

@@ -4,9 +4,9 @@ import classes from './TopicTagGroup.module.css'
 
 type TopicTagGroupProps = React.HTMLAttributes<HTMLElement>
 
-function TopicTagGroup({children, ...rest}: TopicTagGroupProps) {
+function TopicTagGroup({children, className, ...rest}: TopicTagGroupProps) {
   return (
-    <div {...rest} className={clsx(classes.TopicTagGroup)}>
+    <div {...rest} className={clsx(className, classes.TopicTagGroup)}>
       {children}
     </div>
   )