Skip to content

Commit 8235aa5

Browse files
severosevero
committed
move component tests to component subdirectory
1 parent 9dfe0b3 commit 8235aa5

File tree

12 files changed

+40
-41
lines changed

12 files changed

+40
-41
lines changed

test/components/Breadcrumb.test.tsx renamed to src/components/Breadcrumb/Breadcrumb.test.tsx

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
import { render } from '@testing-library/react'
22
import { strict as assert } from 'assert'
3-
import React from 'react'
43
import { describe, expect, it } from 'vitest'
5-
import { Config, ConfigProvider } from '../../src/hooks/useConfig.js'
6-
import { Breadcrumb, getHyperparamSource } from '../../src/index.js'
4+
import { Config, ConfigProvider } from '../../hooks/useConfig.js'
5+
import { getHyperparamSource } from '../../lib/sources/hyperparamSource.js'
6+
import Breadcrumb from './Breadcrumb.js'
77

88
const endpoint = 'http://localhost:3000'
99

test/components/File.test.tsx renamed to src/components/File/File.test.tsx

Lines changed: 10 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
import { render } from '@testing-library/react'
22
import { strict as assert } from 'assert'
3-
import React, { act } from 'react'
3+
import { act } from 'react'
44
import { describe, expect, it, vi } from 'vitest'
5-
import { Config, ConfigProvider } from '../../src/hooks/useConfig.js'
6-
import { File, getHttpSource, getHyperparamSource } from '../../src/index.js'
5+
import { Config, ConfigProvider } from '../../hooks/useConfig.js'
6+
import { getHttpSource, getHyperparamSource } from '../../lib/sources/index.js'
7+
import File from './File.js'
78

89
const endpoint = 'http://localhost:3000'
910

@@ -60,11 +61,11 @@ describe('File Component', () => {
6061
))
6162

6263
const links = getAllByRole('link')
63-
expect(links[0].getAttribute('href')).toBe('/')
64-
expect(links[1].getAttribute('href')).toBe('/files?key=')
65-
expect(links[2].getAttribute('href')).toBe('/files?key=folder1/')
66-
expect(links[3].getAttribute('href')).toBe('/files?key=folder1/folder2/')
67-
expect(links[4].getAttribute('href')).toBe('/files?key=folder1/folder2/folder3/')
68-
expect(links[5].getAttribute('href')).toBe('/files?key=folder1/folder2/folder3/test.txt')
64+
expect(links[0]?.getAttribute('href')).toBe('/')
65+
expect(links[1]?.getAttribute('href')).toBe('/files?key=')
66+
expect(links[2]?.getAttribute('href')).toBe('/files?key=folder1/')
67+
expect(links[3]?.getAttribute('href')).toBe('/files?key=folder1/folder2/')
68+
expect(links[4]?.getAttribute('href')).toBe('/files?key=folder1/folder2/folder3/')
69+
expect(links[5]?.getAttribute('href')).toBe('/files?key=folder1/folder2/folder3/test.txt')
6970
})
7071
})

test/components/Folder.test.tsx renamed to src/components/Folder/Folder.test.tsx

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,10 @@
11
import { fireEvent, render, waitFor } from '@testing-library/react'
22
import { strict as assert } from 'assert'
3-
import React, { act } from 'react'
3+
import { act } from 'react'
44
import { describe, expect, it, test, vi } from 'vitest'
5-
import { Config, ConfigProvider } from '../../src/hooks/useConfig.js'
6-
import { DirSource, FileMetadata, Folder, HyperparamFileMetadata, getHyperparamSource } from '../../src/index.js'
5+
import { Config, ConfigProvider } from '../../hooks/useConfig.js'
6+
import { DirSource, FileMetadata, HyperparamFileMetadata, getHyperparamSource } from '../../lib/sources/index.js'
7+
import Folder from './Folder.js'
78

