Skip to content

Commit 8f2164b

Browse files
committed
wip: tidy
1 parent c7a39cf commit 8f2164b

File tree

6 files changed

+13
-17
lines changed

6 files changed

+13
-17
lines changed

meteor/server/publications/pieceContentStatusUI/__tests__/checkPieceContentStatus.test.ts

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@ import { defaultStudio } from '../../../../__mocks__/defaultCollectionObjects'
3636
import { MediaObjects } from '../../../collections'
3737
import { PieceDependencies } from '../common'
3838
import { DEFAULT_MINIMUM_TAKE_SPAN } from '@sofie-automation/shared-lib/dist/core/constants'
39+
import { PieceContentStatusMessageFactory } from '../messageFactory'
3940

4041
const mockMediaObjectsCollection = MongoMock.getInnerMockCollection<MediaObject>(MediaObjects)
4142

@@ -449,7 +450,9 @@ describe('lib/mediaObjects', () => {
449450
timelineObjectsString: EmptyPieceTimelineObjectsBlob,
450451
})
451452

452-
const status1 = await checkPieceContentStatusAndDependencies(mockStudio, piece1, sourcelayer1)
453+
const messageFactory = new PieceContentStatusMessageFactory(undefined)
454+
455+
const status1 = await checkPieceContentStatusAndDependencies(mockStudio, messageFactory, piece1, sourcelayer1)
453456
expect(status1[0].status).toEqual(PieceStatusCode.OK)
454457
expect(status1[0].messages).toHaveLength(0)
455458
expect(status1[1]).toMatchObject(
@@ -460,7 +463,7 @@ describe('lib/mediaObjects', () => {
460463
})
461464
)
462465

463-
const status2 = await checkPieceContentStatusAndDependencies(mockStudio, piece2, sourcelayer1)
466+
const status2 = await checkPieceContentStatusAndDependencies(mockStudio, messageFactory, piece2, sourcelayer1)
464467
expect(status2[0].status).toEqual(PieceStatusCode.SOURCE_BROKEN)
465468
expect(status2[0].messages).toHaveLength(1)
466469
expect(status2[0].messages[0]).toMatchObject({
@@ -474,7 +477,7 @@ describe('lib/mediaObjects', () => {
474477
})
475478
)
476479

477-
const status3 = await checkPieceContentStatusAndDependencies(mockStudio, piece3, sourcelayer1)
480+
const status3 = await checkPieceContentStatusAndDependencies(mockStudio, messageFactory, piece3, sourcelayer1)
478481
expect(status3[0].status).toEqual(PieceStatusCode.SOURCE_MISSING)
479482
expect(status3[0].messages).toHaveLength(1)
480483
expect(status3[0].messages[0]).toMatchObject({

meteor/server/publications/pieceContentStatusUI/bucket/bucketContentObserver.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import { Meteor } from 'meteor/meteor'
2-
import { BucketId, ShowStyleBaseId } from '@sofie-automation/corelib/dist/dataModel/Ids'
2+
import { BlueprintId, BucketId, ShowStyleBaseId } from '@sofie-automation/corelib/dist/dataModel/Ids'
33
import { logger } from '../../../logging'
44
import {
55
blueprintFieldSpecifier,

meteor/server/publications/pieceContentStatusUI/common.ts

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,4 @@
1-
import {
2-
ExpectedPackageId,
3-
PackageContainerPackageId,
4-
RundownId,
5-
StudioId,
6-
} from '@sofie-automation/corelib/dist/dataModel/Ids'
1+
import { ExpectedPackageId, PackageContainerPackageId, StudioId } from '@sofie-automation/corelib/dist/dataModel/Ids'
72
import { PackageContainerPackageStatusDB } from '@sofie-automation/corelib/dist/dataModel/PackageContainerPackageStatus'
83
import { PackageInfoDB } from '@sofie-automation/corelib/dist/dataModel/PackageInfos'
94
import { MongoFieldSpecifierOnesStrict } from '@sofie-automation/corelib/dist/mongo'

packages/blueprints-integration/src/api/showStyle.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ export interface ShowStyleBlueprintManifest<TRawConfig = IBlueprintConfig, TProc
6464
configPresets: Record<string, IShowStyleConfigPreset<TRawConfig>>
6565

6666
/** Alternate package status messages, to override the builtin ones produced by Sofie */
67-
packageStatusMessages?: Partial<Record<PackageStatusMessage, string>>
67+
packageStatusMessages?: Partial<Record<PackageStatusMessage, string | undefined>>
6868

6969
/** Translations connected to the studio (as stringified JSON) */
7070
translations?: string

packages/corelib/src/dataModel/Blueprint.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ export interface Blueprint {
5555
hasFixUpFunction: boolean
5656

5757
/** The blueprint provided alternate package status messages, if any were provided */
58-
packageStatusMessages?: Record<PackageStatusMessage, string | undefined>
58+
packageStatusMessages?: Partial<Record<PackageStatusMessage, string | undefined>>
5959
}
6060

6161
/** Describes the last state a Blueprint document was in when applying config changes */

packages/meteor-lib/src/collections/ExpectedPackages.ts

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -70,13 +70,11 @@ export function getSideEffect(
7070
)
7171
}
7272

73-
export function getExpectedPackageFileName(
74-
expectedPackage: ExpectedPackage.Any
75-
): string | undefined {
73+
export function getExpectedPackageFileName(expectedPackage: ExpectedPackage.Any): string | undefined {
7674
if (expectedPackage.type === ExpectedPackage.PackageType.MEDIA_FILE) {
7775
return expectedPackage.content.filePath
7876
} else if (expectedPackage.type === ExpectedPackage.PackageType.QUANTEL_CLIP) {
79-
return expectedPackage.content.guid || expectedPackage.content.title
77+
return expectedPackage.content.guid || expectedPackage.content.title
8078
} else if (expectedPackage.type === ExpectedPackage.PackageType.JSON_DATA) {
8179
return undefined // Not supported
8280
} else if (expectedPackage.type === ExpectedPackage.PackageType.HTML_TEMPLATE) {
@@ -85,4 +83,4 @@ export function getExpectedPackageFileName(
8583
assertNever(expectedPackage)
8684
return undefined
8785
}
88-
}
86+
}

0 commit comments

Comments
 (0)