Skip to content

Commit

Permalink
fix(gradle): support gradlePluginPortal() (#11894)
Browse files Browse the repository at this point in the history
  • Loading branch information
JavierSegoviaCordoba committed Oct 3, 2021
1 parent 9e2dd3b commit 71679f4
Show file tree
Hide file tree
Showing 5 changed files with 38 additions and 14 deletions.
2 changes: 2 additions & 0 deletions lib/manager/gradle/deep/__fixtures__/build.gradle.example1
Expand Up @@ -8,13 +8,15 @@ buildscript {
}
mavenCentral()
jcenter()
gradlePluginPortal()
}
dependencies {
classpath("org.springframework.boot:spring-boot-gradle-plugin:${springBootVersion}")
classpath 'com.github.jengelman.gradle.plugins:shadow:1.2.3'
classpath 'com.fkorotkov:gradle-libraries-plugin:0.1'
classpath "gradle.plugin.se.patrikerdes:gradle-use-latest-versions-plugin:0.2.3"
classpath 'org.apache.openjpa:openjpa:3.1.1'
classpath 'com.gradle.publish:plugin-publish-plugin:0.13.0'
}
}

Expand Down
28 changes: 18 additions & 10 deletions lib/manager/gradle/shallow/__snapshots__/parser.spec.ts.snap
Expand Up @@ -15,79 +15,87 @@ Array [
"currentValue": "1.2.3",
"depName": "com.github.jengelman.gradle.plugins:shadow",
"managerData": Object {
"fileReplacePosition": 388,
"fileReplacePosition": 417,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "0.1",
"depName": "com.fkorotkov:gradle-libraries-plugin",
"managerData": Object {
"fileReplacePosition": 452,
"fileReplacePosition": 481,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "0.2.3",
"depName": "gradle.plugin.se.patrikerdes:gradle-use-latest-versions-plugin",
"managerData": Object {
"fileReplacePosition": 539,
"fileReplacePosition": 568,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "3.1.1",
"depName": "org.apache.openjpa:openjpa",
"managerData": Object {
"fileReplacePosition": 592,
"fileReplacePosition": 621,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "0.13.0",
"depName": "com.gradle.publish:plugin-publish-plugin",
"managerData": Object {
"fileReplacePosition": 688,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "6.0.9.RELEASE",
"depName": "org.grails:gorm-hibernate5-spring-boot",
"managerData": Object {
"fileReplacePosition": 1785,
"fileReplacePosition": 1882,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "6.0.5",
"depName": "mysql:mysql-connector-java",
"managerData": Object {
"fileReplacePosition": 1841,
"fileReplacePosition": 1938,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "1.0-groovy-2.4",
"depName": "org.spockframework:spock-spring",
"managerData": Object {
"fileReplacePosition": 1899,
"fileReplacePosition": 1996,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "1.3",
"depName": "org.hamcrest:hamcrest-core",
"managerData": Object {
"fileReplacePosition": 2004,
"fileReplacePosition": 2101,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "3.1",
"depName": "cglib:cglib-nodep",
"managerData": Object {
"fileReplacePosition": 2092,
"fileReplacePosition": 2189,
"packageFile": "build.gradle",
},
},
Object {
"currentValue": "3.1.1",
"depName": "org.apache.openjpa:openjpa",
"managerData": Object {
"fileReplacePosition": 2198,
"fileReplacePosition": 2295,
"packageFile": "build.gradle",
},
},
Expand Down
1 change: 1 addition & 0 deletions lib/manager/gradle/shallow/common.ts
Expand Up @@ -2,6 +2,7 @@ export { MAVEN_REPO } from '../../../datasource/maven/common';

export const JCENTER_REPO = 'https://jcenter.bintray.com/';
export const GOOGLE_REPO = 'https://dl.google.com/android/maven2/';
export const GRADLE_PLUGIN_PORTAL_REPO = 'https://plugins.gradle.org/m2/';

// TODO: convert to types
export enum TokenType {
Expand Down
10 changes: 8 additions & 2 deletions lib/manager/gradle/shallow/parser.spec.ts
@@ -1,5 +1,10 @@
import { loadFixture } from '../../../../test/util';
import { GOOGLE_REPO, JCENTER_REPO, MAVEN_REPO } from './common';
import {
GOOGLE_REPO,
GRADLE_PLUGIN_PORTAL_REPO,
JCENTER_REPO,
MAVEN_REPO,
} from './common';
import { parseGradle, parseProps } from './parser';

describe('manager/gradle/shallow/parser', () => {
Expand Down Expand Up @@ -60,13 +65,14 @@ describe('manager/gradle/shallow/parser', () => {
expect(urls).toStrictEqual(['https://example.com']);

({ urls } = parseGradle(
'mavenCentral(); uri("https://example.com"); jcenter(); google();'
'mavenCentral(); uri("https://example.com"); jcenter(); google(); gradlePluginPortal();'
));
expect(urls).toStrictEqual([
MAVEN_REPO,
'https://example.com',
JCENTER_REPO,
GOOGLE_REPO,
GRADLE_PLUGIN_PORTAL_REPO,
]);

({ urls } = parseGradle(
Expand Down
11 changes: 9 additions & 2 deletions lib/manager/gradle/shallow/parser.ts
Expand Up @@ -4,7 +4,13 @@ import { logger } from '../../../logger';
import { regEx } from '../../../util/regex';
import type { PackageDependency } from '../../types';
import type { GradleManagerData } from '../types';
import { GOOGLE_REPO, JCENTER_REPO, MAVEN_REPO, TokenType } from './common';
import {
GOOGLE_REPO,
GRADLE_PLUGIN_PORTAL_REPO,
JCENTER_REPO,
MAVEN_REPO,
TokenType,
} from './common';
import { tokenize } from './tokenizer';
import type {
MatchConfig,
Expand Down Expand Up @@ -215,6 +221,7 @@ function processPredefinedRegistryUrl({
mavenCentral: MAVEN_REPO,
jcenter: JCENTER_REPO,
google: GOOGLE_REPO,
gradlePluginPortal: GRADLE_PLUGIN_PORTAL_REPO,
}[registryName];
return { urls: [registryUrl] };
}
Expand Down Expand Up @@ -330,7 +337,7 @@ const matcherConfigs: SyntaxMatchConfig[] = [
matchers: [
{
matchType: TokenType.Word,
matchValue: ['mavenCentral', 'jcenter', 'google'],
matchValue: ['mavenCentral', 'jcenter', 'google', 'gradlePluginPortal'],
tokenMapKey: 'registryName',
},
{ matchType: TokenType.LeftParen },
Expand Down

0 comments on commit 71679f4

Please sign in to comment.