Big refactoring #77

Merged
flecomte merged 166 commits from refactoring-component-and-immutable into master 2021-03-24 19:06:07 +01:00
6 changed files with 10 additions and 11 deletions
Showing only changes of commit 9fc21f5459 - Show all commits

View File

@@ -36,7 +36,7 @@ plugins {
id("com.github.johnrengelman.shadow") version "5.2.0" id("com.github.johnrengelman.shadow") version "5.2.0"
id("org.jlleitschuh.gradle.ktlint") version "9.4.1" id("org.jlleitschuh.gradle.ktlint") version "9.4.1"
id("org.owasp.dependencycheck") version "6.0.5" id("org.owasp.dependencycheck") version "6.1.1"
id("org.sonarqube") version "3.1.1" id("org.sonarqube") version "3.1.1"
id("net.nemerosa.versioning") version "2.14.0" id("net.nemerosa.versioning") version "2.14.0"
id("io.gitlab.arturbosch.detekt") version "1.16.0-RC1" id("io.gitlab.arturbosch.detekt") version "1.16.0-RC1"

View File

@@ -53,7 +53,6 @@ import io.ktor.features.ContentNegotiation
import io.ktor.features.DataConversion import io.ktor.features.DataConversion
import io.ktor.features.NotFoundException import io.ktor.features.NotFoundException
import io.ktor.features.StatusPages import io.ktor.features.StatusPages
import io.ktor.features.maxAge
import io.ktor.http.HttpHeaders import io.ktor.http.HttpHeaders
import io.ktor.http.HttpMethod import io.ktor.http.HttpMethod
import io.ktor.http.HttpStatusCode import io.ktor.http.HttpStatusCode
@@ -205,7 +204,7 @@ fun Application.module(env: Env = PROD) {
} }
allowCredentials = true allowCredentials = true
allowSameOrigin = true allowSameOrigin = true
maxAge = Duration.ofDays(1) maxAgeInSeconds = Duration.ofDays(1).seconds
} }
if (env == PROD) { if (env == PROD) {

View File

@@ -2,7 +2,7 @@ package fr.dcproject.application
import com.fasterxml.jackson.databind.DeserializationFeature import com.fasterxml.jackson.databind.DeserializationFeature
import com.fasterxml.jackson.databind.ObjectMapper import com.fasterxml.jackson.databind.ObjectMapper
import com.fasterxml.jackson.databind.PropertyNamingStrategy import com.fasterxml.jackson.databind.PropertyNamingStrategies
import com.fasterxml.jackson.databind.SerializationFeature import com.fasterxml.jackson.databind.SerializationFeature
import com.fasterxml.jackson.databind.module.SimpleModule import com.fasterxml.jackson.databind.module.SimpleModule
import com.fasterxml.jackson.datatype.joda.JodaModule import com.fasterxml.jackson.datatype.joda.JodaModule
@@ -68,7 +68,7 @@ val KoinModule = module {
single<ObjectMapper> { single<ObjectMapper> {
jacksonObjectMapper().apply { jacksonObjectMapper().apply {
registerModule(SimpleModule()) registerModule(SimpleModule())
propertyNamingStrategy = PropertyNamingStrategy.SNAKE_CASE propertyNamingStrategy = PropertyNamingStrategies.LOWER_CAMEL_CASE
registerModule(JodaModule()) registerModule(JodaModule())
disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS) disable(SerializationFeature.WRITE_DATES_AS_TIMESTAMPS)

View File

@@ -94,7 +94,7 @@ class NotificationConsumer(
private suspend fun followersFromMessage(body: ByteArray, action: suspend (DecodedMessage) -> Unit) { private suspend fun followersFromMessage(body: ByteArray, action: suspend (DecodedMessage) -> Unit) {
val rawMessage: String = body.toString(Charsets.UTF_8) val rawMessage: String = body.toString(Charsets.UTF_8)
val notification: EntityNotification = Notification.fromString<EntityNotification>(rawMessage) ?: error("Unable to deserialize notification message from rabbit") val notification: EntityNotification = Notification.fromString(rawMessage)
val follows = when (notification.type) { val follows = when (notification.type) {
"article" -> followArticleRepo.findFollowsByTarget(notification.target) "article" -> followArticleRepo.findFollowsByTarget(notification.target)
"constitution" -> followConstitutionRepo.findFollowsByTarget(notification.target) "constitution" -> followConstitutionRepo.findFollowsByTarget(notification.target)

View File

@@ -114,9 +114,9 @@ class NotificationConsumerTest {
).await() ).await()
/* Check if notifications sent */ /* Check if notifications sent */
verify(timeout = 1000) { followArticleRepo.findFollowsByTarget(any()) } verify(timeout = 2000) { followArticleRepo.findFollowsByTarget(any()) }
verify(timeout = 1000) { emailSender.sendEmail(any()) } verify(timeout = 2000) { emailSender.sendEmail(any()) }
verify(timeout = 1000) { asyncCommand.zadd(any<String>(), any<Double>(), any<String>()) } verify(timeout = 2000) { asyncCommand.zadd(any<String>(), any<Double>(), any<String>()) }
consumer.close() consumer.close()
} }

View File

@@ -78,10 +78,10 @@ fun createCommentOnTarget(
content: String? = null content: String? = null
) { ) {
val commentRepository: CommentRepository by lazy { GlobalContext.get().koin.get() } val commentRepository: CommentRepository by lazy { GlobalContext.get().koin.get() }
val createdBy = createCitizen(createdBy) val creator = createCitizen(createdBy)
val comment = CommentForUpdate( val comment = CommentForUpdate(
id = id ?: UUID.randomUUID(), id = id ?: UUID.randomUUID(),
createdBy = createdBy, createdBy = creator,
target = target, target = target,
content = content ?: LoremIpsum().getParagraphs(1, 3) content = content ?: LoremIpsum().getParagraphs(1, 3)
) )