From bf136e47f20ea0f81c79d6b88b5211ae569a60d7 Mon Sep 17 00:00:00 2001 From: Natan Deitch Date: Thu, 20 Feb 2020 05:39:32 -0300 Subject: [PATCH] #90: Use correct field to retrieve Github repository name The Github ALM Binding Web Service uses the `AlmRepo` field to store the repository name, but the Github decorator was using `AlmSlug` to try and retrieve the repository name, so was getting a `null` value back and failing to find a matching repository. Switching to using `AlmRepo` in the decorator overcomes this issues. --- .../ce/pullrequest/github/v4/GraphqlCheckRunProvider.java | 2 +- .../pullrequest/github/v4/GraphqlCheckRunProviderTest.java | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProvider.java b/src/main/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProvider.java index 88038fe19..2582af93b 100644 --- a/src/main/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProvider.java +++ b/src/main/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProvider.java @@ -95,7 +95,7 @@ public void createCheckRun(AnalysisDetails analysisDetails, AlmSettingDto almSet ProjectAlmSettingDto projectAlmSettingDto) throws IOException, GeneralSecurityException { String apiUrl = almSettingDto.getUrl(); String apiPrivateKey = almSettingDto.getPrivateKey(); - String projectPath = projectAlmSettingDto.getAlmSlug(); + String projectPath = projectAlmSettingDto.getAlmRepo(); String appId = almSettingDto.getAppId(); RepositoryAuthenticationToken repositoryAuthenticationToken = diff --git a/src/test/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProviderTest.java b/src/test/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProviderTest.java index d9472eaf5..0c6f13970 100644 --- a/src/test/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProviderTest.java +++ b/src/test/java/com/github/mc1arke/sonarqube/plugin/ce/pullrequest/github/v4/GraphqlCheckRunProviderTest.java @@ -112,7 +112,7 @@ public void createCheckRunExceptionOnErrorResponse() throws IOException, General when(graphqlProvider.createGraphQLTemplate()).thenReturn(graphQLTemplate); ProjectAlmSettingDto projectAlmSettingDto = mock(ProjectAlmSettingDto.class); - when(projectAlmSettingDto.getAlmSlug()).thenReturn("dummy/repo"); + when(projectAlmSettingDto.getAlmRepo()).thenReturn("dummy/repo"); AlmSettingDto almSettingDto = mock(AlmSettingDto.class); when(almSettingDto.getUrl()).thenReturn("http://host.name"); when(almSettingDto.getAppId()).thenReturn("app id"); @@ -323,7 +323,7 @@ private void createCheckRunHappyPath(QualityGate.Status status) throws IOExcepti when(graphqlProvider.createGraphQLTemplate()).thenReturn(graphQLTemplate); ProjectAlmSettingDto projectAlmSettingDto = mock(ProjectAlmSettingDto.class); - when(projectAlmSettingDto.getAlmSlug()).thenReturn("dummy/repo"); + when(projectAlmSettingDto.getAlmRepo()).thenReturn("dummy/repo"); AlmSettingDto almSettingDto = mock(AlmSettingDto.class); when(almSettingDto.getUrl()).thenReturn("http://host.name"); when(almSettingDto.getAppId()).thenReturn("app id"); @@ -420,4 +420,4 @@ public void checkCorrectDefaultValuesInjected() { .isEqualTo(new GraphqlCheckRunProvider(new DefaultGraphqlProvider(), clock, githubApplicationAuthenticationProvider, server)); } -} \ No newline at end of file +}