Skip to content

Commit 2c93851

Browse files
committed
refactor/qg-290: ещё пачка мелкой полировки реализации пушей
1 parent 4374e32 commit 2c93851

File tree

8 files changed

+32
-25
lines changed

8 files changed

+32
-25
lines changed

app/src/main/kotlin/pro/qyoga/i9ns/pushes/web/model/TherapistWebPushSubscription.kt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,9 @@
11
package pro.qyoga.i9ns.pushes.web.model
22

3+
import org.simpleframework.xml.Version
4+
import org.springframework.data.annotation.CreatedDate
35
import org.springframework.data.annotation.Id
6+
import org.springframework.data.annotation.LastModifiedDate
47
import org.springframework.data.annotation.Transient
58
import org.springframework.data.relational.core.mapping.Table
69
import pro.azhidkov.platform.uuid.UUIDv7
@@ -15,9 +18,9 @@ data class TherapistWebPushSubscription(
1518
val subscription: WebPushSubscription,
1619

1720
@Id val id: UUID = UUIDv7.randomUUID(),
18-
val createdAt: Instant = Instant.now(),
19-
val updatedAt: Instant? = null,
20-
val version: Long = 0
21+
@CreatedDate val createdAt: Instant = Instant.now(),
22+
@LastModifiedDate val updatedAt: Instant? = null,
23+
@Version val version: Long = 0
2124
) {
2225

2326
@Transient

app/src/test/kotlin/pro/qyoga/tests/cases/app/i9ns/pushes/web/DeleteSubscriptionApiTest.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ import io.kotest.matchers.shouldBe
55
import org.junit.jupiter.api.DisplayName
66
import org.junit.jupiter.api.Test
77
import pro.qyoga.tests.clients.api.TrainerAdvisorApis
8+
import pro.qyoga.tests.clients.api.WebPushes
89
import pro.qyoga.tests.fixture.object_mothers.pushes.web.WebPushesObjectMother.aWebPushSubscription
910
import pro.qyoga.tests.fixture.object_mothers.therapists.THE_THERAPIST_REF
1011
import pro.qyoga.tests.fixture.test_apis.WebPushesTestApi

app/src/test/kotlin/pro/qyoga/tests/cases/app/i9ns/pushes/web/GetWebPushesPublicKeyApiTest.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import io.kotest.matchers.shouldBe
44
import org.junit.jupiter.api.DisplayName
55
import org.junit.jupiter.api.Test
66
import pro.qyoga.tests.clients.api.TrainerAdvisorApis
7+
import pro.qyoga.tests.clients.api.WebPushes
78
import pro.qyoga.tests.infra.test_config.spring.context
89
import pro.qyoga.tests.infra.web.QYogaAppBaseTest
910

app/src/test/kotlin/pro/qyoga/tests/cases/app/i9ns/pushes/web/RegisterSubscriptionApiTest.kt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import io.kotest.matchers.shouldNotBe
77
import org.junit.jupiter.api.DisplayName
88
import org.junit.jupiter.api.Test
99
import pro.qyoga.tests.clients.api.TrainerAdvisorApis
10+
import pro.qyoga.tests.clients.api.WebPushes
1011
import pro.qyoga.tests.fixture.object_mothers.pushes.web.WebPushesObjectMother.aWebPushSubscription
1112
import pro.qyoga.tests.fixture.object_mothers.therapists.THE_THERAPIST_REF
1213
import pro.qyoga.tests.fixture.test_apis.NotificationsTestApi

app/src/test/kotlin/pro/qyoga/tests/clients/api/NotificationsTherapistApi.kt renamed to app/src/test/kotlin/pro/qyoga/tests/clients/api/NotificationsApi.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -8,17 +8,17 @@ import pro.qyoga.app.therapist.appointments.core.schedule.settings.Notifications
88
import pro.qyoga.tests.infra.web.mainWebTestClient
99
import pro.qyoga.tests.platform.spring.web_test_client.getBodyAsString
1010

11-
object Notifications {
11+
object NotificationsApiFactory {
1212

1313
fun therapistApi(
1414
principal: Cookie,
1515
) = NotificationsTherapistApi(principal, mainWebTestClient)
1616

1717
}
1818

19-
@Suppress("UnusedReceiverParameter", "RemoveRedundantQualifierName")
19+
@Suppress("UnusedReceiverParameter")
2020
val TrainerAdvisorApis.Notifications
21-
get() = pro.qyoga.tests.clients.api.Notifications
21+
get() = NotificationsApiFactory
2222

2323
class NotificationsTherapistApi(
2424
override val authCookie: Cookie,
Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,4 @@
11
package pro.qyoga.tests.clients.api
22

3-
import io.restassured.http.Cookie
4-
import pro.qyoga.tests.infra.web.mainWebTestClient
53

6-
7-
object TrainerAdvisorApis {
8-
9-
object WebPushes {
10-
11-
val publicApi = WebPushesPublicApi(mainWebTestClient)
12-
13-
fun therapistApi(
14-
principal: Cookie,
15-
) = WebPushesTherapistApi(principal, mainWebTestClient)
16-
17-
}
18-
19-
}
4+
object TrainerAdvisorApis

app/src/test/kotlin/pro/qyoga/tests/clients/api/WebPushesPublicApi.kt

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,26 @@
11
package pro.qyoga.tests.clients.api
22

3+
import io.restassured.http.Cookie
34
import org.springframework.test.web.reactive.server.WebTestClient
45
import org.springframework.test.web.reactive.server.returnResult
56
import pro.qyoga.app.publc.pushes.web.PushesPublicKeyController
7+
import pro.qyoga.tests.infra.web.mainWebTestClient
68

79

10+
object WebPushesApiFactory {
11+
12+
val publicApi = WebPushesPublicApi(mainWebTestClient)
13+
14+
fun therapistApi(
15+
principal: Cookie,
16+
) = WebPushesTherapistApi(principal, mainWebTestClient)
17+
18+
}
19+
20+
@Suppress("UnusedReceiverParameter")
21+
val TrainerAdvisorApis.WebPushes
22+
get() = WebPushesApiFactory
23+
824
class WebPushesPublicApi(
925
private val client: WebTestClient
1026
) {

app/src/testFixtures/kotlin/pro/qyoga/tests/pages/therapist/clients/journal/list/ClientJournalFragment.kt

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,10 +24,10 @@ object ClientJournalFragment {
2424

2525
fun fragmentFor(entries: List<JournalEntry>, hasMore: Boolean = false): ComponentMatcher =
2626
ComponentMatcher("#journal-fragment") { element ->
27-
element shouldHaveComponent addEntryLink
27+
element shouldHaveComponent addEntryLink
2828

29-
ClientJournalEntriesFragment.fragmentFor(entries, hasMore).match(element)
30-
}
29+
ClientJournalEntriesFragment.fragmentFor(entries, hasMore).match(element)
30+
}
3131

3232
}
3333

0 commit comments

Comments
 (0)