Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 849fa5f

Browse files
author
DimitarTachev
committedSep 18, 2018
feat: support hmr in hooks
1 parent 518da37 commit 849fa5f

25 files changed

+175
-47
lines changed
 

‎lib/commands/appstore-upload.ts

+5-1
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,11 @@ export class PublishIOS implements ICommand {
5656
if (!ipaFilePath) {
5757
const platform = this.$devicePlatformsConstants.iOS;
5858
// No .ipa path provided, build .ipa on out own.
59-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: !!this.$options.bundle, release: this.$options.release };
59+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = {
60+
bundle: !!this.$options.bundle,
61+
release: this.$options.release,
62+
useHotModuleReload: false
63+
};
6064
const platformInfo: IPreparePlatformInfo = {
6165
platform,
6266
appFilesUpdaterOptions,

‎lib/commands/build.ts

+5-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,11 @@ export abstract class BuildCommandBase extends ValidatePlatformCommandBase {
1515

1616
public async executeCore(args: string[]): Promise<void> {
1717
const platform = args[0].toLowerCase();
18-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: !!this.$options.bundle, release: this.$options.release };
18+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = {
19+
bundle: !!this.$options.bundle,
20+
release: this.$options.release,
21+
useHotModuleReload: this.$options.hmr
22+
};
1923
const platformInfo: IPreparePlatformInfo = {
2024
platform,
2125
appFilesUpdaterOptions,

‎lib/commands/clean-app.ts

+5-1
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,11 @@ export class CleanAppCommandBase extends ValidatePlatformCommandBase implements
1616
}
1717

1818
public async execute(args: string[]): Promise<void> {
19-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: !!this.$options.bundle, release: this.$options.release };
19+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = {
20+
bundle: !!this.$options.bundle,
21+
release: this.$options.release,
22+
useHotModuleReload: false
23+
};
2024
const platformInfo: IPreparePlatformInfo = {
2125
appFilesUpdaterOptions,
2226
platform: this.platform.toLowerCase(),

‎lib/commands/prepare.ts

+5-1
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,11 @@ export class PrepareCommand extends ValidatePlatformCommandBase implements IComm
1313
}
1414

1515
public async execute(args: string[]): Promise<void> {
16-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: !!this.$options.bundle, release: this.$options.release };
16+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = {
17+
bundle: !!this.$options.bundle,
18+
release: this.$options.release,
19+
useHotModuleReload: this.$options.hmr
20+
};
1721
const platformInfo: IPreparePlatformInfo = {
1822
platform: args[0],
1923
appFilesUpdaterOptions,

‎lib/commands/preview.ts

+3-2
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,11 @@ export class PreviewCommand implements ICommand {
1919
bundle: !!this.$options.bundle,
2020
release: this.$options.release,
2121
env: this.$options.env,
22-
timeout: this.$options.timeout
22+
timeout: this.$options.timeout,
23+
useHotModuleReload: this.$options.hmr
2324
});
2425

25-
await this.$playgroundQrCodeGenerator.generateQrCodeForCurrentApp(this.$options.hmr);
26+
await this.$playgroundQrCodeGenerator.generateQrCodeForCurrentApp({ useHotModuleReload: this.$options.hmr });
2627
}
2728

2829
public async canExecute(args: string[]): Promise<boolean> {

‎lib/declarations.d.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -486,7 +486,7 @@ interface IEnvOptions {
486486

487487
interface IAndroidBuildOptionsSettings extends IAndroidReleaseOptions, IRelease { }
488488

489-
interface IAppFilesUpdaterOptions extends IBundle, IRelease, IOptionalWatchAllFiles { }
489+
interface IAppFilesUpdaterOptions extends IBundle, IRelease, IOptionalWatchAllFiles, IHasUseHotModuleReloadOption { }
490490

491491
interface IPlatformBuildData extends IAppFilesUpdaterOptions, IBuildConfig, IEnvOptions { }
492492

‎lib/definitions/livesync.d.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -180,7 +180,7 @@ interface IHasUseHotModuleReloadOption {
180180
/**
181181
* Defines if the hot module reload should be used.
182182
*/
183-
useHotModuleReload?: boolean;
183+
useHotModuleReload: boolean;
184184
}
185185

186186
interface ILatestAppPackageInstalledSettings extends IDictionary<IDictionary<boolean>> { /* empty */ }

‎lib/definitions/preview-app-livesync.d.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ declare global {
1010
interface IPreviewAppLiveSyncData extends IProjectDir, IAppFilesUpdaterOptionsComposition, IEnvOptions { }
1111

1212
interface IPreviewSdkService {
13-
getQrCodeUrl(options: { useHmr: boolean }): string;
13+
getQrCodeUrl(options: IHasUseHotModuleReloadOption): string;
1414
connectedDevices: Device[];
1515
initialize(getInitialFiles: (device: Device) => Promise<FilesPayload>): void;
1616
applyChanges(filesPayload: FilesPayload): Promise<void>;
@@ -29,6 +29,6 @@ declare global {
2929
interface IPlaygroundQrCodeGenerator {
3030
generateQrCodeForiOS(): Promise<void>;
3131
generateQrCodeForAndroid(): Promise<void>;
32-
generateQrCodeForCurrentApp(useHmr: boolean): Promise<void>;
32+
generateQrCodeForCurrentApp(options: IHasUseHotModuleReloadOption): Promise<void>;
3333
}
3434
}

‎lib/helpers/deploy-command-helper.ts

+5-1
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,11 @@ export class DeployCommandHelper implements IDeployCommandHelper {
77
}
88

99
public getDeployPlatformInfo(platform: string): IDeployPlatformInfo {
10-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: !!this.$options.bundle, release: this.$options.release };
10+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = {
11+
bundle: !!this.$options.bundle,
12+
release: this.$options.release,
13+
useHotModuleReload: this.$options.hmr
14+
};
1115
const deployOptions: IDeployPlatformOptions = {
1216
clean: this.$options.clean,
1317
device: this.$options.device,

‎lib/helpers/livesync-command-helper.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,8 @@ export class LiveSyncCommandHelper implements ILiveSyncCommandHelper {
156156
platform: currentPlatform,
157157
appFilesUpdaterOptions: {
158158
bundle: !!this.$options.bundle,
159-
release: this.$options.release
159+
release: this.$options.release,
160+
useHotModuleReload: this.$options.hmr
160161
},
161162
deployOptions,
162163
buildPlatform: this.$platformService.buildPlatform.bind(this.$platformService),

‎lib/helpers/preview-command-helper.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -50,7 +50,7 @@ export class PreviewCommandHelper implements IPreviewCommandHelper {
5050
await this.$playgroundQrCodeGenerator.generateQrCodeForiOS();
5151
return;
5252
case "c":
53-
await this.$playgroundQrCodeGenerator.generateQrCodeForCurrentApp(this.$options.hmr);
53+
await this.$playgroundQrCodeGenerator.generateQrCodeForCurrentApp({ useHotModuleReload: this.$options.hmr });
5454
return;
5555
}
5656
}

‎lib/services/livesync/ios-livesync-service.ts

+9-3
Original file line numberDiff line numberDiff line change
@@ -14,8 +14,8 @@ export class IOSLiveSyncService extends PlatformLiveSyncServiceBase implements I
1414
$logger: ILogger,
1515
$projectFilesProvider: IProjectFilesProvider,
1616
private $iOSDebuggerPortService: IIOSDebuggerPortService) {
17-
super($fs, $logger, $platformsData, $projectFilesManager, $devicePathProvider, $projectFilesProvider);
18-
}
17+
super($fs, $logger, $platformsData, $projectFilesManager, $devicePathProvider, $projectFilesProvider);
18+
}
1919

2020
public async fullSync(syncInfo: IFullSyncInfo): Promise<ILiveSyncResultInfo> {
2121
const device = syncInfo.device;
@@ -60,7 +60,13 @@ export class IOSLiveSyncService extends PlatformLiveSyncServiceBase implements I
6060
public liveSyncWatchAction(device: Mobile.IDevice, liveSyncInfo: ILiveSyncWatchInfo): Promise<ILiveSyncResultInfo> {
6161
if (liveSyncInfo.isReinstalled) {
6262
// In this case we should execute fullsync because iOS Runtime requires the full content of app dir to be extracted in the root of sync dir.
63-
return this.fullSync({ projectData: liveSyncInfo.projectData, device, syncAllFiles: liveSyncInfo.syncAllFiles, liveSyncDeviceInfo: liveSyncInfo.liveSyncDeviceInfo, watch: true });
63+
return this.fullSync({
64+
projectData: liveSyncInfo.projectData,
65+
device, syncAllFiles: liveSyncInfo.syncAllFiles,
66+
liveSyncDeviceInfo: liveSyncInfo.liveSyncDeviceInfo,
67+
watch: true,
68+
useHotModuleReload: liveSyncInfo.useHotModuleReload
69+
});
6470
} else {
6571
return super.liveSyncWatchAction(device, liveSyncInfo);
6672
}

‎lib/services/livesync/livesync-service.ts

+8-4
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,8 @@ export class LiveSyncService extends EventEmitter implements IDebugLiveSyncServi
328328
this.$previewAppLiveSyncService.initialize({
329329
appFilesUpdaterOptions: {
330330
bundle: liveSyncData.bundle,
331-
release: liveSyncData.release
331+
release: liveSyncData.release,
332+
useHotModuleReload: liveSyncData.useHotModuleReload
332333
},
333334
env: liveSyncData.env,
334335
projectDir: projectData.projectDir
@@ -389,7 +390,8 @@ export class LiveSyncService extends EventEmitter implements IDebugLiveSyncServi
389390
appFilesUpdaterOptions: {
390391
bundle: options.bundle,
391392
release: options.release,
392-
watchAllFiles: options.liveSyncData.watchAllFiles
393+
watchAllFiles: options.liveSyncData.watchAllFiles,
394+
useHotModuleReload: options.liveSyncData.useHotModuleReload
393395
},
394396
projectData: options.projectData,
395397
env: options.env,
@@ -589,7 +591,8 @@ export class LiveSyncService extends EventEmitter implements IDebugLiveSyncServi
589591
await this.$previewAppLiveSyncService.syncFiles({
590592
appFilesUpdaterOptions: {
591593
bundle: liveSyncData.bundle,
592-
release: liveSyncData.release
594+
release: liveSyncData.release,
595+
useHotModuleReload: liveSyncData.useHotModuleReload
593596
},
594597
env: liveSyncData.env,
595598
projectDir: projectData.projectDir
@@ -691,7 +694,8 @@ export class LiveSyncService extends EventEmitter implements IDebugLiveSyncServi
691694
appFilesUpdaterOptions: {
692695
bundle: liveSyncData.bundle,
693696
release: liveSyncData.release,
694-
watchAllFiles: liveSyncData.watchAllFiles
697+
watchAllFiles: liveSyncData.watchAllFiles,
698+
useHotModuleReload: liveSyncData.useHotModuleReload
695699
},
696700
platforms
697701
},

‎lib/services/livesync/playground/preview-sdk-service.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ export class PreviewSdkService implements IPreviewSdkService {
1313
private $config: IConfiguration) {
1414
}
1515

16-
public getQrCodeUrl(options: { useHmr: boolean }): string {
17-
const hmrValue = options.useHmr ? "1" : "0";
16+
public getQrCodeUrl(options: IHasUseHotModuleReloadOption): string {
17+
const hmrValue = options.useHotModuleReload ? "1" : "0";
1818
return `nsplay://boot?instanceId=${this.instanceId}&pKey=${PubnubKeys.PUBLISH_KEY}&sKey=${PubnubKeys.SUBSCRIBE_KEY}&template=play-ng&hmr=${hmrValue}`;
1919
}
2020

‎lib/services/livesync/playground/qr-code-generator.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,8 @@ export class PlaygroundQrCodeGenerator implements IPlaygroundQrCodeGenerator {
1818
await this.generateQrCode(PlaygroundStoreUrls.GOOGLE_PLAY_URL);
1919
}
2020

21-
public async generateQrCodeForCurrentApp(useHmr: boolean): Promise<void> {
22-
await this.generateQrCode(this.$previewSdkService.getQrCodeUrl(useHmr));
21+
public async generateQrCodeForCurrentApp(options: IHasUseHotModuleReloadOption): Promise<void> {
22+
await this.generateQrCode(this.$previewSdkService.getQrCodeUrl(options));
2323
}
2424

2525
private async generateQrCode(url: string): Promise<void> {

‎lib/services/platform-environment-requirements.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -195,7 +195,8 @@ export class PlatformEnvironmentRequirements implements IPlatformEnvironmentRequ
195195
bundle: !!options.bundle,
196196
release: options.release,
197197
env: options.env,
198-
timeout: options.timeout
198+
timeout: options.timeout,
199+
useHotModuleReload: options.hmr
199200
});
200201
}
201202
}

‎lib/services/platform-service.ts

+2-1
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,8 @@ export class PlatformService extends EventEmitter implements IPlatformService {
213213
provision: config.provision,
214214
teamId: config.teamId,
215215
nativePlatformStatus,
216-
skipModulesNativeCheck: preparePlatformInfo.skipModulesNativeCheck
216+
skipModulesNativeCheck: preparePlatformInfo.skipModulesNativeCheck,
217+
useHotModuleReload: appFilesUpdaterOptions.useHotModuleReload
217218
}
218219
});
219220

‎lib/services/test-execution-service.ts

+14-4
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,11 @@ class TestExecutionService implements ITestExecutionService {
5555
const socketIoJsUrl = `http://localhost:${this.$options.port}/socket.io/socket.io.js`;
5656
const socketIoJs = (await this.$httpClient.httpRequest(socketIoJsUrl)).body;
5757
this.$fs.writeFile(path.join(projectDir, TestExecutionService.SOCKETIO_JS_FILE_NAME), socketIoJs);
58-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: !!this.$options.bundle, release: this.$options.release };
58+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = {
59+
bundle: !!this.$options.bundle,
60+
release: this.$options.release,
61+
useHotModuleReload: this.$options.hmr
62+
};
5963
const preparePlatformInfo: IPreparePlatformInfo = {
6064
platform,
6165
appFilesUpdaterOptions,
@@ -125,7 +129,8 @@ class TestExecutionService implements ITestExecutionService {
125129
bundle: !!this.$options.bundle,
126130
release: this.$options.release,
127131
env: this.$options.env,
128-
timeout: this.$options.timeout
132+
timeout: this.$options.timeout,
133+
useHotModuleReload: this.$options.hmr
129134
};
130135

131136
await this.$liveSyncService.liveSync(deviceDescriptors, liveSyncInfo);
@@ -183,7 +188,11 @@ class TestExecutionService implements ITestExecutionService {
183188
this.$fs.writeFile(path.join(projectDir, TestExecutionService.CONFIG_FILE_NAME), configJs);
184189
}
185190

186-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: !!this.$options.bundle, release: this.$options.release };
191+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = {
192+
bundle: !!this.$options.bundle,
193+
release: this.$options.release,
194+
useHotModuleReload: this.$options.hmr
195+
};
187196
const preparePlatformInfo: IPreparePlatformInfo = {
188197
platform,
189198
appFilesUpdaterOptions,
@@ -253,7 +262,8 @@ class TestExecutionService implements ITestExecutionService {
253262
bundle: !!this.$options.bundle,
254263
release: this.$options.release,
255264
env: this.$options.env,
256-
timeout: this.$options.timeout
265+
timeout: this.$options.timeout,
266+
useHotModuleReload: this.$options.hmr
257267
};
258268

259269
await this.$liveSyncService.liveSync(deviceDescriptors, liveSyncInfo);

‎test/helpers/preview-command-helper.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ function createTestInjector() {
2626
generateQrCodeForiOS: async () => {
2727
qrCodesData.isGeneratedForiOS = true;
2828
},
29-
generateQrCodeForCurrentApp: async (useHmr: boolean) => {
29+
generateQrCodeForCurrentApp: async (options: IHasUseHotModuleReloadOption) => {
3030
qrCodesData.isGeneratedForCurrentApp = true;
3131
}
3232
});

‎test/ios-project-service.ts

+4-4
Original file line numberDiff line numberDiff line change
@@ -773,7 +773,7 @@ describe("iOS Project Service Signing", () => {
773773
describe("Check for Changes", () => {
774774
it("sets signingChanged if no Xcode project exists", async () => {
775775
const changes = <IProjectChangesInfo>{};
776-
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined }, projectData);
776+
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined, useHotModuleReload: false }, projectData);
777777
assert.isTrue(!!changes.signingChanged);
778778
});
779779
it("sets signingChanged if the Xcode projects is configured with Automatic signing, but proivsion is specified", async () => {
@@ -787,7 +787,7 @@ describe("iOS Project Service Signing", () => {
787787
};
788788
};
789789
const changes = <IProjectChangesInfo>{};
790-
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined }, projectData);
790+
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined, useHotModuleReload: false }, projectData);
791791
assert.isTrue(!!changes.signingChanged);
792792
});
793793
it("sets signingChanged if the Xcode projects is configured with Manual signing, but the proivsion specified differs the selected in the pbxproj", async () => {
@@ -806,7 +806,7 @@ describe("iOS Project Service Signing", () => {
806806
};
807807
};
808808
const changes = <IProjectChangesInfo>{};
809-
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined }, projectData);
809+
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined, useHotModuleReload: false }, projectData);
810810
assert.isTrue(!!changes.signingChanged);
811811
});
812812
it("does not set signingChanged if the Xcode projects is configured with Manual signing and proivsion matches", async () => {
@@ -825,7 +825,7 @@ describe("iOS Project Service Signing", () => {
825825
};
826826
};
827827
const changes = <IProjectChangesInfo>{};
828-
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined }, projectData);
828+
await iOSProjectService.checkForChanges(changes, { bundle: false, release: false, provision: "NativeScriptDev", teamId: undefined, useHotModuleReload: false }, projectData);
829829
assert.isFalse(!!changes.signingChanged);
830830
});
831831
});

