also looking at this
fix(android): resolve share() with whether the person picked an app
#210
12 files
+399
-56
| @@ -567,6 +567,7 @@ describe('Craft Android builder', () => { | ||
| 567 | 567 | ['OTA download', '_craftOTADownloadResolve', 1], |
| 568 | 568 | ['OTA apply', '_craftOTAApplyResolve', 1], |
| 569 | 569 | ['OTA rollback', '_craftOTARollbackResolve', 1], |
| 570 | ['share', '_craftShareResolve', 1], | |
| 570 | 571 | ] as const) { |
| 571 | 572 | const call = `window.__craftPromise('${channel}', '${resolver}',` |
| 572 | 573 | expect(bridge.match(new RegExp(call.replace(/[.*+?^${}()|[\]\\]/g, '\\$&'), 'g'))?.length).toBe(uses) |
| @@ -581,12 +582,12 @@ describe('Craft Android builder', () => { | ||
| 581 | 582 | const registrations = [...bridge.matchAll( |
| 582 | 583 | /window\.__craftPromise\(\s*'([^']+)'\s*,\s*'([^']+)'\s*,\s*'([^']+)'/g, |
| 583 | 584 | )].map(match => [match[1], match[2], match[3]] as const) |
| 584 | expect(registrations).toHaveLength(65) | |
| 585 | expect(registrations).toHaveLength(66) | |
| 585 | 586 | |
| 586 | 587 | const uniqueRegistrations = new Map( |
| 587 | 588 | registrations.map(registration => [registration.join('\0'), registration]), |
| 588 | 589 | ) |
| 589 | expect(uniqueRegistrations.size).toBe(46) | |
| 590 | expect(uniqueRegistrations.size).toBe(47) | |
| 590 | 591 | |
| 591 | 592 | const channels = new Set<string>() |
| 592 | 593 | const callbackOwners = new Map<string, string>() |
| @@ -599,8 +600,8 @@ describe('Craft Android builder', () => { | ||
| 599 | 600 | callbackOwners.set(callback, channel) |
| 600 | 601 | } |
| 601 | 602 | } |
| 602 | expect(channels.size).toBe(46) | |
| 603 | expect(callbackOwners.size).toBe(92) | |
| 603 | expect(channels.size).toBe(47) | |
| 604 | expect(callbackOwners.size).toBe(94) | |
| 604 | 605 | expect(bridge.match(/webView\.evaluateJavascript\(\s*"window\._craft[A-Za-z]+(?:Resolve|Reject)/g)).toBeNull() |
| 605 | 606 | expect(bridge.match(/window\._craft[A-Za-z]+(?:Resolve|Reject) = (?:resolve|reject)/g)).toBeNull() |
| 606 | 607 | expect(bridge).toContain('_craftShortcutsResolve({set: true, count: ${shortcuts.size}})') |