Reset loaded patches when pressing "Connect"
This commit is contained in:
@@ -1170,13 +1170,23 @@ test.describe('Custom patches', () => {
|
|||||||
await page.click('#btn-mode-next');
|
await page.click('#btn-mode-next');
|
||||||
await expect(page.locator('#step-manual-version')).not.toBeHidden();
|
await expect(page.locator('#step-manual-version')).not.toBeHidden();
|
||||||
|
|
||||||
|
// Confirm version/model (sets patchesLoaded = true), then back out
|
||||||
|
await overrideFirmwareURLs(page);
|
||||||
|
await page.selectOption('#manual-version', '4.45.23646');
|
||||||
|
await expect(page.locator('#manual-model')).not.toBeHidden();
|
||||||
|
await page.selectOption('#manual-model', 'N428');
|
||||||
|
await page.click('#btn-manual-confirm');
|
||||||
|
await expect(page.locator('#step-patches')).not.toBeHidden();
|
||||||
|
|
||||||
// Go back all the way to the connect step
|
// Go back all the way to the connect step
|
||||||
|
await page.click('#btn-patches-back');
|
||||||
|
await expect(page.locator('#step-manual-version')).not.toBeHidden();
|
||||||
await page.click('#btn-manual-version-back');
|
await page.click('#btn-manual-version-back');
|
||||||
await expect(page.locator('#step-mode')).not.toBeHidden();
|
await expect(page.locator('#step-mode')).not.toBeHidden();
|
||||||
await page.click('#btn-mode-back');
|
await page.click('#btn-mode-back');
|
||||||
await expect(page.locator('#step-connect')).not.toBeHidden();
|
await expect(page.locator('#step-connect')).not.toBeHidden();
|
||||||
|
|
||||||
// Change mind: click "Connect to Kobo" — manualMode must be reset
|
// Change mind: click "Connect to Kobo" — manualMode and patchesLoaded must be reset
|
||||||
await injectMockDevice(page, { hasNickelMenu: false, overrideFirmware: true });
|
await injectMockDevice(page, { hasNickelMenu: false, overrideFirmware: true });
|
||||||
await page.click('#btn-connect');
|
await page.click('#btn-connect');
|
||||||
await expect(page.locator('#step-connect-instructions')).not.toBeHidden();
|
await expect(page.locator('#step-connect-instructions')).not.toBeHidden();
|
||||||
|
|||||||
@@ -369,8 +369,10 @@ function displayDeviceInfo(info) {
|
|||||||
btnConnect.addEventListener('click', () => {
|
btnConnect.addEventListener('click', () => {
|
||||||
// Reset any state from a previous manual-mode attempt so it does not
|
// Reset any state from a previous manual-mode attempt so it does not
|
||||||
// leak into the device-connected flow (e.g. back navigation would
|
// leak into the device-connected flow (e.g. back navigation would
|
||||||
// otherwise land on the manual version picker).
|
// otherwise land on the manual version picker, or stale patches from
|
||||||
|
// a manual selection would be used instead of the device's firmware).
|
||||||
state.manualMode = false;
|
state.manualMode = false;
|
||||||
|
state.patchesLoaded = false;
|
||||||
track('flow-start', { method: 'connect' });
|
track('flow-start', { method: 'connect' });
|
||||||
showStep(stepConnectInstructions);
|
showStep(stepConnectInstructions);
|
||||||
});
|
});
|
||||||
|
|||||||
Reference in New Issue
Block a user