89
const endpoint = 'http://localhost:3000'
910
const mockFiles: HyperparamFileMetadata[] = [

test/components/ImageView.test.tsx renamed to src/components/ImageView/ImageView.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import { render } from '@testing-library/react'
22
import { strict as assert } from 'assert'
3-
import React from 'react'
43
import { describe, expect, it, vi } from 'vitest'
5-
import { ImageView, getHyperparamSource } from '../../src/index.js'
4+
import { getHyperparamSource } from '../../lib/sources/index.js'
5+
import ImageView from './ImageView.js'
66

77
globalThis.fetch = vi.fn()
88

test/components/Json.test.tsx renamed to src/components/Json/Json.test.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { fireEvent, render } from '@testing-library/react'
2-
import React from 'react'
32
import { describe, expect, it } from 'vitest'
4-
import Json from '../../src/components/Json/Json.js'
3+
import Json from './Json.js'
54

65
describe('Json Component', () => {
76
it('renders primitive types correctly', () => {

test/components/JsonView.test.tsx renamed to src/components/JsonView/JsonView.test.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
import { render, waitFor } from '@testing-library/react'
2-
import React from 'react'
32
import { describe, expect, it, vi } from 'vitest'
4-
import JsonView from '../../src/components/JsonView/JsonView.js'
5-
import { FileSource } from '../../src/lib/sources/types.js'
3+
import { FileSource } from '../../lib/sources/types.js'
4+
import JsonView from './JsonView.js'
65

76
vi.mock('../../../src/lib/utils.js', async () => {
87
const actual = await vi.importActual('../../../src/lib/utils.js')

test/components/Layout.test.tsx renamed to src/components/Layout/Layout.test.tsx

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { render } from '@testing-library/react'
2-
import React from 'react'
32
import { describe, expect, it } from 'vitest'
4-
import { Layout } from '../../src/index.js'
3+
import Layout from './Layout.js'
54

65
describe('Layout Component', () => {
76
it('renders children', () => {

test/components/Markdown.test.tsx renamed to src/components/Markdown/Markdown.test.tsx

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { render } from '@testing-library/react'
2-
import React from 'react'
32
import { describe, expect, it } from 'vitest'
4-
import { Markdown } from '../../src/index.js'
3+
import Markdown from './Markdown.js'
54

65
describe('Markdown', () => {
76
it('renders plain text as a paragraph', () => {
@@ -57,8 +56,8 @@ describe('Markdown', () => {
5756
expect(getByText('Hyp')).toBeDefined()
5857
expect(getByText('GitHub')).toBeDefined()
5958
const links = getAllByRole('link')
60-
expect(links[0].getAttribute('href')).toBe('https://hyperparam.app')
61-
expect(links[1].getAttribute('href')).toBe('https://github.com/hyparam')
59+
expect(links[0]?.getAttribute('href')).toBe('https://hyperparam.app')
60+
expect(links[1]?.getAttribute('href')).toBe('https://github.com/hyparam')
6261
})
6362

6463
it('renders blockquote', () => {
@@ -247,15 +246,18 @@ describe('Markdown with nested elements', () => {
247246

248247
const images = container.querySelectorAll('img')
249248
expect(images.length).toBe(2)
249+
if (!(0 in images && 1 in images)) {
250+
throw new Error('should not occur')
251+
}
250252

251253
// First link contains first image
252-
expect(links[0].getAttribute('href')).toBe('https://opensource.org/licenses/MIT')
253-
expect(links[0].contains(images[0])).toBe(true)
254+
expect(links[0]?.getAttribute('href')).toBe('https://opensource.org/licenses/MIT')
255+
expect(links[0]?.contains(images[0])).toBe(true)
254256
expect(images[0].getAttribute('alt')).toBe('license')
255257

256258
// Second link contains second image
257-
expect(links[1].getAttribute('href')).toBe('https://www.npmjs.com/package')
258-
expect(links[1].contains(images[1])).toBe(true)
259+
expect(links[1]?.getAttribute('href')).toBe('https://www.npmjs.com/package')
260+
expect(links[1]?.contains(images[1])).toBe(true)
259261
expect(images[1].getAttribute('alt')).toBe('npm')
260262
})
261263

test/components/MarkdownView.test.tsx renamed to src/components/MarkdownView/MarkdownView.test.tsx

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import { render } from '@testing-library/react'
22
import { strict as assert } from 'assert'
3-
import React from 'react'
43
import { describe, expect, it, vi } from 'vitest'
5-
import { MarkdownView, getHyperparamSource } from '../../src/index.js'
4+
import { getHyperparamSource } from '../../lib/sources/index.js'
5+
import MarkdownView from './MarkdownView.js'
66

77
globalThis.fetch = vi.fn()
88

test/components/SlidePanel.test.tsx renamed to src/components/SlidePanel/SlidePanel.test.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,8 @@
11

22
import { act, fireEvent, render } from '@testing-library/react'
3-
import React from 'react'
43
import { beforeEach, describe, expect, it, vi } from 'vitest'
5-
import SlidePanel from '../../src/components/SlidePanel/SlidePanel.js'
6-
import { ConfigProvider } from '../../src/hooks/useConfig.js'
4+
import { ConfigProvider } from '../../hooks/useConfig.js'
5+
import SlidePanel from './SlidePanel.js'
76

87
describe('SlidePanel', () => {
98
// Minimal localStorage mock

0 commit comments

Comments
 (0)