@sprucelabs/spruce-cli 24.1.2 → 24.1.4

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (25) hide show
  1. package/CHANGELOG.md +16 -0
  2. package/build/__tests__/behavioral/tests/migrationToInstance/StaticToInstanceTestFileMigrator.test.d.ts +2 -0
  3. package/build/__tests__/behavioral/tests/migrationToInstance/StaticToInstanceTestFileMigrator.test.js +16 -0
  4. package/build/__tests__/behavioral/tests/migrationToInstance/StaticToInstanceTestFileMigrator.test.js.map +1 -1
  5. package/build/__tests__/behavioral/tests/migrationToInstance/support/AbstractInstanceTest2.txt +24 -0
  6. package/build/__tests__/behavioral/tests/migrationToInstance/support/AbstractStaticTest2.txt +24 -0
  7. package/build/__tests__/behavioral/tests/migrationToInstance/support/InstanceTest.txt +8 -0
  8. package/build/__tests__/behavioral/tests/migrationToInstance/support/InstanceTest3.txt +1 -1
  9. package/build/__tests__/behavioral/tests/migrationToInstance/support/InstanceTest4.txt +531 -0
  10. package/build/__tests__/behavioral/tests/migrationToInstance/support/StaticTest.txt +8 -0
  11. package/build/__tests__/behavioral/tests/migrationToInstance/support/StaticTest3.txt +1 -1
  12. package/build/__tests__/behavioral/tests/migrationToInstance/support/StaticTest4.txt +530 -0
  13. package/build/tests/staticToInstanceMigration/StaticToInstanceTestFileMigrator.js +17 -13
  14. package/build/tests/staticToInstanceMigration/StaticToInstanceTestFileMigrator.js.map +1 -1
  15. package/package.json +3 -3
  16. package/src/__tests__/behavioral/tests/migrationToInstance/StaticToInstanceTestFileMigrator.test.ts +16 -0
  17. package/src/__tests__/behavioral/tests/migrationToInstance/support/AbstractInstanceTest2.txt +24 -0
  18. package/src/__tests__/behavioral/tests/migrationToInstance/support/AbstractStaticTest2.txt +24 -0
  19. package/src/__tests__/behavioral/tests/migrationToInstance/support/InstanceTest.txt +8 -0
  20. package/src/__tests__/behavioral/tests/migrationToInstance/support/InstanceTest3.txt +1 -1
  21. package/src/__tests__/behavioral/tests/migrationToInstance/support/InstanceTest4.txt +531 -0
  22. package/src/__tests__/behavioral/tests/migrationToInstance/support/StaticTest.txt +8 -0
  23. package/src/__tests__/behavioral/tests/migrationToInstance/support/StaticTest3.txt +1 -1
  24. package/src/__tests__/behavioral/tests/migrationToInstance/support/StaticTest4.txt +530 -0
  25. package/src/tests/staticToInstanceMigration/StaticToInstanceTestFileMigrator.ts +23 -16
