Skip to content

Commit bc7909c

Browse files
authored
test: remove references to fully rolled out netlify_build_reduced_output (#6055)
* test: remove references to fully rolled out netlify_build_reduced_output * test: update snapshots * test: remove expectation of hidden logs from test
1 parent 700c36b commit bc7909c

File tree

44 files changed

+9
-690
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

44 files changed

+9
-690
lines changed

packages/build/src/core/feature_flags.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ const getFeatureFlag = function (name: string): FeatureFlags {
1717
export const DEFAULT_FEATURE_FLAGS: FeatureFlags = {
1818
buildbot_zisi_trace_nft: false,
1919
buildbot_zisi_esbuild_parser: false,
20-
netlify_build_reduced_output: false,
2120
netlify_build_updated_plugin_compatibility: false,
2221
netlify_build_plugin_system_log: false,
2322
}

packages/build/src/steps/run_step.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -124,13 +124,7 @@ export const runStep = async function ({
124124
// no-op
125125
}
126126

127-
let outputFlusher: OutputFlusher | undefined
128-
129-
if (featureFlags.netlify_build_reduced_output) {
130-
outputFlusher = new OutputFlusher(logPluginStart)
131-
} else {
132-
logPluginStart()
133-
}
127+
const outputFlusher = new OutputFlusher(logPluginStart)
134128

135129
const fireStep = getFireStep(packageName, coreStepId, event)
136130
const {

packages/build/tests/blobs_upload/tests.js

Lines changed: 4 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -80,10 +80,7 @@ test.serial(
8080
test.serial('Blobs upload step uploads files to deploy store (legacy API)', async (t) => {
8181
const fixture = await new Fixture('./fixtures/src_with_blobs_legacy').withCopyRoot({ git: false })
8282

83-
const {
84-
success,
85-
logs: { stdout },
86-
} = await fixture
83+
const { success } = await fixture
8784
.withFlags({ deployId: 'abc123', siteId: 'test', token: TOKEN, offline: true, cwd: fixture.repositoryRoot })
8885
.runBuildProgrammatic()
8986

@@ -117,17 +114,12 @@ test.serial('Blobs upload step uploads files to deploy store (legacy API)', asyn
117114
const blob3 = await store.getWithMetadata('nested/file.txt')
118115
t.is(blob3.data, 'file value')
119116
t.deepEqual(blob3.metadata, { some: 'metadata' })
120-
121-
t.true(stdout.join('\n').includes('Uploading blobs to deploy store'))
122117
})
123118

124119
test.serial('Blobs upload step uploads files to deploy store (legacy deploy config API)', async (t) => {
125120
const fixture = await new Fixture('./fixtures/src_with_blobs_legacy_deploy_config').withCopyRoot({ git: false })
126121

127-
const {
128-
success,
129-
logs: { stdout },
130-
} = await fixture
122+
const { success } = await fixture
131123
.withFlags({ deployId: 'abc123', siteId: 'test', token: TOKEN, offline: true, cwd: fixture.repositoryRoot })
132124
.runBuildProgrammatic()
133125
t.true(success)
@@ -160,17 +152,12 @@ test.serial('Blobs upload step uploads files to deploy store (legacy deploy conf
160152
const blob3 = await store.getWithMetadata('nested/file.txt')
161153
t.is(blob3.data, 'file value')
162154
t.deepEqual(blob3.metadata, { some: 'metadata' })
163-
164-
t.true(stdout.join('\n').includes('Uploading blobs to deploy store'))
165155
})
166156

167157
test.serial('Blobs upload step uploads files to deploy store', async (t) => {
168158
const fixture = await new Fixture('./fixtures/src_with_blobs').withCopyRoot({ git: false })
169159

170-
const {
171-
success,
172-
logs: { stdout },
173-
} = await fixture
160+
const { success } = await fixture
174161
.withFlags({ deployId: 'abc123', siteId: 'test', token: TOKEN, offline: true, cwd: fixture.repositoryRoot })
175162
.runBuildProgrammatic()
176163

@@ -206,8 +193,6 @@ test.serial('Blobs upload step uploads files to deploy store', async (t) => {
206193
const blob3 = await store.getWithMetadata('nested/blob')
207194
t.is(blob3.data, 'file value')
208195
t.deepEqual(blob3.metadata, { some: 'metadata' })
209-
210-
t.true(stdout.join('\n').includes('Uploading blobs to deploy store'))
211196
})
212197

213198
test.serial('Blobs upload step cancels deploy if blob metadata is malformed', async (t) => {
@@ -229,10 +214,7 @@ test.serial('Blobs upload step cancels deploy if blob metadata is malformed', as
229214
if (semver.gte(nodeVersion, '16.9.0')) {
230215
test.serial('monorepo > blobs upload, uploads files to deploy store', async (t) => {
231216
const fixture = await new Fixture('./fixtures/monorepo').withCopyRoot({ git: false })
232-
const {
233-
success,
234-
logs: { stdout },
235-
} = await fixture
217+
const { success } = await fixture
236218
.withFlags({ deployId: 'abc123', siteId: 'test', token: TOKEN, offline: true, packagePath: 'apps/app-1' })
237219
.runBuildProgrammatic()
238220

@@ -258,7 +240,5 @@ if (semver.gte(nodeVersion, '16.9.0')) {
258240
const blob3 = await store.getWithMetadata('nested/file.txt')
259241
t.is(blob3.data, 'file value')
260242
t.deepEqual(blob3.metadata, { some: 'metadata' })
261-
262-
t.true(stdout.join('\n').includes('Uploading blobs to deploy store'))
263243
})
264244
}

packages/build/tests/constants/snapshots/tests.js.md

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# Snapshot report for `packages/build/tests/constants/tests.js`
1+
# Snapshot report for `tests/constants/tests.js`
22

33
The actual snapshot is saved in `tests.js.snap`.
44

@@ -670,12 +670,6 @@ Generated by [AVA](https://avajs.dev).
670670
671671
> Loading plugins␊
672672
- ./plugin from netlify.toml␊
673-
674-
./plugin (onPreBuild event) ␊
675-
────────────────────────────────────────────────────────────────␊
676-
677-
678-
(./plugin onPreBuild completed in 1ms)␊
679673
Build step duration: ./plugin onPreBuild completed in 1ms␊
680674
681675
Functions bundling ␊
-3 Bytes
Binary file not shown.

packages/build/tests/core/snapshots/tests.js.md

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1458,10 +1458,6 @@ Generated by [AVA](https://avajs.dev).
14581458
> Loading plugins␊
14591459
- ./[email protected] from netlify.toml␊
14601460
1461-
./plugin.js (onBuild event) ␊
1462-
────────────────────────────────────────────────────────────────␊
1463-
1464-
14651461
Build canceled by ./plugin.js ␊
14661462
────────────────────────────────────────────────────────────────␊
14671463
-7 Bytes
Binary file not shown.
-7 Bytes
Binary file not shown.

packages/build/tests/edge_functions/snapshots/tests.js.md

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -165,12 +165,6 @@ Generated by [AVA](https://avajs.dev).
165165
166166
(./plugin onPreBuild completed in 1ms)␊
167167
168-
Edge Functions bundling ␊
169-
────────────────────────────────────────────────────────────────␊
170-
171-
172-
(Edge Functions bundling completed in 1ms)␊
173-
174168
Netlify Build Complete ␊
175169
────────────────────────────────────────────────────────────────␊
176170
-29 Bytes
Binary file not shown.

0 commit comments

Comments
 (0)