‎test/npm-support.ts

+1-1
Original file line numberDiff line numberDiff line change
@@ -218,7 +218,7 @@ async function preparePlatform(testInjector: IInjector): Promise<void> {
218218

219219
await platformService.preparePlatform({
220220
platform: "android",
221-
appFilesUpdaterOptions: { bundle: !!options.bundle, release: options.release },
221+
appFilesUpdaterOptions: { bundle: !!options.bundle, release: options.release, useHotModuleReload: false },
222222
platformTemplate: "",
223223
projectData,
224224
config: options,

‎test/platform-service.ts

+2-2
Original file line numberDiff line numberDiff line change
@@ -466,7 +466,7 @@ describe('Platform Service Tests', () => {
466466
projectData.appResourcesDirectoryPath = path.join(testDirData.appFolderPath, "App_Resources");
467467

468468
platformService = testInjector.resolve("platformService");
469-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: false, release: release };
469+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: false, release: release, useHotModuleReload: false };
470470
await platformService.preparePlatform({
471471
platform: platformToTest,
472472
appFilesUpdaterOptions,
@@ -903,7 +903,7 @@ describe('Platform Service Tests', () => {
903903
let warnings: string = "";
904904
try {
905905
testInjector.resolve("$logger").warn = (text: string) => warnings += text;
906-
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: false, release: false };
906+
const appFilesUpdaterOptions: IAppFilesUpdaterOptions = { bundle: false, release: false, useHotModuleReload: false };
907907
await platformService.preparePlatform({
908908
platform: "android",
909909
appFilesUpdaterOptions,

‎test/project-changes-service.ts

+4-2
Original file line numberDiff line numberDiff line change
@@ -156,7 +156,8 @@ describe("Project Changes Service Tests", () => {
156156
bundle: false,
157157
release: false,
158158
provision: undefined,
159-
teamId: undefined
159+
teamId: undefined,
160+
useHotModuleReload: false
160161
}
161162
});
162163
assert.isTrue(!!iOSChanges.signingChanged, "iOS signingChanged expected to be true");
@@ -168,7 +169,8 @@ describe("Project Changes Service Tests", () => {
168169
bundle: false,
169170
release: false,
170171
provision: undefined,
171-
teamId: undefined
172+
teamId: undefined,
173+
useHotModuleReload: false
172174
}
173175
});
174176
assert.isFalse(!!androidChanges.signingChanged, "Android signingChanged expected to be false");

‎test/services/playground/preview-app-livesync-service.ts

+51-7
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
import { Yok } from "../../../lib/common/yok";
2+
import * as _ from 'lodash';
23
import { LoggerStub, ErrorsStub } from "../../stubs";
34
import { FilePayload, Device, FilesPayload } from "nativescript-preview-sdk";
45
import { PreviewAppLiveSyncService } from "../../../lib/services/livesync/playground/preview-app-livesync-service";
@@ -16,12 +17,14 @@ interface ITestCase {
1617

1718
interface IActOptions {
1819
callGetInitialFiles: boolean;
20+
hmr: boolean;
1921
}
2022

2123
interface IAssertOptions {
2224
checkWarnings?: boolean;
2325
isComparePluginsOnDeviceCalled?: boolean;
2426
checkInitialFiles?: boolean;
27+
hmr?: boolean;
2528
}
2629

2730
interface IActInput {
@@ -32,6 +35,7 @@ interface IActInput {
3235
}
3336

3437
let isComparePluginsOnDeviceCalled = false;
38+
let isHookCalledWithHMR = false;
3539
let applyChangesParams: FilePayload[] = [];
3640
let initialFiles: FilePayload[] = [];
3741
let readTextParams: string[] = [];
@@ -57,15 +61,16 @@ const syncFilesMockData = {
5761
projectDir: projectDirPath,
5862
appFilesUpdaterOptions: {
5963
release: false,
60-
bundle: false
64+
bundle: false,
65+
useHotModuleReload: false
6166
},
6267
env: {}
6368
};
6469

6570
class PreviewSdkServiceMock implements IPreviewSdkService {
6671
public getInitialFiles: (device: Device) => Promise<FilesPayload>;
6772

68-
public getQrCodeUrl(options: { useHmr: boolean }) {
73+
public getQrCodeUrl(options: IHasUseHotModuleReloadOption) {
6974
return "my_cool_qr_code_url";
7075
}
7176

@@ -146,7 +151,9 @@ function createTestInjector(options?: {
146151
}
147152
});
148153
injector.register("hooksService", {
149-
executeBeforeHooks: () => ({})
154+
executeBeforeHooks: (name: string, args: any) => {
155+
isHookCalledWithHMR = args.hookArgs.config.appFilesUpdaterOptions.useHotModuleReload;
156+
}
150157
});
151158

152159
return injector;
@@ -169,8 +176,9 @@ async function initialSync(input?: IActInput) {
169176
input = input || {};
170177

171178
const { previewAppLiveSyncService, previewSdkService, actOptions } = input;
172-
173-
await previewAppLiveSyncService.initialize(syncFilesMockData);
179+
const syncFilesData = _.cloneDeep(syncFilesMockData);
180+
syncFilesData.appFilesUpdaterOptions.useHotModuleReload = actOptions.hmr;
181+
await previewAppLiveSyncService.initialize(syncFilesData);
174182
if (actOptions.callGetInitialFiles) {
175183
await previewSdkService.getInitialFiles(deviceMockData);
176184
}
@@ -181,7 +189,9 @@ async function syncFiles(input?: IActInput) {
181189

182190
const { previewAppLiveSyncService, previewSdkService, projectFiles, actOptions } = input;
183191

184-
await previewAppLiveSyncService.initialize(syncFilesMockData);
192+
const syncFilesData = _.cloneDeep(syncFilesMockData);
193+
syncFilesData.appFilesUpdaterOptions.useHotModuleReload = actOptions.hmr;
194+
await previewAppLiveSyncService.initialize(syncFilesData);
185195
if (actOptions.callGetInitialFiles) {
186196
await previewSdkService.getInitialFiles(deviceMockData);
187197
}
@@ -193,6 +203,7 @@ async function assert(expectedFiles: string[], options?: IAssertOptions) {
193203
options = options || {};
194204
const actualFiles = options.checkInitialFiles ? initialFiles : applyChangesParams;
195205

206+
chai.assert.equal(isHookCalledWithHMR, options.hmr || false);
196207
chai.assert.deepEqual(actualFiles, mapFiles(expectedFiles));
197208

198209
if (options.checkWarnings) {
@@ -206,6 +217,7 @@ async function assert(expectedFiles: string[], options?: IAssertOptions) {
206217

207218
function reset() {
208219
isComparePluginsOnDeviceCalled = false;
220+
isHookCalledWithHMR = false;
209221
applyChangesParams = [];
210222
initialFiles = [];
211223
readTextParams = [];
@@ -230,7 +242,8 @@ function mapFiles(files: string[]): FilePayload[] {
230242
function setDefaults(testCase: ITestCase): ITestCase {
231243
if (!testCase.actOptions) {
232244
testCase.actOptions = {
233-
callGetInitialFiles: true
245+
callGetInitialFiles: true,
246+
hmr: false
234247
};
235248
}
236249

@@ -327,6 +340,33 @@ describe("previewAppLiveSyncService", () => {
327340
}
328341
];
329342

343+
const hmrTestCases: ITestCase[] = [
344+
{
345+
name: "when set to true",
346+
appFiles: [],
347+
expectedFiles: [],
348+
actOptions: {
349+
hmr: true,
350+
callGetInitialFiles: true
351+
},
352+
assertOptions: {
353+
hmr: true
354+
}
355+
},
356+
{
357+
name: "when set to false",
358+
appFiles: [],
359+
expectedFiles: [],
360+
actOptions: {
361+
hmr: false,
362+
callGetInitialFiles: true
363+
},
364+
assertOptions: {
365+
hmr: false
366+
}
367+
}
368+
];
369+
330370
const noAppFilesTestCases: ITestCase[] = [
331371
{
332372
name: "should transfer correctly default project files",
@@ -352,6 +392,10 @@ describe("previewAppLiveSyncService", () => {
352392
{
353393
name: "should handle correctly when no files are provided",
354394
testCases: noAppFilesTestCases
395+
},
396+
{
397+
name: "should pass the hmr option to the hook",
398+
testCases: hmrTestCases
355399
}
356400
];
357401

‎test/services/preview-sdk-service.ts

+38
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
import { PreviewSdkService } from "../../lib/services/livesync/playground/preview-sdk-service";
2+
import { Yok } from "../../lib/common/yok";
3+
import { assert } from "chai";
4+
import { LoggerStub, ErrorsStub } from "../stubs";
5+
6+
const getPreviewSdkService = (): IPreviewSdkService => {
7+
const testInjector = new Yok();
8+
testInjector.register("logger", LoggerStub);
9+
testInjector.register("errors", ErrorsStub);
10+
testInjector.register("config", {});
11+
testInjector.register("previewSdkService", PreviewSdkService);
12+
13+
testInjector.register("httpClient", {
14+
httpRequest: async (options: any, proxySettings?: IProxySettings): Promise<Server.IResponse> => undefined
15+
});
16+
17+
return testInjector.resolve("previewSdkService");
18+
};
19+
20+
describe('PreviewSdkService', () => {
21+
describe('getQrCodeUrl', () => {
22+
it('sets hmr to 1 when useHmr is true', async () => {
23+
const sdk = getPreviewSdkService();
24+
25+
const previewUrl = sdk.getQrCodeUrl({ useHmr: true });
26+
27+
assert.isTrue(previewUrl.indexOf("hmr=1") > -1);
28+
});
29+
});
30+
31+
it('sets hmr to 0 when useHmr is false', async () => {
32+
const sdk = getPreviewSdkService();
33+
34+
const previewUrl = sdk.getQrCodeUrl({ useHmr: false });
35+
36+
assert.isTrue(previewUrl.indexOf("hmr=0") > -1);
37+
});
38+
});

0 commit comments

Comments
 (0)
Please sign in to comment.