Skip to content

Commit 44a2efe

Browse files
mikehardymikehardy
committed
style(lint): result of yarn lint:js --fix after lint update
1 parent 583c1d1 commit 44a2efe

40 files changed

+232
-0
lines changed

‎packages/analytics/__tests__/analytics.test.ts

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,7 @@ describe('Analytics', function () {
6666
"'name' expected a string value",
6767
);
6868
});
69+
6970
it('throws if value is invalid', function () {
7071
// @ts-ignore test
7172
expect(() => firebase.analytics().setUserProperty('invertase3', 33.3333)).toThrowError(
@@ -79,6 +80,7 @@ describe('Analytics', function () {
7980
"'properties' expected an object of key/value pairs",
8081
);
8182
});
83+
8284
it('throws if property value is invalid', function () {
8385
const props = {
8486
test: '123',
@@ -91,6 +93,7 @@ describe('Analytics', function () {
9193
"'properties' value for parameter 'foo' is invalid",
9294
);
9395
});
96+
9497
it('throws if value is a number', function () {
9598
// @ts-ignore test
9699
expect(() => firebase.analytics().setUserProperties({ invertase1: 123 })).toThrowError(
@@ -104,6 +107,7 @@ describe('Analytics', function () {
104107
'The supplied arg must be an object of key/values.',
105108
);
106109
});
110+
107111
it('throws if consentSettings is invalid', function () {
108112
const consentSettings = {
109113
ad_storage: true,
@@ -116,6 +120,7 @@ describe('Analytics', function () {
116120
"'consentSettings' value for parameter 'foo' is invalid, expected a boolean.",
117121
);
118122
});
123+
119124
it('throws if one value of consentSettings is a number', function () {
120125
// @ts-ignore test
121126
expect(() => firebase.analytics().setConsent({ ad_storage: 123 })).toThrowError(
@@ -164,6 +169,7 @@ describe('Analytics', function () {
164169
'firebase.analytics().logScreenView(*):',
165170
);
166171
});
172+
167173
it('accepts arbitrary custom event parameters while rejecting defined parameters with wrong types', function () {
168174
expect(() => firebase.analytics().logScreenView({ foo: 'bar' })).not.toThrow();
169175
expect(() =>
@@ -180,6 +186,7 @@ describe('Analytics', function () {
180186
'firebase.analytics().logAddPaymentInfo(*):',
181187
);
182188
});
189+
183190
it('errors when compound values are not set', function () {
184191
expect(() =>
185192
firebase.analytics().logAddPaymentInfo({
@@ -206,6 +213,7 @@ describe('Analytics', function () {
206213
'firebase.analytics().logAddToCart(*):',
207214
);
208215
});
216+
209217
it('errors when compound values are not set', function () {
210218
expect(() =>
211219
firebase.analytics().logAddToCart({
@@ -222,6 +230,7 @@ describe('Analytics', function () {
222230
'firebase.analytics().logAddShippingInfo(*):',
223231
);
224232
});
233+
225234
it('errors when compound values are not set', function () {
226235
expect(() =>
227236
firebase.analytics().logAddShippingInfo({
@@ -238,6 +247,7 @@ describe('Analytics', function () {
238247
'firebase.analytics().logAddToWishlist(*):',
239248
);
240249
});
250+
241251
it('errors when compound values are not set', function () {
242252
expect(() =>
243253
firebase.analytics().logAddToWishlist({
@@ -254,6 +264,7 @@ describe('Analytics', function () {
254264
'firebase.analytics().logBeginCheckout(*):',
255265
);
256266
});
267+
257268
it('errors when compound values are not set', function () {
258269
expect(() =>
259270
firebase.analytics().logBeginCheckout({
@@ -270,6 +281,7 @@ describe('Analytics', function () {
270281
'firebase.analytics().logGenerateLead(*):',
271282
);
272283
});
284+
273285
it('errors when compound values are not set', function () {
274286
expect(() =>
275287
firebase.analytics().logGenerateLead({
@@ -439,6 +451,7 @@ describe('Analytics', function () {
439451
'firebase.analytics().logPurchase(*):',
440452
);
441453
});
454+
442455
it('errors when compound values are not set', function () {
443456
expect(() =>
444457
firebase.analytics().logPurchase({
@@ -472,6 +485,7 @@ describe('Analytics', function () {
472485
'firebase.analytics().logViewCart(*):',
473486
);
474487
});
488+
475489
it('errors when compound values are not set', function () {
476490
expect(() =>
477491
firebase.analytics().logViewCart({
@@ -488,6 +502,7 @@ describe('Analytics', function () {
488502
'firebase.analytics().logViewItem(*):',
489503
);
490504
});
505+
491506
it('errors when compound values are not set', function () {
492507
expect(() =>
493508
firebase.analytics().logViewItem({
@@ -513,6 +528,7 @@ describe('Analytics', function () {
513528
'firebase.analytics().logRemoveFromCart(*):',
514529
);
515530
});
531+
516532
it('errors when compound values are not set', function () {
517533
expect(() =>
518534
firebase.analytics().logRemoveFromCart({

‎packages/analytics/e2e/analytics.e2e.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -460,6 +460,7 @@ describe('analytics() modular', function () {
460460
};
461461
await firebase.analytics().setConsent(consentSettings);
462462
});
463+
463464
it('set ad_storage=false and analytics_storage=true on consentSettings', async function () {
464465
const consentSettings = {
465466
ad_storage: false,
@@ -492,6 +493,7 @@ describe('analytics() modular', function () {
492493
.analytics()
493494
.initiateOnDeviceConversionMeasurementWithPhoneNumber('+14155551212');
494495
});
496+
495497
it('handles mal-formatted phone number', async function () {
496498
try {
497499
await firebase
@@ -1053,6 +1055,7 @@ describe('analytics() modular', function () {
10531055
analyticsModular;
10541056
await initiateOnDeviceConversionMeasurementWithPhoneNumber(getAnalytics(), '+14155551212');
10551057
});
1058+
10561059
it('handles mal-formatted phone number', async function () {
10571060
try {
10581061
const { getAnalytics, initiateOnDeviceConversionMeasurementWithPhoneNumber } =
@@ -1093,6 +1096,7 @@ describe('analytics() modular', function () {
10931096
const { getAnalytics, setConsent } = analyticsModular;
10941097
await setConsent(getAnalytics(), consentSettings);
10951098
});
1099+
10961100
it('set ad_storage=false and analytics_storage=true on consentSettings', async function () {
10971101
const consentSettings = {
10981102
ad_storage: false,

‎packages/app-check/e2e/appcheck.e2e.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -199,6 +199,7 @@ describe('appCheck() modular', function () {
199199

200200
describe('modular', function () {
201201
var appCheckInstance;
202+
202203
before(async function () {
203204
const { initializeAppCheck } = appCheckModular;
204205

‎packages/app-distribution/e2e/app-distribution.e2e.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ describe('appDistribution()', function () {
4242
}
4343
});
4444
});
45+
4546
describe('signOutTester()', function () {
4647
it('signs out a tester', async function () {
4748
if (device.getPlatform() === 'ios') {
@@ -96,6 +97,7 @@ describe('appDistribution()', function () {
9697
}
9798
});
9899
});
100+
99101
describe('signOutTester()', function () {
100102
it('signs out a tester', async function () {
101103
const { getAppDistribution, signOutTester } = appDistributionModular;

‎packages/auth/e2e/auth.e2e.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@ describe('auth() modular', function () {
6868
.applyActionCode('fooby shooby dooby')
6969
.then($ => $);
7070
});
71+
7172
it('errors on invalid code', async function () {
7273
try {
7374
await firebase
@@ -1157,13 +1158,15 @@ describe('auth() modular', function () {
11571158
return newApp.delete();
11581159
});
11591160
});
1161+
11601162
describe('applyActionCode()', function () {
11611163
// Needs a different setup to work against the auth emulator
11621164
xit('works as expected', async function () {
11631165
const { applyActionCode, getAuth } = authModular;
11641166
const defaultAuth = getAuth(firebase.app());
11651167
await applyActionCode(defaultAuth, 'fooby shooby dooby').then($ => $);
11661168
});
1169+
11671170
it('errors on invalid code', async function () {
11681171
const { applyActionCode, getAuth } = authModular;
11691172
const defaultAuth = getAuth(firebase.app());

‎packages/auth/e2e/multiFactor.e2e.js

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -20,6 +20,7 @@ describe('multi-factor modular', function () {
2020
await Utils.sleep(50);
2121
}
2222
});
23+
2324
it('has no multi-factor information if not enrolled', async function () {
2425
await firebase.auth().signInWithEmailAndPassword(TEST_EMAIL, TEST_PASS);
2526
const { multiFactor } = firebase.auth().currentUser;
@@ -84,6 +85,7 @@ describe('multi-factor modular', function () {
8485
new Error('Multi-factor users need to handle an exception on sign-in'),
8586
);
8687
});
88+
8789
it('reports an error when providing an invalid sms code', async function () {
8890
if (device.getPlatform() === 'ios') {
8991
this.skip();
@@ -130,6 +132,7 @@ describe('multi-factor modular', function () {
130132
}
131133
return Promise.reject();
132134
});
135+
133136
it('reports an error when providing an invalid verification code', async function () {
134137
if (device.getPlatform() === 'ios') {
135138
this.skip();
@@ -165,6 +168,7 @@ describe('multi-factor modular', function () {
165168
}
166169
return Promise.reject();
167170
});
171+
168172
it('reports an error when using an unknown factor', async function () {
169173
const { email, password } = await createUserWithMultiFactor();
170174

@@ -253,6 +257,7 @@ describe('multi-factor modular', function () {
253257
}
254258
return Promise.resolve();
255259
});
260+
256261
it('can enroll new factor without display name', async function () {
257262
if (device.getPlatform() === 'ios') {
258263
this.skip();
@@ -283,6 +288,7 @@ describe('multi-factor modular', function () {
283288
}
284289
return Promise.resolve();
285290
});
291+
286292
it('can enroll multiple factors', async function () {
287293
if (device.getPlatform() === 'ios') {
288294
this.skip();
@@ -314,6 +320,7 @@ describe('multi-factor modular', function () {
314320

315321
return Promise.resolve();
316322
});
323+
317324
it('can not enroll the same factor twice', async function () {
318325
this.skip();
319326
// This test should probably be implemented but doesn't work:
@@ -389,6 +396,7 @@ describe('multi-factor modular', function () {
389396
}
390397
return Promise.reject();
391398
});
399+
392400
it('throws an error for unknown sessions', async function () {
393401
const { email, password } = await createUserWithMultiFactor();
394402
let resolver = null;
@@ -415,6 +423,7 @@ describe('multi-factor modular', function () {
415423
}
416424
return Promise.reject();
417425
});
426+
418427
it('throws an error for unknown verification code', async function () {
419428
const { email, password } = await createUserWithMultiFactor();
420429

@@ -482,6 +491,7 @@ describe('multi-factor modular', function () {
482491
new Error('Enrolling a second factor when using phone authentication is not supported.'),
483492
);
484493
});
494+
485495
it('can not enroll when phone number is missing + sign', async function () {
486496
await createVerifiedUser('[email protected]', 'test123');
487497
const multiFactorUser = firebase.auth().multiFactor(firebase.auth().currentUser);
@@ -516,6 +526,7 @@ describe('multi-factor modular', function () {
516526
await Utils.sleep(50);
517527
}
518528
});
529+
519530
it('has no multi-factor information if not enrolled', async function () {
520531
const { signInWithEmailAndPassword, getAuth } = authModular;
521532

@@ -804,6 +815,7 @@ describe('multi-factor modular', function () {
804815
}
805816
return Promise.resolve();
806817
});
818+
807819
it('can enroll new factor without display name', async function () {
808820
if (device.getPlatform() === 'ios') {
809821
this.skip();
@@ -842,6 +854,7 @@ describe('multi-factor modular', function () {
842854
}
843855
return Promise.resolve();
844856
});
857+
845858
it('can enroll multiple factors', async function () {
846859
if (device.getPlatform() === 'ios') {
847860
this.skip();
@@ -881,6 +894,7 @@ describe('multi-factor modular', function () {
881894

882895
return Promise.resolve();
883896
});
897+
884898
it('can not enroll the same factor twice', async function () {
885899
this.skip();
886900
// This test should probably be implemented but doesn't work:
@@ -962,6 +976,7 @@ describe('multi-factor modular', function () {
962976
}
963977
return Promise.reject();
964978
});
979+
965980
it('throws an error for unknown sessions', async function () {
966981
const { getAuth, signInWithEmailAndPassword, getMultiFactorResolver } = authModular;
967982

@@ -994,6 +1009,7 @@ describe('multi-factor modular', function () {
9941009
}
9951010
return Promise.reject();
9961011
});
1012+
9971013
it('throws an error for unknown verification code', async function () {
9981014
const { getAuth, signInWithEmailAndPassword, getMultiFactorResolver } = authModular;
9991015

@@ -1075,6 +1091,7 @@ describe('multi-factor modular', function () {
10751091
new Error('Enrolling a second factor when using phone authentication is not supported.'),
10761092
);
10771093
});
1094+
10781095
it('can not enroll when phone number is missing + sign', async function () {
10791096
const { getAuth, multiFactor } = authModular;
10801097

‎packages/auth/e2e/user.e2e.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -428,6 +428,7 @@ describe('auth().currentUser', function () {
428428
}
429429
await firebase.auth().currentUser.delete();
430430
});
431+
431432
it('should error if actionCodeSettings.url is not present', async function () {
432433
const random = Utils.randString(12, '#aA');
433434
const random2 = Utils.randString(12, '#aA');
@@ -672,6 +673,7 @@ describe('auth().currentUser', function () {
672673
return Promise.resolve();
673674
});
674675
});
676+
675677
describe('unlink()', function () {
676678
it('should unlink the email address', async function () {
677679
const random = Utils.randString(12, '#aA');

‎packages/crashlytics/e2e/crashlytics.e2e.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -141,6 +141,7 @@ describe('crashlytics()', function () {
141141
e.message.should.containEql("has been set to 'true', all reports are automatically sent");
142142
}
143143
});
144+
144145
it("checks device cache for unsent crashlytic's reports", async function () {
145146
await firebase.crashlytics().setCrashlyticsCollectionEnabled(false);
146147
const anyUnsentReports = await firebase.crashlytics().checkForUnsentReports();
@@ -327,6 +328,7 @@ describe('crashlytics()', function () {
327328
e.message.should.containEql("has been set to 'true', all reports are automatically sent");
328329
}
329330
});
331+
330332
it("checks device cache for unsent crashlytic's reports", async function () {
331333
const { getCrashlytics, setCrashlyticsCollectionEnabled, checkForUnsentReports } =
332334
crashlyticsModular;

‎packages/database/e2e/issues.e2e.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ describe('database issues', function () {
2323
before(async function () {
2424
await seed(TEST_PATH);
2525
});
26+
2627
after(async function () {
2728
await wipe(TEST_PATH);
2829
});

‎packages/database/e2e/query/endAt.e2e.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ describe('database().ref().endAt()', function () {
2323
before(async function () {
2424
await seed(TEST_PATH);
2525
});
26+
2627
after(async function () {
2728
await wipe(TEST_PATH);
2829
});

0 commit comments

Comments
 (0)