Skip to content

Commit ce22f17

Browse files
committed
build: skip yt-dlp tests on ci
1 parent cf1c6d0 commit ce22f17

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.github/workflows/test.yml

-1
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,5 @@ jobs:
1717
uses: actions/setup-node@v4
1818
with:
1919
node-version: ${{ matrix.node-version }}
20-
- run: pip install yt-dlp
2120
- run: npm ci
2221
- run: npm test

test/youtube.test.js

+2
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@ import { test } from 'node:test'
22
import assert from 'assert';
33
import { getVideosFor, getVideo, extractIdFromUrl, isYouTubeUrl, isUnsupportedUrl, isVideoId } from '../lib/youtube.js'
44

5+
if (!process.env.CI) {
56
test('gets videos for channel', async () => {
67
const videos = await getVideosFor('veritasium')
78
const video = videos[0]
@@ -30,6 +31,7 @@ test('gets single video', async () => {
3031
assert.ok(video.viewCount)
3132
assert.equal(video.duration, '33:36')
3233
})
34+
}
3335

3436
test('extracts id from url', () => {
3537
assert.equal(extractIdFromUrl('https://www.youtube.com/watch?v=SOME_ID&pp=something'), 'SOME_ID')

0 commit comments

Comments
 (0)