yetangitu/Peertube
1
0
Fork 0
mirror of https://github.com/Chocobozzz/PeerTube.git synced 2025-10-05 02:39:33 +02:00
Code Issues Releases Wiki Activity

Fix flaky tests

Browse source
This commit is contained in:
Chocobozzz 2025-03-19 16:23:55 +01:00
parent ad2eab26b1
commit 56a28cb95e
No known key found for this signature in database
GPG key ID: 583A612D890159BE
3 changed files with 14 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

2
.github/workflows/test.yml vendored
View file

@ -111,7 +111,7 @@ jobs:
) || \ ) || \
echo "parse-log.js script does not exist, skipping." echo "parse-log.js script does not exist, skipping."
- name: Upload logs - name: Upload logs and database
uses: actions/upload-artifact@v4 uses: actions/upload-artifact@v4
if: failure() if: failure()
with: with:

2
packages/server-commands/src/server/servers-command.ts
View file

@ -49,7 +49,7 @@ export class ServersCommand extends AbstractCommand {
if (!isGithubCI()) return if (!isGithubCI()) return
await ensureDir('artifacts') await ensureDir('artifacts')
const destname = join('artifacts', 'databases', `peertube-${this.server.internalServerNumber}.sql`) const destname = join('artifacts', `peertube-${this.server.internalServerNumber}.sql`)
console.log('Saving database %s.', destname) console.log('Saving database %s.', destname)
exec(`pg_dump peertube_test${this.server.internalServerNumber} > ${destname}`) exec(`pg_dump peertube_test${this.server.internalServerNumber} > ${destname}`)

15
packages/tests/src/api/videos/resumable-upload.ts
View file

@ -135,7 +135,6 @@ describe('Test resumable upload', function () {
}) })
describe('Directory cleaning', function () { describe('Directory cleaning', function () {
it('Should correctly delete files after an upload', async function () { it('Should correctly delete files after an upload', async function () {
const uploadId = await prepareUpload() const uploadId = await prepareUpload()
await sendChunks({ pathUploadId: uploadId }) await sendChunks({ pathUploadId: uploadId })
@ -171,7 +170,6 @@ describe('Test resumable upload', function () {
}) })
describe('Resumable upload and chunks', function () { describe('Resumable upload and chunks', function () {
it('Should accept the same amount of chunks', async function () { it('Should accept the same amount of chunks', async function () {
const uploadId = await prepareUpload() const uploadId = await prepareUpload()
await sendChunks({ pathUploadId: uploadId }) await sendChunks({ pathUploadId: uploadId })
@ -207,7 +205,18 @@ describe('Test resumable upload', function () {
const size = 1000 const size = 1000
const contentRangeBuilder = (start: number) => `bytes ${start}-${start + size - 1}/${size}` const contentRangeBuilder = (start: number) => `bytes ${start}-${start + size - 1}/${size}`
await sendChunks({ pathUploadId: uploadId, expectedStatus: HttpStatusCode.CONFLICT_409, contentRangeBuilder, contentLength: size })
try {
await sendChunks({ pathUploadId: uploadId, expectedStatus: HttpStatusCode.CONFLICT_409, contentRangeBuilder, contentLength: size })
} catch (err) {
await sendChunks({
pathUploadId: uploadId,
expectedStatus: HttpStatusCode.BAD_REQUEST_400,
contentRangeBuilder,
contentLength: size
})
}
await checkFileSize(uploadId, 0) await checkFileSize(uploadId, 0)
}) })