Skip to content

Commit 8c68295

Browse files
committed
Merge remote-tracking branch 'origin/main' into feat/hooks-esm-support
2 parents fa07e37 + 431c133 commit 8c68295

File tree

7 files changed

+620
-454
lines changed

7 files changed

+620
-454
lines changed

Gruntfile.js

Lines changed: 32 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -248,15 +248,40 @@ function registerTestingDependenciesTasks(grunt) {
248248
const generatedVersionFilePath = path.join(configsBasePath, "test-deps-versions-generated.json");
249249

250250
grunt.registerTask("generate_unit_testing_dependencies", async function () {
251-
var done = this.async();
251+
const done = this.async();
252+
252253
const dependenciesVersions = {};
253-
const testDependencies = grunt.file.readJSON(path.join(configsBasePath, "test-dependencies.json"));
254-
for (var dependency of testDependencies) {
255-
const dependencyVersion = dependency.version || await latestVersion(dependency.name);
256-
dependenciesVersions[dependency.name] = dependencyVersion;
254+
let testDependencies;
255+
256+
try {
257+
testDependencies = grunt.file.readJSON(path.join(configsBasePath, "test-dependencies.json"));
258+
} catch (err) {
259+
grunt.log.error("Could not read test-dependencies.json:", err);
260+
return done(false);
257261
}
258-
grunt.file.write(generatedVersionFilePath, JSON.stringify(dependenciesVersions));
259-
done();
262+
263+
264+
// Kick off all version resolutions in parallel
265+
const versionPromises = testDependencies.map(dep => {
266+
if (dep.version) {
267+
dependenciesVersions[dep.name] = dep.version;
268+
return Promise.resolve();
269+
}
270+
return latestVersion(dep.name).then(v => {
271+
dependenciesVersions[dep.name] = v;
272+
});
273+
});
274+
275+
Promise.all(versionPromises)
276+
.then(() => {
277+
grunt.file.write(generatedVersionFilePath, JSON.stringify(dependenciesVersions, null, 2));
278+
grunt.log.writeln("Wrote", generatedVersionFilePath);
279+
done();
280+
})
281+
.catch(err => {
282+
grunt.log.error(err);
283+
done(false);
284+
});
260285
});
261286

262287
grunt.registerTask("verify_unit_testing_dependencies", function () {

lib/common/mobile/android/android-emulator-services.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -235,7 +235,7 @@ export class AndroidEmulatorServices
235235
const minVersion = semver.coerce(AndroidVirtualDevice.MIN_ANDROID_VERSION);
236236
const bestVersion = best && best.version && semver.coerce(best.version);
237237

238-
return bestVersion && semver.gte(bestVersion, minVersion) ? best : null;
238+
return !bestVersion || semver.gte(bestVersion, minVersion) ? best : null;
239239
}
240240

241241
private async waitForEmulatorBootToComplete(

lib/common/mobile/emulator-helper.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import { injector } from "../yok";
55
export class EmulatorHelper implements Mobile.IEmulatorHelper {
66
// https://developer.android.com/guide/topics/manifest/uses-sdk-element
77
public mapAndroidApiLevelToVersion = {
8+
"android-36": "16.0.0",
89
"android-35": "15.0.0",
910
"android-34": "14.0.0",
1011
"android-33": "13.0.0",

0 commit comments

Comments
 (0)