Skip to content

Commit

Permalink
do not refresh stream bounds
Browse files Browse the repository at this point in the history
  • Loading branch information
rassokhin-s committed Jul 17, 2023
1 parent ad45cf0 commit 08a006e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 48 deletions.
40 changes: 0 additions & 40 deletions core/internal/ingest/post.int.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -485,46 +485,6 @@ describe('POST internal/ingest/streams/:id/stream-source-file-and-segments', ()
})
})

describe('stream bounds update', () => {
test('stream start, end and max_sample_rate are set for empty stream', async () => {
await commonSetup()
await request(app).post(`/streams/${stream.id}/stream-source-file-and-segments`).send(testPayload)

const streamFromDb = await models.Stream.findOne({ where: { id: stream.id } })
expect(streamFromDb.maxSampleRate).toBe(testPayload.stream_source_file.sample_rate)
expect(streamFromDb.start).toEqual(moment.utc(testPayload.stream_segments[0].start).toDate())
expect(streamFromDb.end).toEqual(moment.utc(testPayload.stream_segments[0].end).toDate())
})

test('stream start, end and max_sample_rate are updated if new values are bigger/smaller', async () => {
await commonSetup()
const stream = await models.Stream.create(
{ id: 'abcdsaqwery2', name: 'my stream 2', createdById: seedValues.primaryUserId, start: '2021-04-18T12:12:10.000Z', end: '2021-04-18T12:12:20.000Z', maxSampleRate: 24000 }
)

await request(app).post(`/streams/${stream.id}/stream-source-file-and-segments`).send(testPayload)

const streamFromDb = await models.Stream.findOne({ where: { id: stream.id } })
expect(streamFromDb.maxSampleRate).toBe(testPayload.stream_source_file.sample_rate)
expect(streamFromDb.start).toEqual(moment.utc(testPayload.stream_segments[0].start).toDate())
expect(streamFromDb.end).toEqual(moment.utc(testPayload.stream_segments[0].end).toDate())
})

test('stream start, end and max_sample_rate are not updated if new values are not bigger/smaller', async () => {
await commonSetup()
const stream = await models.Stream.create(
{ id: 'abcdsaqwery3', name: 'my stream 3', createdById: seedValues.primaryUserId, start: '2020-01-01 00:00:00', end: '2021-05-05 00:00:00', maxSampleRate: 128000 }
)

await request(app).post(`/streams/${stream.id}/stream-source-file-and-segments`).send(testPayload)

const streamFromDb = await models.Stream.findOne({ where: { id: stream.id } })
expect(streamFromDb.maxSampleRate).toBe(stream.maxSampleRate)
expect(streamFromDb.start).toEqual(moment.utc(stream.start).toDate())
expect(streamFromDb.end).toEqual(moment.utc(stream.end).toDate())
})
})

describe('unavailable segments', () => {
test('returns 1 unavailable segment id', async () => {
await commonSetup()
Expand Down
9 changes: 1 addition & 8 deletions core/internal/ingest/post.js
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ module.exports = function (req, res) {
const sfParams = await sfConverter.validate() // validate stream_source_file attributes
const transformedArray = await segConverter.validate() // validate stream_segment[] attributes

const stream = await streamDao.get(streamId, { transaction })
await streamDao.get(streamId, { transaction })
// Set missing stream_source_file attributes and create a db row
sfParams.stream_id = streamId
streamSourceFileDao.transformMetaAttr(sfParams)
Expand All @@ -98,13 +98,6 @@ module.exports = function (req, res) {
}

const createdSegments = segments.filter(s => s.created)
// Refresh stream max_sample rate, start and end if needed
const maxEnd = moment.max(transformedArray.map(s => s.end))
await streamDao.refreshStreamBoundVars(stream, {
start: minStart.toDate(),
end: maxEnd.toDate(),
sampleRate: streamSourceFile.sample_rate
}, { transaction })

if (arbimonService.isEnabled && createdSegments.length) {
await arbimonService.createRecordingsFromSegments(sfParams, createdSegments, { transaction })
Expand Down

0 comments on commit 08a006e

Please sign in to comment.