“開発モード” の表示 (#389) #393
@@ -1,9 +1,10 @@
|
||||
import { act, fireEvent, screen, waitFor } from '@testing-library/react'
|
||||
import { act, fireEvent, screen, waitFor, within } from '@testing-library/react'
|
||||
import { Route, Routes } from 'react-router-dom'
|
||||
import { beforeEach, describe, expect, it, vi } from 'vitest'
|
||||
|
||||
import TheatreDetailPage from '@/pages/theatres/TheatreDetailPage'
|
||||
import { buildPost,
|
||||
buildTag,
|
||||
buildTheatre,
|
||||
buildTheatreComment,
|
||||
buildTheatreInfo,
|
||||
@@ -96,6 +97,9 @@ const renderPage = (user = buildUser ({ id: 1, role: 'member' })) =>
|
||||
{ route: '/theatres/7' },
|
||||
)
|
||||
|
||||
const tagSection = (): HTMLElement =>
|
||||
screen.getAllByRole ('heading', { name: 'タグ' })[0].closest ('section')!
|
||||
|
||||
const mockDefaultApi = () => {
|
||||
api.apiGet.mockImplementation ((path: string) => {
|
||||
switch (path)
|
||||
@@ -245,6 +249,53 @@ describe ('TheatreDetailPage', () => {
|
||||
expect (postEmbed.seek).not.toHaveBeenCalledWith (0)
|
||||
})
|
||||
|
||||
it ('shows child tags from the post tag tree in both vertical and horizontal layouts', async () => {
|
||||
const childTag = buildTag ({ id: 12, name: '子タグ', category: 'general' })
|
||||
const parentTag = buildTag ({
|
||||
id: 11,
|
||||
name: '親タグ',
|
||||
category: 'general',
|
||||
children: [childTag],
|
||||
})
|
||||
const duplicateParentTag = buildTag ({
|
||||
id: 13,
|
||||
name: '別親タグ',
|
||||
category: 'general',
|
||||
children: [childTag],
|
||||
})
|
||||
|
||||
postsApi.fetchPost.mockResolvedValueOnce (buildPost ({
|
||||
...currentPost,
|
||||
tags: [parentTag, duplicateParentTag],
|
||||
}))
|
||||
|
||||
renderPage ()
|
||||
|
||||
await screen.findByText ('Embed:上映中の投稿')
|
||||
|
||||
expect (within (tagSection ()).getByRole ('link', { name: '親タグ' }))
|
||||
.toBeInTheDocument ()
|
||||
expect (within (tagSection ()).getByRole ('link', { name: '別親タグ' }))
|
||||
.toBeInTheDocument ()
|
||||
expect (within (tagSection ()).getAllByRole ('link', { name: '子タグ' })[0])
|
||||
.toBeInTheDocument ()
|
||||
expect (within (tagSection ()).getAllByText ('↳').length).toBeGreaterThan (0)
|
||||
|
||||
fireEvent.click (screen.getByRole ('button', { name: '2 列 A 型' }))
|
||||
fireEvent.click (screen.getAllByRole ('button', { name: '横並び' })[0])
|
||||
|
||||
await waitFor (() => {
|
||||
expect (within (tagSection ()).getByRole ('link', { name: '親タグ' }))
|
||||
.toBeInTheDocument ()
|
||||
expect (within (tagSection ()).getByRole ('link', { name: '別親タグ' }))
|
||||
.toBeInTheDocument ()
|
||||
expect (within (tagSection ()).getByRole ('link', { name: '子タグ' }))
|
||||
.toBeInTheDocument ()
|
||||
expect (within (tagSection ()).getAllByRole ('link', { name: '子タグ' }))
|
||||
.toHaveLength (1)
|
||||
})
|
||||
})
|
||||
|
||||
it ('does not advance host post while video length is unknown', async () => {
|
||||
api.apiPut.mockImplementation ((path: string) => {
|
||||
switch (path)
|
||||
|
||||
@@ -102,6 +102,28 @@ const compareTagName = (a: Tag, b: Tag): number =>
|
||||
a.name === b.name ? 0 : (a.name < b.name ? -1 : 1)
|
||||
|
||||
|
||||
const flattenTags = (tags: Tag[]): Tag[] => {
|
||||
const flattened: Tag[] = []
|
||||
const seen = new Set<number> ()
|
||||
|
||||
const visit = (tag: Tag) => {
|
||||
if (seen.has (tag.id))
|
||||
return
|
||||
|
||||
seen.add (tag.id)
|
||||
flattened.push (tag)
|
||||
|
||||
for (const child of tag.children ?? [])
|
||||
visit (child)
|
||||
}
|
||||
|
||||
for (const tag of tags)
|
||||
visit (tag)
|
||||
|
||||
return flattened
|
||||
}
|
||||
|
||||
|
||||
const tagsByCategory = (tags: Tag[]): Partial<Record<Category, Tag[]>> => {
|
||||
const grouped: Partial<Record<Category, Tag[]>> = { }
|
||||
|
||||
@@ -118,15 +140,39 @@ const tagsByCategory = (tags: Tag[]): Partial<Record<Category, Tag[]>> => {
|
||||
}
|
||||
|
||||
|
||||
const renderReadonlyTagTree = (
|
||||
tag: Tag,
|
||||
nestLevel: number,
|
||||
path: string,
|
||||
ancestors: Set<number> = new Set<number> (),
|
||||
): ReactNode[] => {
|
||||
const key = `${ path }-${ tag.id }`
|
||||
const nextAncestors = new Set (ancestors)
|
||||
nextAncestors.add (tag.id)
|
||||
|
||||
return [
|
||||
(
|
||||
<li key={key} className="text-left leading-tight">
|
||||
<TagLink tag={tag} nestLevel={nestLevel} withCount={false}/>
|
||||
</li>),
|
||||
...((tag.children ?? [])
|
||||
.filter (child => !(nextAncestors.has (child.id)))
|
||||
.sort (compareTagName)
|
||||
.flatMap (child =>
|
||||
renderReadonlyTagTree (child, nestLevel + 1, key, nextAncestors)))]
|
||||
}
|
||||
|
||||
|
||||
const TagList: FC<{ tags: Tag[]; compact?: boolean; flow?: TagFlow }> = (
|
||||
{ tags, compact, flow = 'vertical' }) => {
|
||||
const grouped = tagsByCategory (tags)
|
||||
const horizontalGrouped = tagsByCategory (flattenTags (tags))
|
||||
const verticalGrouped = tagsByCategory (tags)
|
||||
|
||||
if (flow === 'horizontal')
|
||||
{
|
||||
return (
|
||||
<ul className={cn ('flex flex-wrap gap-x-3 gap-y-1', compact && 'text-sm')}>
|
||||
{CATEGORIES.flatMap (cat => grouped[cat] ?? []).map (tag => (
|
||||
{CATEGORIES.flatMap (cat => horizontalGrouped[cat] ?? []).map (tag => (
|
||||
<li key={tag.id} className="text-left leading-tight">
|
||||
<TagLink tag={tag} withCount={false}/>
|
||||
</li>))}
|
||||
@@ -136,7 +182,7 @@ const TagList: FC<{ tags: Tag[]; compact?: boolean; flow?: TagFlow }> = (
|
||||
return (
|
||||
<div className="space-y-3">
|
||||
{CATEGORIES.map (cat => {
|
||||
const rows = grouped[cat] ?? []
|
||||
const rows = verticalGrouped[cat] ?? []
|
||||
if (rows.length === 0)
|
||||
return null
|
||||
|
||||
@@ -146,10 +192,8 @@ const TagList: FC<{ tags: Tag[]; compact?: boolean; flow?: TagFlow }> = (
|
||||
{CATEGORY_NAMES[cat]}
|
||||
</div>
|
||||
<ul className={cn ('space-y-1', compact && 'text-sm')}>
|
||||
{rows.map (tag => (
|
||||
<li key={tag.id} className="text-left leading-tight">
|
||||
<TagLink tag={tag} withCount={false}/>
|
||||
</li>))}
|
||||
{rows.flatMap (tag =>
|
||||
renderReadonlyTagTree (tag, 0, `cat-${ cat }`))}
|
||||
</ul>
|
||||
</div>)
|
||||
})}
|
||||
|
||||
新しい課題から参照
ユーザをブロックする