Skip to content

Commit 6eb70d4

Browse files
authored
Merge pull request #3098 from codecrafters-io/fix-leaderboard-flag-tests
test: fix header leaderboard feature flag tests and add waitUntil
2 parents 2d7289e + 9bce696 commit 6eb70d4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/acceptance/header-test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ module('Acceptance | header-test', function (hooks) {
3232

3333
test('header should show generic leaderboard link if user has feature flag enabled and leaderboard entries', async function (assert) {
3434
const user = signIn(this.owner, this.server);
35-
user.update('featureFlags', { 'should-see-leaderboard': 'true' });
35+
user.update('featureFlags', { 'should-see-leaderboard': 'test' });
3636

3737
await catalogPage.visit();
3838
assert.true(catalogPage.header.hasLink('Leaderboard'), 'expect leaderboard link to be visible');
@@ -43,7 +43,7 @@ module('Acceptance | header-test', function (hooks) {
4343

4444
test('header should show custom leaderboard link if user has feature flag enabled', async function (assert) {
4545
const user = signIn(this.owner, this.server);
46-
user.update('featureFlags', { 'should-see-leaderboard': 'true' });
46+
user.update('featureFlags', { 'should-see-leaderboard': 'test' });
4747

4848
const python = this.server.schema.languages.findBy({ name: 'Python' });
4949

0 commit comments

Comments
 (0)