Calculate contest rating for projects and organizations (#1181)

### What's done:
* Calculate contest rating for projects and organizations

db/v-2/tables/project.xml

@@ -64,4 +64,8 @@
         </addColumn>
     </changeSet>
 
+    <changeSet id="project-make-contest-rating-double" author="kgevorkyan">
+        <modifyDataType tableName="project" columnName="contest_rating" newDataType="double DEFAULT 0.0"/>
+    </changeSet>
+
 </databaseChangeLog>

save-backend/backend-api-docs.json

@@ -3598,8 +3598,8 @@
         "tags": [
           "projects"
         ],
-        "summary": "Get all avaliable projects.",
-        "description": "Get all projects, avaliable for current user.",
+        "summary": "Get all available projects.",
+        "description": "Get all projects, available for current user.",
         "operationId": "getProjects",
         "parameters": [
           {
@@ -4352,6 +4352,73 @@
         ]
       }
     },
+    "/api/v1/organizations/{organizationName}/get-organization-contest-rating": {
+      "get": {
+        "tags": [
+          "organizations"
+        ],
+        "summary": "Get organization contest rating.",
+        "description": "Get organization contest rating.",
+        "operationId": "getOrganizationContestRating",
+        "parameters": [
+          {
+            "name": "organizationName",
+            "in": "path",
+            "description": "name of an organization",
+            "required": true,
+            "schema": {
+              "type": "string"
+            }
+          },
+          {
+            "name": "X-Authorization-Source",
+            "in": "header",
+            "required": true,
+            "example": "basic"
+          }
+        ],
+        "responses": {
+          "200": {
+            "description": "Successfully get an organization contest rating.",
+            "content": {
+              "*/*": {
+                "schema": {
+                  "type": "number",
+                  "format": "double"
+                }
+              }
+            }
+          },
+          "404": {
+            "description": "Could not find an organization with such name.",
+            "content": {
+              "*/*": {
+                "schema": {
+                  "type": "number",
+                  "format": "double"
+                }
+              }
+            }
+          },
+          "401": {
+            "description": "Unauthorized",
+            "content": {
+              "*/*": {
+                "schema": {
+                  "type": "number",
+                  "format": "double"
+                }
+              }
+            }
+          }
+        },
+        "security": [
+          {
+            "basic": []
+          }
+        ]
+      }
+    },
     "/api/v1/organizations/{organizationName}/avatar": {
       "get": {
         "tags": [
@@ -7053,8 +7120,8 @@
             "$ref": "#/components/schemas/Organization"
           },
           "contestRating": {
-            "type": "integer",
-            "format": "int64"
+            "type": "number",
+            "format": "double"
           },
           "id": {
             "type": "integer",

save-backend/src/main/kotlin/com/saveourtool/save/backend/controllers/OrganizationController.kt

@@ -7,6 +7,7 @@ import com.saveourtool.save.backend.security.OrganizationPermissionEvaluator
 import com.saveourtool.save.backend.service.GitService
 import com.saveourtool.save.backend.service.LnkUserOrganizationService
 import com.saveourtool.save.backend.service.OrganizationService
+import com.saveourtool.save.backend.service.ProjectService
 import com.saveourtool.save.backend.service.TestSuitesService
 import com.saveourtool.save.backend.service.TestSuitesSourceService
 import com.saveourtool.save.backend.storage.TestSuitesSourceSnapshotStorage
@@ -59,6 +60,7 @@ internal class OrganizationController(
     private val testSuitesSourceService: TestSuitesSourceService,
     private val testSuitesService: TestSuitesService,
     private val testSuitesSourceSnapshotStorage: TestSuitesSourceSnapshotStorage,
+    private val projectService: ProjectService
 ) {
     @GetMapping("/all")
     @PreAuthorize("permitAll()")
@@ -427,6 +429,41 @@ internal class OrganizationController(
             ResponseEntity.ok("Git credentials and corresponding data successfully deleted")
         }
 
+    /**
+     * @param organizationName
+     * @param authentication
+     * @return contest rating for organization
+     */
+    @GetMapping("/{organizationName}/get-organization-contest-rating")
+    @RequiresAuthorizationSourceHeader
+    @PreAuthorize("isAuthenticated()")
+    @Operation(
+        method = "Get",
+        summary = "Get organization contest rating.",
+        description = "Get organization contest rating.",
+    )
+    @Parameters(
+        Parameter(name = "organizationName", `in` = ParameterIn.PATH, description = "name of an organization", required = true),
+    )
+    @ApiResponse(responseCode = "200", description = "Successfully get an organization contest rating.")
+    @ApiResponse(responseCode = "404", description = "Could not find an organization with such name.")
+    fun getOrganizationContestRating(
+        @PathVariable organizationName: String,
+        authentication: Authentication,
+    ): Mono<Double> = Mono.just(organizationName)
+        .flatMap {
+            organizationService.findByName(it).toMono()
+        }
+        .switchIfEmptyToNotFound {
+            "Could not find an organization with name $organizationName."
+        }
+        .flatMap {
+            projectService.getNotDeletedProjectsByOrganizationName(organizationName, authentication).collectList()
+        }
+        .map { projectsList ->
+            projectsList.sumOf { it.contestRating }
+        }
+
     private fun cleanupStorageData(testSuite: TestSuite) {
         testSuitesSourceSnapshotStorage.findKey(
             testSuite.source.organization.name,

save-backend/src/main/kotlin/com/saveourtool/save/backend/controllers/ProjectController.kt

@@ -69,8 +69,8 @@ class ProjectController(
     @PreAuthorize("permitAll()")
     @Operation(
         method = "GET",
-        summary = "Get all avaliable projects.",
-        description = "Get all projects, avaliable for current user.",
+        summary = "Get all available projects.",
+        description = "Get all projects, available for current user.",
     )
     @ApiResponse(responseCode = "200", description = "Projects successfully fetched.")
     fun getProjects(
@@ -159,13 +159,7 @@ class ProjectController(
     fun getNonDeletedProjectsByOrganizationName(
         @RequestParam organizationName: String,
         authentication: Authentication?,
-    ): Flux<Project> = projectService.findByOrganizationName(organizationName)
-        .filter {
-            it.status != ProjectStatus.DELETED
-        }
-        .filter {
-            projectPermissionEvaluator.hasPermission(authentication, it, Permission.READ)
-        }
+    ): Flux<Project> = projectService.getNotDeletedProjectsByOrganizationName(organizationName, authentication)
 
     @PostMapping("/save")
     @RequiresAuthorizationSourceHeader

save-backend/src/main/kotlin/com/saveourtool/save/backend/service/LnkContestProjectService.kt

@@ -16,6 +16,7 @@ import org.springframework.transaction.annotation.Transactional
 class LnkContestProjectService(
     private val lnkContestProjectRepository: LnkContestProjectRepository,
     private val lnkContestExecutionService: LnkContestExecutionService,
+    private val projectService: ProjectService,
 ) {
     /**
      * @param contestName name of a [Contest]
@@ -96,9 +97,21 @@ class LnkContestProjectService(
             lnkContestProject.bestExecution = newExecution
             lnkContestProject.bestScore = newExecution.score
             lnkContestProjectRepository.save(lnkContestProject)
+
+            updateProjectContestRating(project)
         }
     }
 
+    private fun updateProjectContestRating(project: Project) {
+        val projectContestRating = lnkContestProjectRepository.findByProject(project).mapNotNull {
+            it.bestScore
+        }.sum()
+
+        projectService.updateProject(project.apply {
+            this.contestRating = projectContestRating
+        })
+    }
+
     /**
      * @param projectCoordinates
      * @param contestName

save-backend/src/main/kotlin/com/saveourtool/save/backend/service/ProjectService.kt

@@ -95,6 +95,22 @@ class ProjectService(
         return projects
     }
 
+    /**
+     * @param organizationName
+     * @param authentication
+     * @return list of not deleted projects
+     */
+    fun getNotDeletedProjectsByOrganizationName(
+        organizationName: String,
+        authentication: Authentication?,
+    ): Flux<Project> = findByOrganizationName(organizationName)
+        .filter {
+            it.status != ProjectStatus.DELETED
+        }
+        .filter {
+            projectPermissionEvaluator.hasPermission(authentication, it, Permission.READ)
+        }
+
     /**
      * @param projectFilters
      * @return project's with filter

save-backend/src/main/kotlin/com/saveourtool/save/backend/service/TestExecutionService.kt

@@ -18,6 +18,7 @@ import com.saveourtool.save.utils.ScoreType
 import com.saveourtool.save.utils.calculateScore
 import com.saveourtool.save.utils.debug
 import com.saveourtool.save.utils.getLogger
+import com.saveourtool.save.utils.isValidScore
 
 import org.apache.commons.io.FilenameUtils
 import org.slf4j.Logger
@@ -246,7 +247,12 @@ class TestExecutionService(private val testExecutionRepository: TestExecutionRep
                     expectedChecks += counters.expectedChecks
                     unexpectedChecks += counters.unexpectedChecks
 
-                    score = toDto().calculateScore(scoreType = ScoreType.F_MEASURE)
+                    val executionScore = toDto().calculateScore(scoreType = ScoreType.F_MEASURE)
+
+                    if (!executionScore.isValidScore()) {
+                        log.error("Execution score for execution id $id is invalid: $executionScore")
+                    }
+                    score = executionScore
                 }
                 executionRepository.save(execution)
             }

save-backend/src/test/kotlin/com/saveourtool/save/backend/service/LnkContestProjectServiceTest.kt

@@ -10,13 +10,17 @@ import org.junit.jupiter.api.extension.ExtendWith
 import org.mockito.kotlin.*
 import org.springframework.beans.factory.annotation.Autowired
 import org.springframework.boot.test.mock.mockito.MockBean
+import org.springframework.boot.test.mock.mockito.MockBeans
 import org.springframework.context.annotation.Import
 import org.springframework.test.context.junit.jupiter.SpringExtension
 import java.util.Optional
 import kotlin.math.abs
 
 @ExtendWith(SpringExtension::class)
 @Import(LnkContestProjectService::class)
+@MockBeans(
+    MockBean(ProjectService::class),
+)
 @Suppress("UnsafeCallOnNullableType")
 class LnkContestProjectServiceTest {
     @Autowired private lateinit var lnkContestProjectService: LnkContestProjectService

save-cloud-common/src/commonMain/kotlin/com/saveourtool/save/entities/Project.kt

@@ -43,7 +43,7 @@ data class Project(
         columnDefinition = "",
     )
     var organization: Organization,
-    var contestRating: Long = 0,
+    var contestRating: Double = 0.0,
 ) {
     /**
      * id of project

save-cloud-common/src/commonMain/kotlin/com/saveourtool/save/utils/ExecutionScoreUtils.kt

@@ -45,13 +45,23 @@ fun ExecutionDto.calculateScore(scoreType: ScoreType): Double = when (type) {
     else -> 0.0
 }
 
+/**
+ * @return true if value is in range (0, 100); false otherwise
+ */
+fun Double.isValidScore() = this.toInt().isValidScore()
+
+/**
+ * @return true if value is in range (0, 100); false otherwise
+ */
+fun Int.isValidScore() = this in 0..100
+
 private fun ExecutionDto.calculateScoreForContestMode(scoreType: ScoreType): Double = when (scoreType) {
     ScoreType.F_MEASURE -> calculateFmeasure()
     else -> TODO("Invalid score type for contest mode!")
 }
 
 private fun ExecutionDto.calculateFmeasure(): Double {
-    val denominator = getPrecisionRate() + getRecallRate()
+    val denominator = (getPrecisionRate() + getRecallRate())
     return if (denominator == 0) {
         0.0
     } else {

save-frontend/src/main/kotlin/com/saveourtool/save/frontend/components/basic/ExecutionLabels.kt

@@ -14,6 +14,7 @@ import com.saveourtool.save.frontend.externals.fontawesome.fontAwesomeIcon
 import com.saveourtool.save.utils.calculateRate
 import com.saveourtool.save.utils.getPrecisionRate
 import com.saveourtool.save.utils.getRecallRate
+import com.saveourtool.save.utils.isValidScore
 
 import csstype.ClassName
 import csstype.Width
@@ -100,12 +101,22 @@ internal class ExecutionStatisticsValues(executionDto: ExecutionDto?) {
             ?: "0"
         this.precisionRate = executionDto
             ?.let {
-                "${it.getPrecisionRate()}"
+                val precisionRate = it.getPrecisionRate()
+                if (precisionRate.isValidScore()) {
+                    precisionRate.toString()
+                } else {
+                    "N/A"
+                }
             }
             ?: "0"
         this.recallRate = executionDto
             ?.let {
-                "${it.getRecallRate()}"
+                val recallRate = it.getRecallRate()
+                if (recallRate.isValidScore()) {
+                    recallRate.toString()
+                } else {
+                    "N/A"
+                }
             }
             ?: "0"
     }

save-frontend/src/main/kotlin/com/saveourtool/save/frontend/components/views/OrganizationView.kt

@@ -613,7 +613,7 @@ class OrganizationView : AbstractView<OrganizationProps, OrganizationViewState>(
             topProject?.let {
                 scoreCard {
                     name = it.name
-                    contestScore = it.contestRating.toDouble()
+                    contestScore = it.contestRating
                     url = "#/${props.organizationName}/${it.name}"
                 }
             }