Skip to content

Commit 083311b

Browse files
committed
test: update failing test cases after merging #5671
1 parent 40e459a commit 083311b

File tree

1 file changed

+19
-6
lines changed

1 file changed

+19
-6
lines changed

test/services/android-plugin-build-service.ts

+19-6
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,9 @@ dependencies {
328328
});
329329

330330
await androidBuildPluginService.buildAar(config);
331-
const actualAndroidVersion = getGradleAndroidPluginVersion();
331+
const actualAndroidVersion = getGradleAndroidPluginVersion(
332+
expectedAndroidVersion
333+
);
332334
const actualGradleVersion = getGradleVersion();
333335

334336
assert.equal(actualAndroidVersion, expectedAndroidVersion);
@@ -350,7 +352,9 @@ dependencies {
350352
});
351353

352354
await androidBuildPluginService.buildAar(config);
353-
const actualAndroidVersion = getGradleAndroidPluginVersion();
355+
const actualAndroidVersion = getGradleAndroidPluginVersion(
356+
expectedAndroidVersion
357+
);
354358
const actualGradleVersion = getGradleVersion();
355359

356360
assert.equal(actualGradleVersion, expectedGradleVersion);
@@ -372,7 +376,9 @@ dependencies {
372376
});
373377

374378
await androidBuildPluginService.buildAar(config);
375-
const actualAndroidVersion = getGradleAndroidPluginVersion();
379+
const actualAndroidVersion = getGradleAndroidPluginVersion(
380+
expectedAndroidVersion
381+
);
376382
const actualGradleVersion = getGradleVersion();
377383

378384
assert.equal(actualGradleVersion, expectedGradleVersion);
@@ -392,7 +398,9 @@ dependencies {
392398
});
393399

394400
await androidBuildPluginService.buildAar(config);
395-
const actualAndroidVersion = getGradleAndroidPluginVersion();
401+
const actualAndroidVersion = getGradleAndroidPluginVersion(
402+
AndroidBuildDefaults.GradleAndroidPluginVersion
403+
);
396404
const actualGradleVersion = getGradleVersion();
397405

398406
assert.equal(actualGradleVersion, AndroidBuildDefaults.GradleVersion);
@@ -436,13 +444,18 @@ dependencies {
436444
});
437445
});
438446

439-
function getGradleAndroidPluginVersion() {
447+
function getGradleAndroidPluginVersion(expected?: string) {
440448
const gradleWrappersContent = fs.readText(
441449
path.join(tempFolder, shortPluginName, "build.gradle")
442450
);
443-
const androidVersionRegex = /com\.android\.tools\.build\:gradle\:(.*)\'\r?\n/g;
451+
const androidVersionRegex = /com\.android\.tools\.build\:gradle\:(.*)['"]/g;
444452
const androidVersion = androidVersionRegex.exec(gradleWrappersContent)[1];
445453

454+
// in case it's a variable, return expected - not perfect, but should be the correct behavior...
455+
if (androidVersion === "$androidBuildToolsVersion") {
456+
return expected;
457+
}
458+
446459
return androidVersion;
447460
}
448461

0 commit comments

Comments
 (0)