Merge pull request #310 from amihaiemil/308

#308 removed Projects.getByWebHookToken
self-xdsd · Aug 28, 2021 · 2d0474b · 2d0474b
2 parents 04e58e5 + 73d5200
commit 2d0474b
Show file tree

Hide file tree

Showing 3 changed files with 1 addition and 56 deletions.
diff --git a/pom.xml b/pom.xml
@@ -10,7 +10,7 @@
     <name>Self XDSD Storage Module</name>
     <description>Storage Module for Self XDSD</description>
     <properties>
-        <self.core.version>0.0.90</self.core.version>
+        <self.core.version>0.0.92</self.core.version>
         <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
     </properties>
     <build>

diff --git a/src/main/java/com/selfxdsd/storage/SelfProjects.java b/src/main/java/com/selfxdsd/storage/SelfProjects.java
@@ -230,26 +230,6 @@ public Project getProjectById(
         return project;
     }
 
-    @Override
-    public Project getByWebHookToken(final String webHookToken) {
-        return this.database.jooq()
-            .select()
-            .from(SLF_PROJECTS_XDSD)
-            .join(SLF_USERS_XDSD)
-            .on(
-                SLF_PROJECTS_XDSD.USERNAME.eq(SLF_USERS_XDSD.USERNAME).and(
-                    SLF_PROJECTS_XDSD.PROVIDER.eq(SLF_USERS_XDSD.PROVIDER)
-                )
-            )
-            .join(SLF_PMS_XDSD)
-            .on(SLF_PROJECTS_XDSD.PMID.eq(SLF_PMS_XDSD.ID))
-            .where(SLF_PROJECTS_XDSD.WEBHOOK_TOKEN.eq(webHookToken))
-            .stream()
-            .map(rec -> projectFromRecord(rec, false))
-            .findFirst()
-            .orElse(null);
-    }
-
     @Override
     public Projects page(final Page page) {
         return new SelfProjects(

diff --git a/src/test/java/com/selfxdsd/storage/SelfProjectsITCase.java b/src/test/java/com/selfxdsd/storage/SelfProjectsITCase.java
@@ -84,41 +84,6 @@ public void returnsNullIfProjectMissing() {
         );
     }
 
-    /**
-     * Returns a project by its WebHook Token.
-     */
-    @Test
-    public void getsProjectByWebHookToken() {
-        final Projects projects = new SelfJooq(new H2Database()).projects();
-        final Project found = projects.getByWebHookToken("whtoken123");
-        MatcherAssert.assertThat(
-            found.repoFullName(),
-            Matchers.equalTo("amihaiemil/docker-java-api")
-        );
-        MatcherAssert.assertThat(
-            found.owner().username(),
-            Matchers.equalTo("amihaiemil")
-        );
-        MatcherAssert.assertThat(
-            found.projectManager().id(),
-            Matchers.equalTo(1)
-        );
-        MatcherAssert.assertThat(
-            found.projectManager().provider().name(),
-            Matchers.equalTo(Provider.Names.GITHUB)
-        );
-    }
-
-    /**
-     * Returns null the project is not found by webhook token.
-     */
-    @Test
-    public void returnsNullIfProjectNotFountByWebHookToken() {
-        final Projects projects = new SelfJooq(new H2Database()).projects();
-        final Project found = projects.getByWebHookToken("missing-token-123");
-        MatcherAssert.assertThat(found, Matchers.nullValue());
-    }
-
     /**
      * SelfProjects can return the Projects owned by a certain User.
      */