spinner to comply with mvp.cs

querwurzel · Nov 13, 2023 · b740fae · b740fae
1 parent 08aa279
commit b740fae
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 12 deletions.
diff --git a/backend/src/main/java/com/github/binpastes/paste/domain/Paste.java b/backend/src/main/java/com/github/binpastes/paste/domain/Paste.java
@@ -131,7 +131,7 @@ public boolean isErasable(String remoteAddress) {
             final var createdBySameAuthor = Objects.equals(remoteAddress, this.getRemoteAddress());
 
             if (createdBySameAuthor) {
-                return this.getDateCreated().isAfter(LocalDateTime.now().minusHours(1));
+                return LocalDateTime.now().minusHours(1).isBefore(this.getDateCreated());
             }
         }
 

diff --git a/backend/src/test/java/com/github/binpastes/paste/api/PasteControllerTest.java b/backend/src/test/java/com/github/binpastes/paste/api/PasteControllerTest.java
@@ -66,7 +66,7 @@ void searchPastes() {
         doReturn(Flux.empty()).when(pasteService).findByFullText(anyString());
 
         webClient.get()
-                .uri("/api/v1/paste/search?term=/{term}", "foobar")
+                .uri("/api/v1/paste/search?term={term}", "foobar")
                 .exchange()
                 .expectStatus().isOk()
                 .expectHeader().cacheControl(CacheControl.maxAge(60, TimeUnit.SECONDS))

diff --git a/backend/src/test/java/com/github/binpastes/paste/domain/PasteTest.java b/backend/src/test/java/com/github/binpastes/paste/domain/PasteTest.java
@@ -40,13 +40,6 @@ void newInstanceDefaults() {
         assertThat(newPaste.getId()).isNotEmpty();
         assertThat(newPaste.getViews()).isZero();
         assertThat(newPaste.getLastViewed()).isNull();
-
-        assertThat(newPaste.getTitle()).isNull();
-        assertThat(newPaste.getContent()).isEqualTo("someContent");
-        assertThat(newPaste.isPermanent()).isTrue();
-        assertThat(newPaste.isEncrypted()).isFalse();
-        assertThat(newPaste.isPublic()).isTrue();
-        assertThat(newPaste.getRemoteAddress()).isNull();
     }
 
     @Test
@@ -65,7 +58,7 @@ void trackPasteViewCount() {
     void trackPasteLastViewed() {
         var newPaste = Paste.newInstance(null, "someContent", null, false, PasteExposure.PUBLIC, null);
         var now = LocalDateTime.now();
-        var yesterday = LocalDateTime.now().minusDays(1);
+        var yesterday = now.minusDays(1);
 
         newPaste.trackView(now);
         newPaste.trackView(yesterday);
@@ -87,7 +80,7 @@ private static Stream<Arguments> pastesToErase() {
         var oneTimePaste = Paste.newInstance(null, "someContent", null, false, PasteExposure.ONCE, "Alice");
 
         var publicPasteRecentlyCreatedByAlice = Paste.newInstance(null, "someContent", null, false, PasteExposure.PUBLIC, "Alice");
-        publicPasteRecentlyCreatedByAlice.setDateCreated(LocalDateTime.now().minusMinutes(59));
+        publicPasteRecentlyCreatedByAlice.setDateCreated(LocalDateTime.now().minusMinutes(60).plusSeconds(1));
 
         var publicPasteCreatedSomeTimeAgoByAlice = Paste.newInstance(null, "someContent", null, false, PasteExposure.PUBLIC, "Alice");
         publicPasteCreatedSomeTimeAgoByAlice.setDateCreated(LocalDateTime.now().minusMinutes(60));

diff --git a/frontend/src/components/Spinner/spinner.css b/frontend/src/components/Spinner/spinner.css
@@ -5,7 +5,7 @@
   height: 80px;
 }
 .lds-ellipsis div {
-  background: #000;
+  background: var(--color-text);
   position: absolute;
   top: 33px;
   width: 13px;