Skip to content

Commit b0fead5

Browse files
committed
Update eslint
1 parent 5f76ba2 commit b0fead5

26 files changed

+2198
-1344
lines changed

firestore.rules.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -99,6 +99,7 @@ describe.skip('firestore', () => {
9999

100100
it('data validation', () => {
101101
expect(updateDoc(ownDocRef, {})).toAllow();
102+
102103
[
103104
{ contentRating: -1 },
104105
{ contentRating: 11 },

package-lock.json

+2,155-1,336
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+8-8
Original file line numberDiff line numberDiff line change
@@ -83,18 +83,18 @@
8383
"@types/node": "^22.13.10",
8484
"@types/nunjucks": "^3.2.6",
8585
"@types/polymer": "^1.2.16",
86-
"@typescript-eslint/eslint-plugin": "^7.4.0",
86+
"@typescript-eslint/eslint-plugin": "^7.18.0",
8787
"@typescript-eslint/parser": "^7.7.1",
8888
"@web/rollup-plugin-html": "^2.3.0",
8989
"concurrently": "^9.1.2",
90-
"eslint": "^8.57.0",
90+
"eslint": "^8.57.1",
9191
"eslint-config-prettier": "^9.1.0",
92-
"eslint-import-resolver-typescript": "^3.6.1",
93-
"eslint-plugin-html": "^8.0.0",
94-
"eslint-plugin-import": "^2.29.1",
95-
"eslint-plugin-jest": "^28.5.0",
96-
"eslint-plugin-jest-dom": "^5.4.0",
97-
"eslint-plugin-testing-library": "^6.2.0",
92+
"eslint-import-resolver-typescript": "^3.8.3",
93+
"eslint-plugin-html": "^8.1.2",
94+
"eslint-plugin-import": "^2.31.0",
95+
"eslint-plugin-jest": "^28.11.0",
96+
"eslint-plugin-jest-dom": "^5.5.0",
97+
"eslint-plugin-testing-library": "^6.5.0",
9898
"firebase-admin": "^12.7.0",
9999
"firebase-tools": "^13.33.0",
100100
"jest": "^29.7.0",

src/components/about-block.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ describe('about-block', () => {
3838
const { getByText } = within(shadowRootForWithin);
3939

4040
fireEvent.click(getByText(aboutBlock.callToAction.howItWas.label));
41+
4142
expect(mockToggleVideoDialogs).toHaveBeenCalledTimes(1);
4243
expect(mockToggleVideoDialogs).toHaveBeenCalledWith({
4344
title: aboutBlock.callToAction.howItWas.label,

src/components/auth-required.test.ts

+6
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,9 @@ describe('auth-required', () => {
4040

4141
it('shows unauthenticated prompt', () => {
4242
expect(shadowRoot.querySelector<HTMLDivElement>('mwc-button')).not.toHaveAttribute('hidden');
43+
4344
const slots = shadowRoot.querySelectorAll('slot');
45+
4446
expect(slots).toHaveLength(2);
4547
expect(slots[0]).not.toHaveAttribute('hidden');
4648
expect(slots[0]).toHaveAttribute('name', 'prompt');
@@ -52,6 +54,7 @@ describe('auth-required', () => {
5254

5355
it('opens dialog on tap', () => {
5456
fireEvent.click(shadowRoot.querySelector('mwc-button')!);
57+
5558
expect(mockOpenDialog).toHaveBeenCalledTimes(1);
5659
});
5760

@@ -61,8 +64,11 @@ describe('auth-required', () => {
6164
payload: { uid: '1' } as User,
6265
});
6366
await element.updateComplete;
67+
6468
expect(shadowRoot.querySelector<HTMLDivElement>('mwc-button')).toHaveAttribute('hidden');
69+
6570
const slots = shadowRoot.querySelectorAll('slot');
71+
6672
expect(slots).toHaveLength(2);
6773
expect(slots[0]).toHaveAttribute('hidden');
6874
expect(slots[1]).not.toHaveAttribute('hidden');

src/components/hero/hero-block.test.ts

+5
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ describe('hero-block', () => {
2222

2323
it('has default values', async () => {
2424
const { element, shadowRoot } = await fixture<HeroBlock>(html`<hero-block></hero-block>`);
25+
2526
expect(element.backgroundImage).toBe('');
2627
expect(element.backgroundColor).toBe('#fff');
2728
expect(element.fontColor).toBe('#000');
@@ -39,6 +40,7 @@ describe('hero-block', () => {
3940
hide-logo
4041
></hero-block>
4142
`);
43+
4244
expect(element.backgroundImage).toBe('/example.jpg');
4345
expect(element.backgroundColor).toBe('#000');
4446
expect(element.fontColor).toBe('#fff');
@@ -55,6 +57,7 @@ describe('hero-block', () => {
5557
</hero-block>
5658
`);
5759
const slots = shadowRoot.querySelectorAll('slot');
60+
5861
expect(slots).toHaveLength(2);
5962
expect(screen.getByText('default slot')).toBeVisible();
6063
expect(slots[0]).not.toHaveAttribute('name');
@@ -68,11 +71,13 @@ describe('hero-block', () => {
6871
const { shadowRoot } = await fixture<HeroBlock>(
6972
html`<hero-block background-image="/example.jpg"></hero-block>`,
7073
);
74+
7175
expect(shadowRoot.querySelector('.hero-image')).toHaveAttribute('src', '/example.jpg');
7276
});
7377

7478
it('setHeroSettings notifies', async () => {
7579
await fixture<HeroBlock>(html`<hero-block background-image="/example.jpg"></hero-block>`);
80+
7681
expect(mockSetHeroSettings).toHaveBeenCalledTimes(1);
7782
expect(mockSetHeroSettings).toHaveBeenCalledWith({
7883
backgroundColor: '#fff',

src/models/badge.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('badge', () => {
77
it('matches the shape of the default data', () => {
88
const badges: Badge[] = Object.values(data['speakers']['dmytro_danylyk']['badges']);
99
const keys: Array<keyof Badge> = ['description', 'link', 'name'];
10+
1011
expect(badges).toHaveLength(2);
1112
expect(allKeys(badges)).toStrictEqual(keys);
1213
});

src/models/day.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('day', () => {
77
it('matches the shape of the default data', () => {
88
const days: Day[] = Object.values(data['schedule']);
99
const keys: Array<keyof Day> = ['date', 'dateReadable', 'timeslots', 'tracks'];
10+
1011
expect(days).toHaveLength(2);
1112
expect(allKeys(days)).toStrictEqual(keys);
1213
});

src/models/feedback.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ describe('feedback', () => {
1212
},
1313
];
1414
const keys: Array<keyof FeedbackData> = ['comment', 'contentRating', 'styleRating'];
15+
1516
expect(feedback).toHaveLength(1);
1617
expect(allKeys(feedback)).toStrictEqual(keys);
1718
});

src/models/hero.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ describe('hero', () => {
1313
},
1414
];
1515
const keys: Array<keyof Hero> = ['backgroundColor', 'backgroundImage', 'fontColor', 'hideLogo'];
16+
1617
expect(heros).toHaveLength(1);
1718
expect(allKeys(heros)).toStrictEqual(keys);
1819
});

src/models/member.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,7 @@ describe('partner', () => {
1414
'socials',
1515
'title',
1616
];
17+
1718
expect(members).toHaveLength(8);
1819
expect(allKeys(members)).toStrictEqual(keys);
1920
});

src/models/partner-group.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('partner', () => {
77
it('matches the shape of the default data', () => {
88
const partner: PartnerGroupData[] = Object.values(data['partners']);
99
const keys: Array<keyof PartnerGroupData> = ['items', 'order', 'title'];
10+
1011
expect(partner).toHaveLength(2);
1112
expect(allKeys(partner)).toStrictEqual(keys);
1213
});

src/models/partner.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('partner', () => {
77
it('matches the shape of the default data', () => {
88
const posts: PartnerData[] = Object.values(data['partners'][1]!['items']);
99
const keys: Array<keyof PartnerData> = ['logoUrl', 'name', 'order', 'url'];
10+
1011
expect(posts).toHaveLength(11);
1112
expect(allKeys(posts)).toStrictEqual(keys);
1213
});

src/models/post.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ describe('post', () => {
1515
'source',
1616
'title',
1717
];
18+
1819
expect(posts).toHaveLength(5);
1920
expect(allKeys(posts)).toStrictEqual(keys);
2021
});

src/models/previous-session.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ describe('speaker', () => {
99
data['previousSpeakers']['adrian_kajda']['sessions']['2016'],
1010
);
1111
const keys: Array<keyof PreviousSession> = ['presentation', 'tags', 'title', 'videoId'];
12+
1213
expect(sessions).toHaveLength(1);
1314
expect(allKeys(sessions)).toStrictEqual(keys);
1415
});

src/models/previous-speaker.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ describe('speaker', () => {
1919
'socials',
2020
'title',
2121
];
22+
2223
expect(speakers).toHaveLength(22);
2324
expect(allKeys(speakers)).toStrictEqual(keys);
2425
});

src/models/session.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@ describe('session', () => {
1919
'title',
2020
'videoId',
2121
];
22+
2223
expect(sessions).toHaveLength(40);
2324
expect(allKeys(sessions)).toStrictEqual(keys);
2425
});

src/models/social.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('speaker', () => {
77
it('matches the shape of the default data', () => {
88
const socials: Social[] = Object.values(data['speakers']['aleksander_piotrowski']['socials']);
99
const keys: Array<keyof Social> = ['icon', 'link', 'name'];
10+
1011
expect(socials).toHaveLength(4);
1112
expect(allKeys(socials)).toStrictEqual(keys);
1213
});

src/models/speaker.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ describe('speaker', () => {
2323
'socials',
2424
'title',
2525
];
26+
2627
expect(speakers).toHaveLength(27);
2728
expect(allKeys(speakers)).toStrictEqual(keys);
2829
});

src/models/team.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ describe('partner', () => {
1212
it('matches the shape of the default data', () => {
1313
const teams: Team[] = Object.values(data['team']);
1414
const keys: Array<keyof Team> = ['members', 'title'];
15+
1516
expect(teams).toHaveLength(2);
1617
expect(allKeys(teams)).toStrictEqual(keys);
1718
});

src/models/ticket.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ describe('ticket', () => {
2020
'starts',
2121
'url',
2222
];
23+
2324
expect(tickets).toHaveLength(5);
2425
expect(allKeys(tickets)).toStrictEqual(keys);
2526
});

src/models/time.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('time', () => {
77
it('matches the shape of the default data', () => {
88
const times: Time[] = data['schedule']['2016-09-09']['timeslots'][3]!['sessions'];
99
const keys: Array<keyof Time> = ['extend', 'items'];
10+
1011
expect(times).toHaveLength(3);
1112
expect(allKeys(times)).toStrictEqual(keys);
1213
});

src/models/timeslot.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('timeslot', () => {
77
it('matches the shape of the default data', () => {
88
const days: Timeslot[] = data['schedule']['2016-09-09']['timeslots'];
99
const keys: Array<keyof Timeslot> = ['endTime', 'sessions', 'startTime'];
10+
1011
expect(days).toHaveLength(13);
1112
expect(allKeys(days)).toStrictEqual(keys);
1213
});

src/models/track.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('track', () => {
77
it('matches the shape of the default data', () => {
88
const days: Track[] = data['schedule']['2016-09-09']['tracks'];
99
const keys: Array<keyof Track> = ['title'];
10+
1011
expect(days).toHaveLength(3);
1112
expect(allKeys(days)).toStrictEqual(keys);
1213
});

src/models/video.test.ts

+1
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ describe('video', () => {
77
it('matches the shape of the default data', () => {
88
const videos: Video[] = Object.values(data['videos']);
99
const keys: Array<keyof Video> = ['speakers', 'thumbnail', 'title', 'youtubeId'];
10+
1011
expect(videos).toHaveLength(22);
1112
expect(allKeys(videos)).toStrictEqual(keys);
1213
});

src/utils/share.test.ts

+3
Original file line numberDiff line numberDiff line change
@@ -28,11 +28,13 @@ describe('share', () => {
2828

2929
it('shares to Twitter', () => {
3030
share({ currentTarget: fixture('twitter') } as any as PointerEvent);
31+
3132
expect(window.open).toHaveBeenCalledWith(twitterUrl, 'share', features(275));
3233
});
3334

3435
it('shares to Facebook', () => {
3536
share({ currentTarget: fixture('facebook') } as any as PointerEvent);
37+
3638
expect(window.open).toHaveBeenCalledWith(
3739
'https://www.facebook.com/sharer.php?u=http%3A%2F%2Flocalhost%2F&t=Awesome%20Schedule',
3840
'share',
@@ -42,6 +44,7 @@ describe('share', () => {
4244

4345
it('throws on unknown', () => {
4446
const event = { currentTarget: fixture('unknown') } as any as PointerEvent;
47+
4548
expect(() => share(event)).toThrow('Unknown share target');
4649
});
4750
});

0 commit comments

Comments
 (0)