@@ -0,0 +1,530 @@
1
+ import {
2
+ FormViewController,
3
+ SkillViewControllerId,
4
+ buttonAssert,
5
+ formAssert,
6
+ interactor,
7
+ navigationAssert,
8
+ vcAssert,
9
+ } from '@sprucelabs/heartwood-view-controllers'
10
+ import { selectAssert } from '@sprucelabs/schema'
11
+ import { SelectChoice } from '@sprucelabs/spruce-core-schemas'
12
+ import { FormCardViewController } from '@sprucelabs/spruce-form-utils'
13
+ import { eventFaker, fake, seed } from '@sprucelabs/spruce-test-fixtures'
14
+ import { assert, generateId, test } from '@sprucelabs/test-utils'
15
+ import GenerateSkillViewController, {
16
+ CurrentChallengeSchema,
17
+ GenerateStorySchema,
18
+ } from '../../../generation/Generate.svc'
19
+ import { storyElements } from '../../../generation/storyElements'
20
+ import AbstractEightBitTest from '../../support/AbstractEightBitTest'
21
+ import {
22
+ GenerateStoryTargetAndPayload,
23
+ GetStoryStatusTargetAndPayload,
24
+ } from '../../support/EventFaker'
25
+
26
+ @fake.login()
27
+ export default class GenerateSkillViewTest extends AbstractEightBitTest {
28
+ private static vc: SpyGenerateSkillView
29
+ private static checkStatusIntervalCb: undefined | (() => Promise<void>)
30
+ private static checkStatusIntervalMs: number | undefined
31
+ private static intervalId: string
32
+ private static passedIntervalIdToClear?: string
33
+
34
+ @seed('familyMembers', 3)
35
+ protected static async beforeEach(): Promise<void> {
36
+ await super.beforeEach()
37
+
38
+ this.views.setController(
39
+ 'eightbitstories.generate',
40
+ SpyGenerateSkillView
41
+ )
42
+ this.views.setController('forms.card', SpyFormCard)
43
+
44
+ this.vc = this.Vc()
45
+
46
+ await this.eventFaker.fakeListFamilyMembers(() => this.members.find({}))
47
+ await this.loadVc()
48
+
49
+ delete this.checkStatusIntervalCb
50
+ delete this.checkStatusIntervalMs
51
+ delete this.passedIntervalIdToClear
52
+
53
+ this.intervalId = generateId()
54
+
55
+ //@ts-ignore
56
+ GenerateSkillViewController.setInterval = (
57
+ cb: () => Promise<void>,
58
+ intervalMs: number
59
+ ) => {
60
+ this.checkStatusIntervalMs = intervalMs
61
+ this.checkStatusIntervalCb = cb
62
+ return this.intervalId
63
+ }
64
+
65
+ //@ts-ignore
66
+ GenerateSkillViewController.clearInterval = (id: string) => {
67
+ this.passedIntervalIdToClear = id
68
+ }
69
+
70
+ console.log(this.locationIds)
71
+ console.log(this.organizationIds)
72
+ }
73
+
74
+ private static get locationIds() {
75
+ console.log('here')
76
+ }
77
+
78
+ private static set organizationIds() {
79
+ console.log('here')
80
+ }
81
+
82
+ @test()
83
+ protected static async requiresLogin() {
84
+ await vcAssert.assertLoginIsRequired(this.vc)
85
+ }
86
+
87
+ @test()
88
+ protected static async rendersExpectedCards() {
89
+ vcAssert.assertSkillViewRendersCards(this.vc, [
90
+ 'elements',
91
+ 'members',
92
+ 'currentChallenge',
93
+ 'controls',
94
+ ])
95
+ }
96
+
97
+ @test()
98
+ protected static async controlsCardRendersExpectedButtons() {
99
+ buttonAssert.cardRendersButtons(this.controlsVc, ['back', 'generate'])
100
+ }
101
+
102
+ @test()
103
+ protected static async rendersAlertAndRedirectsIfNoMembers() {
104
+ await this.eventFaker.fakeListFamilyMembers(() => [])
105
+ this.vc = this.Vc()
106
+ await vcAssert.assertRendersAlertThenRedirects({
107
+ vc: this.vc,
108
+ router: this.views.getRouter(),
109
+ destination: {
110
+ id: 'eightbitstories.root',
111
+ },
112
+ action: () => this.loadVc(),
113
+ })
114
+ }
115
+
116
+ @test()
117
+ protected static async clickingBackGoesBackToRoot() {
118
+ await vcAssert.assertActionRedirects({
119
+ action: () => interactor.clickButton(this.controlsVc, 'back'),
120
+ destination: {
121
+ id: 'eightbitstories.root',
122
+ },
123
+ router: this.views.getRouter(),
124
+ })
125
+ }
126
+
127
+ @test()
128
+ protected static elementsAndMembersCardsRendersForms() {
129
+ formAssert.cardRendersForm(this.elementsVc)
130
+ formAssert.cardRendersForm(this.membersVc)
131
+ formAssert.cardRendersForm(this.currentChallengeVc)
132
+ }
133
+
134
+ @test()
135
+ protected static async formCardsDoNotRenderButtons() {
136
+ assert.isFalse(this.elementsFormVc.getShouldRenderSubmitControls())
137
+ assert.isFalse(this.membersFormVc.getShouldRenderSubmitControls())
138
+ assert.isFalse(
139
+ this.currentChallengeFormVc.getShouldRenderSubmitControls()
140
+ )
141
+ }
142
+
143
+ @test()
144
+ protected static async elementsFormRendersExpectedFields() {
145
+ formAssert.formRendersFields(this.elementsFormVc, ['elements'])
146
+ }
147
+
148
+ @test()
149
+ protected static async elementsFormRendersExpectedChoices() {
150
+ const schema = this.elementsFormVc.getSchema()
151
+ selectAssert.assertSelectChoicesMatch(
152
+ schema.fields.elements.options.choices,
153
+ storyElements.map((element) => element.id)
154
+ )
155
+ }
156
+
157
+ @test()
158
+ protected static async rendersElementsAsTags() {
159
+ formAssert.formFieldRendersAs(this.elementsFormVc, 'elements', 'tags')
160
+ }
161
+
162
+ @test()
163
+ protected static async membersFormRendersExpectedFields() {
164
+ formAssert.formRendersFields(this.membersFormVc, ['members'])
165
+ }
166
+
167
+ @test()
168
+ protected static async membersFormRendersAsTags() {
169
+ formAssert.formFieldRendersAs(this.membersFormVc, 'members', 'tags')
170
+ }
171
+
172
+ @test()
173
+ protected static async membersRendersExpectedChoices() {
174
+ const members = await this.getAllMembers()
175
+ const expected = members.map((member) => member.id)
176
+
177
+ const schema = this.membersFormVc.getSchema()
178
+ selectAssert.assertSelectChoicesMatch(
179
+ schema.fields.members.options.choices as SelectChoice[],
180
+ expected
181
+ )
182
+ }
183
+
184
+ @test()
185
+ protected static async currentChallengeFormRendersAsExpected() {
186
+ formAssert.formRendersField(
187
+ this.currentChallengeFormVc,
188
+ 'currentChallenge'
189
+ )
190
+ formAssert.formFieldRendersAs(
191
+ this.currentChallengeFormVc,
192
+ 'currentChallenge',
193
+ 'textarea'
194
+ )
195
+ }
196
+
197
+ @test()
198
+ protected static async clickingGenerateSetsControlsToBusy() {
199
+ await this.eventFaker.fakeGenerateStory(() => {})
200
+
201
+ await this.selectFirstMember()
202
+ await this.selectFirstElement()
203
+
204
+ const promise = this.clickGenerateAndAssertRedirect()
205
+ this.assertFooterIsBusy()
206
+
207
+ await promise
208
+ }
209
+
210
+ @test()
211
+ protected static async rendersAlertIfFailsToGenerateStory() {
212
+ await eventFaker.makeEventThrow(
213
+ 'eightbitstories.generate-story::v2023_09_05'
214
+ )
215
+
216
+ const alertVc = await vcAssert.assertRendersAlert(this.vc, () =>
217
+ this.clickGenerate()
218
+ )
219
+
220
+ this.assertFooterIsBusy()
221
+
222
+ await alertVc.hide()
223
+
224
+ this.assertFooterIsNotBusy()
225
+ }
226
+
227
+ @test('submits selected members and elements 1', [0], [0])
228
+ @test('submits selected members and elements 2', [1], [2])
229
+ @test('submits selected members and elements 3', [0, 1], [2, 3])
230
+ protected static async generatePassesSelectedMembersAndElements(
231
+ memberIdxs: number[],
232
+ elementIdxs: number[]
233
+ ) {
234
+ let passedPayload: GenerateStoryTargetAndPayload['payload'] | undefined
235
+
236
+ await this.eventFaker.fakeGenerateStory(({ payload }) => {
237
+ passedPayload = payload
238
+ })
239
+
240
+ const selectedMembers = await this.selectMembers(memberIdxs)
241
+ const selectedElements = await this.selectElements(elementIdxs)
242
+
243
+ const currentChallenge = generateId()
244
+ await this.currentChallengeFormVc.setValue(
245
+ 'currentChallenge',
246
+ currentChallenge
247
+ )
248
+
249
+ await this.clickGenerateAndAssertRedirect()
250
+
251
+ assert.isEqualDeep(passedPayload, {
252
+ familyMembers: selectedMembers,
253
+ storyElements: selectedElements,
254
+ currentChallenge,
255
+ storyHash: this.vc.getHash(),
256
+ })
257
+ }
258
+
259
+ @test()
260
+ protected static async generatingStoryRedirectsToStoryWithArgs() {
261
+ await this.eventFaker.fakeGenerateStory()
262
+
263
+ await this.selectFirstElement()
264
+ await this.selectFirstMember()
265
+
266
+ const destination = {
267
+ id: 'eightbitstories.story' as SkillViewControllerId,
268
+ args: {
269
+ story: generateId(),
270
+ },
271
+ }
272
+
273
+ await this.clickGenerateAndAssertRedirect(destination)
274
+ }
275
+
276
+ @test()
277
+ protected static async callingDestroyRemovesDidGenerateListener() {
278
+ await this.vc.destroy()
279
+
280
+ await eventFaker.handleReactiveEvent(
281
+ 'eightbitstories.did-generate-story::v2023_09_05'
282
+ )
283
+
284
+ await this.emitDidGenerate()
285
+ }
286
+
287
+ @test()
288
+ protected static async rendersNullNavigation() {
289
+ navigationAssert.skillViewDoesNotRenderNavigation(this.vc)
290
+ }
291
+
292
+ @test()
293
+ protected static async checksForGeneratedStoryAfterSubmitting() {
294
+ let passedTarget: GetStoryStatusTargetAndPayload['target'] | undefined
295
+ await this.eventFaker.fakeGetStoryGenerationStatus(({ target }) => {
296
+ passedTarget = target
297
+ })
298
+
299
+ await this.fakeGenerateSelectEverythingClickGenerateAndInvokeIntervalCb()
300
+
301
+ assert.isEqualDeep(passedTarget, {
302
+ storyHash: this.vc.getHash(),
303
+ })
304
+ }
305
+
306
+ @test()
307
+ protected static async passesExpectedIntervalToChecksAfterSubmit() {
308
+ await this.eventFaker.fakeGenerateStory()
309
+ await this.selectElementFamilyMemberAndClickGenerate()
310
+ assert.isEqual(this.checkStatusIntervalMs, 1000 * 10)
311
+ }
312
+
313
+ @test()
314
+ protected static async doesNotSetIntervalIfGenerateThrows() {
315
+ await eventFaker.makeEventThrow(
316
+ 'eightbitstories.generate-story::v2023_09_05'
317
+ )
318
+
319
+ await vcAssert.assertRendersAlert(this.vc, () =>
320
+ this.selectElementFamilyMemberAndClickGenerate()
321
+ )
322
+ assert.isUndefined(
323
+ this.checkStatusIntervalCb,
324
+ 'should not have been set'
325
+ )
326
+ }
327
+
328
+ @test()
329
+ protected static async redirectsIfResponseIsStoryGenerated() {
330
+ const storyId = generateId()
331
+ await this.eventFaker.fakeGetStoryGenerationStatus(() => {
332
+ return {
333
+ status: 'ready',
334
+ storyId,
335
+ }
336
+ })
337
+
338
+ await this.fakeGenerateSelectEverythingAndClickGenerate()
339
+ await vcAssert.assertActionRedirects({
340
+ action: () => this.checkStatusIntervalCb?.(),
341
+ destination: {
342
+ id: 'eightbitstories.story',
343
+ args: {
344
+ story: storyId,
345
+ },
346
+ },
347
+ router: this.views.getRouter(),
348
+ })
349
+ }
350
+
351
+ @test()
352
+ protected static async clearsTimeoutOnBlur() {
353
+ await this.fakeGenerateSelectEverythingAndClickGenerate()
354
+ assert.isFalsy(this.passedIntervalIdToClear)
355
+ await interactor.blur(this.vc)
356
+ assert.isEqual(
357
+ this.passedIntervalIdToClear,
358
+ this.intervalId,
359
+ 'did not pass response to setInterval to clearInterval'
360
+ )
361
+ }
362
+
363
+ private static async fakeGenerateSelectEverythingClickGenerateAndInvokeIntervalCb() {
364
+ await this.fakeGenerateSelectEverythingAndClickGenerate()
365
+ await this.checkStatusIntervalCb?.()
366
+ }
367
+
368
+ private static async fakeGenerateSelectEverythingAndClickGenerate() {
369
+ await this.eventFaker.fakeGenerateStory()
370
+ await this.selectElementFamilyMemberAndClickGenerate()
371
+ }
372
+
373
+ private static async selectElementFamilyMemberAndClickGenerate() {
374
+ await this.selectFirstElement()
375
+ await this.selectFirstMember()
376
+ await this.clickGenerate()
377
+ }
378
+
379
+ private static async clickGenerateAndAssertRedirect(destination?: {
380
+ id: SkillViewControllerId
381
+ args: { story: string }
382
+ }) {
383
+ await vcAssert.assertActionRedirects({
384
+ action: async () => {
385
+ await this.clickGenerate()
386
+ await this.emitDidGenerate(destination?.args?.story)
387
+ },
388
+ router: this.views.getRouter(),
389
+ destination,
390
+ })
391
+ }
392
+
393
+ private static async emitDidGenerate(storyId?: string) {
394
+ await this.fakedClient.emitAndFlattenResponses(
395
+ 'eightbitstories.did-generate-story::v2023_09_05',
396
+ {
397
+ target: {
398
+ personId: generateId(),
399
+ },
400
+ payload: {
401
+ storyId: storyId ?? generateId(),
402
+ },
403
+ }
404
+ )
405
+ }
406
+
407
+ private static async selectFirstElement() {
408
+ const selectedElement = await this.selectElement(0)
409
+ return selectedElement
410
+ }
411
+
412
+ private static async selectElement(idx: number) {
413
+ const selectedElements = await this.selectElements([idx])
414
+ return selectedElements[0]
415
+ }
416
+
417
+ private static async selectElements(allIdxs: number[]) {
418
+ const selectedElements = allIdxs.map((idx) => storyElements[idx].id)
419
+ await this.elementsFormVc.setValue('elements', selectedElements)
420
+ return selectedElements
421
+ }
422
+
423
+ private static async selectFirstMember() {
424
+ const selectedMember = await this.selectMember(0)
425
+ return selectedMember
426
+ }
427
+
428
+ private static async selectMember(idx: number) {
429
+ const selectedMembers = await this.selectMembers([idx])
430
+ return selectedMembers[0]
431
+ }
432
+
433
+ private static async selectMembers(allIdxs: number[]) {
434
+ const members = await this.getAllMembers()
435
+ const selectedMembers = allIdxs.map((idx) => members[idx].id)
436
+ await this.membersFormVc.setValue('members', selectedMembers as any)
437
+ return selectedMembers
438
+ }
439
+
440
+ private static async getAllMembers() {
441
+ return await this.members.find({})
442
+ }
443
+
444
+ private static assertFooterIsNotBusy() {
445
+ assert.isFalse(this.getIsFooterBusy())
446
+ }
447
+
448
+ private static assertFooterIsBusy() {
449
+ assert.isTrue(this.getIsFooterBusy())
450
+ }
451
+
452
+ private static getIsFooterBusy(): boolean | null | undefined {
453
+ return this.controlsVc.getFooter()?.isBusy
454
+ }
455
+
456
+ private static async clickGenerate() {
457
+ await interactor.clickButton(this.controlsVc, 'generate')
458
+ }
459
+
460
+ private static get membersFormVc() {
461
+ return this.vc.getMembersFormVc()
462
+ }
463
+
464
+ private static get elementsFormVc() {
465
+ return this.vc.getElementsFormVc()
466
+ }
467
+
468
+ private static async loadVc() {
469
+ await this.views.load(this.vc)
470
+ }
471
+
472
+ private static get membersVc() {
473
+ return this.vc.getMembersVc()
474
+ }
475
+
476
+ private static get currentChallengeVc() {
477
+ return this.vc.getCurrentChallengeVc()
478
+ }
479
+
480
+ private static get currentChallengeFormVc() {
481
+ return this.currentChallengeVc.getFormVc() as FormViewController<CurrentChallengeSchema>
482
+ }
483
+
484
+ private static get elementsVc() {
485
+ return this.vc.getElementsVc()
486
+ }
487
+
488
+ private static get controlsVc() {
489
+ return this.vc.getControlsCardVc()
490
+ }
491
+
492
+ private static Vc(): SpyGenerateSkillView {
493
+ return this.views.Controller(
494
+ 'eightbitstories.generate',
495
+ {}
496
+ ) as SpyGenerateSkillView
497
+ }
498
+ }
499
+
500
+ class SpyGenerateSkillView extends GenerateSkillViewController {
501
+ public getHash() {
502
+ return this.storyHash!
503
+ }
504
+
505
+ public getCurrentChallengeVc() {
506
+ return this.currentChallengeVc
507
+ }
508
+
509
+ public getMembersFormVc() {
510
+ return this.getMembersVc().getFormVc() as FormViewController<GenerateStorySchema>
511
+ }
512
+ public getElementsFormVc() {
513
+ return this.getElementsVc().getFormVc() as FormViewController<GenerateStorySchema>
514
+ }
515
+ public getElementsVc() {
516
+ return this.elementsVc as SpyFormCard
517
+ }
518
+ public getMembersVc() {
519
+ return this.membersVc as SpyFormCard
520
+ }
521
+ public getControlsCardVc() {
522
+ return this.controlsVc
523
+ }
524
+ }
525
+
526
+ class SpyFormCard extends FormCardViewController {
527
+ public getFormVc() {
528
+ return this.formVc
529
+ }
530
+ }
@@ -12,14 +12,14 @@ class StaticToInstanceTestFileMigratorImpl {
12
12
  // that has the `@test()` decorator
13
13
  // 1b. If the contents include `export default abstract class`,
14
14
  // remove `static` from all methods
15
- const includesAbstractExport = contents.includes('export default abstract class');
16
- let cleanedUp = includesAbstractExport
15
+ const isAbstractTest = /export\s+default\s+(?:abstract\s+class\b|class\s+(Abstract\w*))/m.test(contents);
16
+ let cleanedUp = isAbstractTest
17
17
  ? contents.replaceAll(' static ', ' ')
18
18
  : contents.replace(
19
19
  // Matches @test() or @seed(...) followed (on next line) by optional visibility and `static`.
20
- /(@(?:test\(\)|seed\([^)]*\))\s*\n\s*(?:public|protected)\s+)static\s+/g, '$1');
20
+ /(@(?:(?:test|seed)\([\s\S]*?\))\s*\n\s*(?:public|protected)\s+)static\s+/g, '$1');
21
21
  // 2. Add `@suite()` above `export default class` if it's not already present
