Skip to content

Commit 9d4706e

Browse files
committed
Don't display object count of 0
1 parent 8ec4e01 commit 9d4706e

File tree

2 files changed

+13
-3
lines changed

2 files changed

+13
-3
lines changed

packages/app/src/cli/services/bulk-operations/format-bulk-operation-status.test.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,12 @@ describe('formatBulkOperationStatus', () => {
3434
expect(result.value).toContain('(42 objects written)')
3535
})
3636

37+
test('formats RUNNING status without object count when count is 0', () => {
38+
const result = formatBulkOperationStatus(createMockOperation({status: 'RUNNING', type: 'QUERY', objectCount: '0'}))
39+
expect(result.value).toBe('Bulk operation in progress')
40+
expect(result.value).not.toContain('objects read')
41+
})
42+
3743
test('formats CREATED status', () => {
3844
const result = formatBulkOperationStatus(createMockOperation({status: 'CREATED'}))
3945
expect(result.value).toBe('Starting')

packages/app/src/cli/services/bulk-operations/format-bulk-operation-status.ts

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,13 @@ export function formatBulkOperationStatus(
66
): TokenizedString {
77
switch (operation.status) {
88
case 'RUNNING':
9-
return outputContent`Bulk operation in progress ${outputToken.gray(
10-
`(${String(operation.objectCount)} objects ${operation.type === 'MUTATION' ? 'written' : 'read'})`,
11-
)}`
9+
return outputContent`Bulk operation in progress${
10+
(operation.objectCount as number) > 0
11+
? outputToken.gray(
12+
` (${String(operation.objectCount)} objects ${operation.type === 'MUTATION' ? 'written' : 'read'})`,
13+
)
14+
: ''
15+
}`
1216
case 'CREATED':
1317
return outputContent`Starting`
1418
case 'COMPLETED':

0 commit comments

Comments
 (0)