Skip to content

Commit

Permalink
fix: update ps:type test and remove ps:scale test
Browse files Browse the repository at this point in the history
  • Loading branch information
k80bowman committed Aug 20, 2024
1 parent c066a79 commit 8f798d0
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 27 deletions.
24 changes: 0 additions & 24 deletions packages/cli/test/unit/commands/ps/scale.unit.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -37,30 +37,6 @@ describe('ps:scale', function () {
api.done()
})

it('scales up a new large dyno size if feature flag is enabled', async function () {
const api = nock('https://api.heroku.com:443')
.get('/account/features/frontend-larger-dynos')
.reply(200, featureFlagPayload(true))
.patch('/apps/myapp/formation', {updates: [{type: 'web', quantity: '1', size: 'Performance-L-RAM'}]})
.reply(200, [{type: 'web', quantity: 1, size: 'Performance-L-RAM'}])
.get('/apps/myapp')
.reply(200, {name: 'myapp'})

await runCommand(Cmd, [
'--app',
'myapp',
'web=1:Performance-L-RAM',
])

api.done()

expect(stdout.output).to.eq('')
expect(stderr.output).to.equal(heredoc`
Scaling dynos...
Scaling dynos... done, now running web at 1:Performance-L-RAM
`)
})

it('shows formation with shield dynos for apps in a shielded private space', async function () {
const api = nock('https://api.heroku.com')
.get('/apps/myapp/formation')
Expand Down
4 changes: 1 addition & 3 deletions packages/cli/test/unit/commands/ps/type.unit.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -78,10 +78,8 @@ describe('ps:type', function () {
api.done()
})

it('switches to performance-l-ram dyno when feature flag is enabled', async function () {
it('switches to performance-l-ram dyno', async function () {
const api = nock('https://api.heroku.com')
.get('/account/features/frontend-larger-dynos')
.reply(200, featureFlagPayload(true))
.get('/apps/myapp')
.reply(200, app())
.get('/apps/myapp/formation')
Expand Down

0 comments on commit 8f798d0

Please sign in to comment.