22
- if (!cleanedUp.includes('@suite')) {
22
+ if (!isAbstractTest && !cleanedUp.includes('@suite')) {
23
23
  cleanedUp = cleanedUp.replace(/export default class/, '@suite()\nexport default class');
24
24
  }
25
25
  // 3. Ensure `suite` is imported from `@sprucelabs/test-utils`
@@ -41,9 +41,11 @@ class StaticToInstanceTestFileMigratorImpl {
41
41
  // 4. lifecicle methods
42
42
  const methods = ['beforeEach', 'afterEach'];
43
43
  for (const method of methods) {
44
- cleanedUp = cleanedUp.replace(`protected static async ${method}()`, `protected async ${method}()`);
44
+ cleanedUp = cleanedUp.replace(`static async ${method}()`, `async ${method}()`);
45
45
  }
46
46
  cleanedUp = this.fixNonNullAssertions(cleanedUp);
47
+ cleanedUp = cleanedUp.replaceAll('= >', '=>');
48
+ cleanedUp = cleanedUp.replaceAll('! =', ' =');
47
49
  return cleanedUp;
48
50
  }
49
51
  hasSuiteImport(text) {
@@ -51,8 +53,9 @@ class StaticToInstanceTestFileMigratorImpl {
51
53
  return pattern.test(text);
52
54
  }
53
55
  findThisCalls(contents) {
54
- // Matches `this.myProp` if followed by space, punctuation, parentheses, or end of string
55
- const thisPropertyRegex = /this\.(\w+)(?=[\s.(),;]|$)/g;
56
+ // Matches either `this.myProp` or `delete this.myProp`
57
+ // if followed by space, punctuation, parentheses, or end of string
58
+ const thisPropertyRegex = /(?:delete\s+)?this\.(\w+)(?=[\s.(),;]|$)/g;
56
59
  const names = [];
57
60
  let match;
58
61
  while ((match = thisPropertyRegex.exec(contents)) !== null) {
@@ -83,19 +86,20 @@ class StaticToInstanceTestFileMigratorImpl {
83
86
  return `${g1 ?? ''}${asyncPart}${accessorPart}${g4}(`;
84
87
  });
85
88
  /**
86
- * 2) Remove `static` from property declarations and add a non-null assertion.
89
+ * 2) Remove `static` from property declarations and add a non-null assertion
90
+ * if the property is not optional.
87
91
  * e.g.
88
92
  * private static myProp: Type => private myProp!: Type
93
+ * private static passedIntervalIdToClear?: string => private passedIntervalIdToClear?: string
89
94
  */
90
95
  const propertyPattern = new RegExp(`((?:public|protected|private)?\\s+)?` + // group 1: optional visibility
91
96
  `static\\s+` + // literal "static "
92
- `(${name})` + // group 2: the property name
97
+ `(${name})(\\?)?` + // group 2: property name, group 3: optional "?"
93
98
  `(?=[\\s=:\\[;]|$)`, // lookahead: space, '=', ':', '[', ';', or end-of-string
94
99
  'g');
95
- updated = updated.replace(propertyPattern, (match, g1, g2) => {
96
- // g1 = "private " / "public " / "protected " or empty
97
- // g2 = property name
98
- return `${g1 ?? ''}${g2}!`;
100
+ updated = updated.replace(propertyPattern, (match, g1, g2, g3) => {
101
+ // If the property is optional (g3 is "?"), leave it. Otherwise, add a non-null assertion.
102
+ return `${g1 ?? ''}${g2}${g3 !== undefined ? g3 : '!'}`;
99
103
  });
100
104
  return updated;
101
105
  }
@@ -1 +1 @@
1
- {"version":3,"file":"StaticToInstanceTestFileMigrator.js","sourceRoot":"","sources":["../../../src/tests/staticToInstanceMigration/StaticToInstanceTestFileMigrator.ts"],"names":[],"mappings":";;AAAA,+CAAkD;AAElD,MAAqB,oCAAoC;IAG9C,MAAM,CAAC,KAAK,CAA6C;IAEzD,MAAM,CAAC,QAAQ;QAClB,OAAO,IAAI,CAAC,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,EAAE,CAAA;IACrC,CAAC;IAEM,OAAO,CAAC,QAAgB;QAC3B,IAAA,sBAAa,EAAC,EAAE,QAAQ,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAA;QAEzC,wEAAwE;QACxE,sCAAsC;QACtC,+DAA+D;QAC/D,uCAAuC;QACvC,MAAM,sBAAsB,GAAG,QAAQ,CAAC,QAAQ,CAC5C,+BAA+B,CAClC,CAAA;QACD,IAAI,SAAS,GAAG,sBAAsB;YAClC,CAAC,CAAC,QAAQ,CAAC,UAAU,CAAC,UAAU,EAAE,GAAG,CAAC;YACtC,CAAC,CAAC,QAAQ,CAAC,OAAO;YACZ,6FAA6F;YAC7F,wEAAwE,EACxE,IAAI,CACP,CAAA;QAEP,6EAA6E;QAC7E,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;YAChC,SAAS,GAAG,SAAS,CAAC,OAAO,CACzB,sBAAsB,EACtB,gCAAgC,CACnC,CAAA;QACL,CAAC;QAED,8DAA8D;QAC9D,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC,SAAS,CAAC,EAAE,CAAC;YAClC,IAAI,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC/B,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,EAAE,eAAe,CAAC,CAAA;YAC5D,CAAC;iBAAM,IAAI,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBACtC,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,EAAE,eAAe,CAAC,CAAA;YAC5D,CAAC;iBAAM,CAAC;gBACJ,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAA;YAC/D,CAAC;QACL,CAAC;QAED,MAAM,aAAa,GAAG,IAAI,CAAC,aAAa,CAAC,SAAS,CAAC,CAAA;QACnD,KAAK,MAAM,IAAI,IAAI,aAAa,EAAE,CAAC;YAC/B,SAAS,GAAG,IAAI,CAAC,2BAA2B,CAAC,SAAS,EAAE,IAAI,CAAC,CAAA;QACjE,CAAC;QAED,uBAAuB;QACvB,MAAM,OAAO,GAAG,CAAC,YAAY,EAAE,WAAW,CAAC,CAAA;QAC3C,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;YAC3B,SAAS,GAAG,SAAS,CAAC,OAAO,CACzB,0BAA0B,MAAM,IAAI,EACpC,mBAAmB,MAAM,IAAI,CAChC,CAAA;QACL,CAAC;QAED,SAAS,GAAG,IAAI,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAA;QAEhD,OAAO,SAAS,CAAA;IACpB,CAAC;IAEO,cAAc,CAAC,IAAY;QAC/B,MAAM,OAAO,GAAG,IAAI,MAAM,CACtB,yFAAyF,CAC5F,CAAA;QACD,OAAO,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAC7B,CAAC;IAEO,aAAa,CAAC,QAAgB;QAClC,yFAAyF;QACzF,MAAM,iBAAiB,GAAG,6BAA6B,CAAA;QACvD,MAAM,KAAK,GAAa,EAAE,CAAA;QAC1B,IAAI,KAA6B,CAAA;QAEjC,OAAO,CAAC,KAAK,GAAG,iBAAiB,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YACzD,MAAM,QAAQ,GAAG,KAAK,CAAC,CAAC,CAAC,CAAA;YACzB,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC5B,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;YACxB,CAAC;QACL,CAAC;QAED,OAAO,KAAK,CAAA;IAChB,CAAC;IAEO,2BAA2B,CAC/B,QAAgB,EAChB,IAAY;QAEZ;;;;;WAKG;QACH,MAAM,aAAa,GAAG,IAAI,MAAM,CAC5B,sCAAsC,GAAG,UAAU;YAC/C,YAAY,GAAG,oBAAoB;YACnC,kBAAkB,GAAG,oBAAoB;YACzC,oBAAoB,GAAG,2BAA2B;YAClD,IAAI,IAAI,UAAU,EAAE,gCAAgC;QACxD,GAAG,CACN,CAAA;QACD,IAAI,OAAO,GAAG,QAAQ,CAAC,OAAO,CAC1B,aAAa,EACb,CAAC,KAAK,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE;YACtB,MAAM,SAAS,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;YACpC,MAAM,YAAY,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;YACvC,2CAA2C;YAC3C,OAAO,GAAG,EAAE,IAAI,EAAE,GAAG,SAAS,GAAG,YAAY,GAAG,EAAE,GAAG,CAAA;QACzD,CAAC,CACJ,CAAA;QAED;;;;WAIG;QACH,MAAM,eAAe,GAAG,IAAI,MAAM,CAC9B,sCAAsC,GAAG,+BAA+B;YACpE,YAAY,GAAG,oBAAoB;YACnC,IAAI,IAAI,GAAG,GAAG,6BAA6B;YAC3C,mBAAmB,EAAE,yDAAyD;QAClF,GAAG,CACN,CAAA;QACD,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,eAAe,EAAE,CAAC,KAAK,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE;YACzD,sDAAsD;YACtD,qBAAqB;YACrB,OAAO,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,GAAG,CAAA;QAC9B,CAAC,CAAC,CAAA;QAEF,OAAO,OAAO,CAAA;IAClB,CAAC;IAEO,oBAAoB,CAAC,QAAgB;QACzC,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,CAAC,IAAI,CAAC,CAAA;QAElC,MAAM,aAAa,GACf,uFAAuF,CAAA;QAE3F,MAAM,YAAY,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,YAAY,EAAE,EAAE;YAC5C,iCAAiC;YACjC,IAAI,YAAY,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAClC,OAAO,YAAY,CAAA;YACvB,CAAC;YAED,MAAM,KAAK,GAAG,YAAY,CAAC,KAAK,CAAC,aAAa,CAAC,CAAA;YAC/C,IAAI,CAAC,KAAK,EAAE,CAAC;gBACT,OAAO,YAAY,CAAA;YACvB,CAAC;YAED,IAAI,CACA,AADC,EAED,iBAAiB,EACjB,UAAU,EACV,YAAY,GAAG,EAAE,EACjB,QAAQ,EACR,WAAW,EACX,QAAQ,EACR,UAAU,EACb,GAAG,KAAK,CAAA;YAET,yCAAyC;YACzC,QAAQ,GAAG,QAAQ,CAAC,IAAI,EAAE,CAAA;YAE1B,IAAI,UAAU,EAAE,CAAC;gBACb,2CAA2C;gBAC3C,WAAW,GAAG,EAAE,CAAA;gBAEhB,4BAA4B;gBAC5B,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,CAAA;gBAEzC,2CAA2C;gBAC3C,sCAAsC;gBACtC,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,OAAO,EAAE,KAAK,CAAC,CAAA;YACnD,CAAC;iBAAM,CAAC;gBACJ,0BAA0B;gBAC1B,WAAW,GAAG,GAAG,CAAA;YACrB,CAAC;YAED,+CAA+C;YAC/C,MAAM,OAAO,GAAG,GAAG,iBAAiB,GAAG,UAAU,GAAG,YAAY,IAAI,QAAQ,GAAG,WAAW,KAAK,QAAQ,GAAG,UAAU,IAAI,EAAE,EAAE,CAAA;YAC5H,OAAO,OAAO,CAAA;QAClB,CAAC,CAAC,CAAA;QAEF,OAAO,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;CACJ;AA9LD,uDA8LC"}
1
+ {"version":3,"file":"StaticToInstanceTestFileMigrator.js","sourceRoot":"","sources":["../../../src/tests/staticToInstanceMigration/StaticToInstanceTestFileMigrator.ts"],"names":[],"mappings":";;AAAA,+CAAkD;AAElD,MAAqB,oCAAoC;IAG9C,MAAM,CAAC,KAAK,CAA6C;IAEzD,MAAM,CAAC,QAAQ;QAClB,OAAO,IAAI,CAAC,IAAI,CAAC,KAAK,IAAI,IAAI,CAAC,EAAE,CAAA;IACrC,CAAC;IAEM,OAAO,CAAC,QAAgB;QAC3B,IAAA,sBAAa,EAAC,EAAE,QAAQ,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAA;QAEzC,wEAAwE;QACxE,sCAAsC;QACtC,+DAA+D;QAC/D,uCAAuC;QACvC,MAAM,cAAc,GAChB,kEAAkE,CAAC,IAAI,CACnE,QAAQ,CACX,CAAA;QAEL,IAAI,SAAS,GAAG,cAAc;YAC1B,CAAC,CAAC,QAAQ,CAAC,UAAU,CAAC,UAAU,EAAE,GAAG,CAAC;YACtC,CAAC,CAAC,QAAQ,CAAC,OAAO;YACZ,6FAA6F;YAC7F,2EAA2E,EAC3E,IAAI,CACP,CAAA;QAEP,6EAA6E;QAC7E,IAAI,CAAC,cAAc,IAAI,CAAC,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;YACnD,SAAS,GAAG,SAAS,CAAC,OAAO,CACzB,sBAAsB,EACtB,gCAAgC,CACnC,CAAA;QACL,CAAC;QAED,8DAA8D;QAC9D,IAAI,CAAC,IAAI,CAAC,cAAc,CAAC,SAAS,CAAC,EAAE,CAAC;YAClC,IAAI,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC/B,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,EAAE,eAAe,CAAC,CAAA;YAC5D,CAAC;iBAAM,IAAI,SAAS,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBACtC,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,QAAQ,EAAE,eAAe,CAAC,CAAA;YAC5D,CAAC;iBAAM,CAAC;gBACJ,SAAS,GAAG,SAAS,CAAC,OAAO,CAAC,OAAO,EAAE,mBAAmB,CAAC,CAAA;YAC/D,CAAC;QACL,CAAC;QAED,MAAM,aAAa,GAAG,IAAI,CAAC,aAAa,CAAC,SAAS,CAAC,CAAA;QACnD,KAAK,MAAM,IAAI,IAAI,aAAa,EAAE,CAAC;YAC/B,SAAS,GAAG,IAAI,CAAC,2BAA2B,CAAC,SAAS,EAAE,IAAI,CAAC,CAAA;QACjE,CAAC;QAED,uBAAuB;QACvB,MAAM,OAAO,GAAG,CAAC,YAAY,EAAE,WAAW,CAAC,CAAA;QAC3C,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;YAC3B,SAAS,GAAG,SAAS,CAAC,OAAO,CACzB,gBAAgB,MAAM,IAAI,EAC1B,SAAS,MAAM,IAAI,CACtB,CAAA;QACL,CAAC;QAED,SAAS,GAAG,IAAI,CAAC,oBAAoB,CAAC,SAAS,CAAC,CAAA;QAEhD,SAAS,GAAG,SAAS,CAAC,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,CAAA;QAC7C,SAAS,GAAG,SAAS,CAAC,UAAU,CAAC,KAAK,EAAE,IAAI,CAAC,CAAA;QAE7C,OAAO,SAAS,CAAA;IACpB,CAAC;IAEO,cAAc,CAAC,IAAY;QAC/B,MAAM,OAAO,GAAG,IAAI,MAAM,CACtB,yFAAyF,CAC5F,CAAA;QACD,OAAO,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAC7B,CAAC;IAEO,aAAa,CAAC,QAAgB;QAClC,uDAAuD;QACvD,mEAAmE;QACnE,MAAM,iBAAiB,GAAG,2CAA2C,CAAA;QACrE,MAAM,KAAK,GAAa,EAAE,CAAA;QAC1B,IAAI,KAA6B,CAAA;QAEjC,OAAO,CAAC,KAAK,GAAG,iBAAiB,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC;YACzD,MAAM,QAAQ,GAAG,KAAK,CAAC,CAAC,CAAC,CAAA;YACzB,IAAI,CAAC,KAAK,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC5B,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAA;YACxB,CAAC;QACL,CAAC;QAED,OAAO,KAAK,CAAA;IAChB,CAAC;IAEO,2BAA2B,CAC/B,QAAgB,EAChB,IAAY;QAEZ;;;;;WAKG;QACH,MAAM,aAAa,GAAG,IAAI,MAAM,CAC5B,sCAAsC,GAAG,UAAU;YAC/C,YAAY,GAAG,oBAAoB;YACnC,kBAAkB,GAAG,oBAAoB;YACzC,oBAAoB,GAAG,2BAA2B;YAClD,IAAI,IAAI,UAAU,EAAE,gCAAgC;QACxD,GAAG,CACN,CAAA;QACD,IAAI,OAAO,GAAG,QAAQ,CAAC,OAAO,CAC1B,aAAa,EACb,CAAC,KAAK,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE;YACtB,MAAM,SAAS,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;YACpC,MAAM,YAAY,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,GAAG,CAAC,CAAC,CAAC,EAAE,CAAA;YACvC,2CAA2C;YAC3C,OAAO,GAAG,EAAE,IAAI,EAAE,GAAG,SAAS,GAAG,YAAY,GAAG,EAAE,GAAG,CAAA;QACzD,CAAC,CACJ,CAAA;QAED;;;;;;WAMG;QACH,MAAM,eAAe,GAAG,IAAI,MAAM,CAC9B,sCAAsC,GAAG,+BAA+B;YACpE,YAAY,GAAG,oBAAoB;YACnC,IAAI,IAAI,SAAS,GAAG,gDAAgD;YACpE,mBAAmB,EAAE,yDAAyD;QAClF,GAAG,CACN,CAAA;QACD,OAAO,GAAG,OAAO,CAAC,OAAO,CAAC,eAAe,EAAE,CAAC,KAAK,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE;YAC7D,0FAA0F;YAC1F,OAAO,GAAG,EAAE,IAAI,EAAE,GAAG,EAAE,GAAG,EAAE,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,EAAE,CAAA;QAC3D,CAAC,CAAC,CAAA;QAEF,OAAO,OAAO,CAAA;IAClB,CAAC;IAEO,oBAAoB,CAAC,QAAgB;QACzC,MAAM,KAAK,GAAG,QAAQ,CAAC,KAAK,CAAC,IAAI,CAAC,CAAA;QAElC,MAAM,aAAa,GACf,uFAAuF,CAAA;QAE3F,MAAM,YAAY,GAAG,KAAK,CAAC,GAAG,CAAC,CAAC,YAAY,EAAE,EAAE;YAC5C,iCAAiC;YACjC,IAAI,YAAY,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAClC,OAAO,YAAY,CAAA;YACvB,CAAC;YAED,MAAM,KAAK,GAAG,YAAY,CAAC,KAAK,CAAC,aAAa,CAAC,CAAA;YAC/C,IAAI,CAAC,KAAK,EAAE,CAAC;gBACT,OAAO,YAAY,CAAA;YACvB,CAAC;YAED,IAAI,CACA,AADC,EAED,iBAAiB,EACjB,UAAU,EACV,YAAY,GAAG,EAAE,EACjB,QAAQ,EACR,WAAW,EACX,QAAQ,EACR,UAAU,EACb,GAAG,KAAK,CAAA;YAET,yCAAyC;YACzC,QAAQ,GAAG,QAAQ,CAAC,IAAI,EAAE,CAAA;YAE1B,IAAI,UAAU,EAAE,CAAC;gBACb,2CAA2C;gBAC3C,WAAW,GAAG,EAAE,CAAA;gBAEhB,4BAA4B;gBAC5B,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,CAAA;gBAEzC,2CAA2C;gBAC3C,sCAAsC;gBACtC,UAAU,GAAG,UAAU,CAAC,OAAO,CAAC,OAAO,EAAE,KAAK,CAAC,CAAA;YACnD,CAAC;iBAAM,CAAC;gBACJ,0BAA0B;gBAC1B,WAAW,GAAG,GAAG,CAAA;YACrB,CAAC;YAED,+CAA+C;YAC/C,MAAM,OAAO,GAAG,GAAG,iBAAiB,GAAG,UAAU,GAAG,YAAY,IAAI,QAAQ,GAAG,WAAW,KAAK,QAAQ,GAAG,UAAU,IAAI,EAAE,EAAE,CAAA;YAC5H,OAAO,OAAO,CAAA;QAClB,CAAC,CAAC,CAAA;QAEF,OAAO,YAAY,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;IAClC,CAAC;CACJ;AArMD,uDAqMC"}
package/package.json CHANGED
@@ -4,7 +4,7 @@
4
4
  "publishConfig": {
5
5
  "access": "public"
6
6
  },
7
- "version": "24.1.2",
7
+ "version": "24.1.4",
8
8
  "skill": {
9
9
  "namespace": "spruce-cli",
10
10
  "upgradeIgnoreList": [
@@ -84,7 +84,7 @@
84
84
  "@sprucelabs/spruce-core-schemas": "^40.1.645",
85
85
  "@sprucelabs/spruce-event-utils": "^40.2.44",
86
86
  "@sprucelabs/spruce-skill-utils": "^31.2.60",
87
- "@sprucelabs/spruce-templates": "^24.1.2",
87
+ "@sprucelabs/spruce-templates": "^24.1.4",
88
88
  "@typescript-eslint/eslint-plugin": "^7.7.1",
89
89
  "@typescript-eslint/parser": "^7.7.1",
90
90
  "cfonts": "^3.3.0",
@@ -571,5 +571,5 @@
571
571
  "open"
572
572
  ]
573
573
  },
574
- "gitHead": "0d5c0969ffb26aec1498c147e8676846c75f3013"
574
+ "gitHead": "707d106956af7e757498d86e84108a3f62915db6"
575
575
  }
@@ -147,6 +147,14 @@ protected suiteMethod() {}`
147
147
  this.assertMigratedEquals(before, expected)
148
148
  }
149
149
 
150
+ @test()
151
+ protected properlyMigrationsAbstractTests2() {
152
+ const before = this.readTestFile('AbstractStaticTest2.txt')
153
+ const expected = this.readTestFile('AbstractInstanceTest2.txt')
154
+
155
+ this.assertMigratedEquals(before, expected)
156
+ }
157
+
150
158
  @test()
151
159
  protected canMigrateMoreComplexTest() {
152
160
  const before = this.readTestFile('StaticTest2.txt')
@@ -163,6 +171,14 @@ protected suiteMethod() {}`
163
171
  this.assertMigratedEquals(before, expected)
164
172
  }
165
173
 
174
+ @test()
175
+ protected canMigrateAnotherComplexTest2() {
176
+ const before = this.readTestFile('StaticTest4.txt')
177
+ const expected = this.readTestFile('InstanceTest4.txt')
178
+
179
+ this.assertMigratedEquals(before, expected)
180
+ }
181
+
166
182
  private readTestFile(name: string) {
167
183
  return diskUtil.readFile(this.resolveTestClassPath(name))
168
184
  }