EPMRPP-82327 || Formatting code according to code style (#1688)

* EPMRPP-82327-google-style google-style applied * EPMRPP-82327-google-style formatted new changes * EPMRPP-82327-google-style fixed import in test
reportportal · Mar 16, 2023 · 741f4eb · 741f4eb
1 parent eb754da
commit 741f4eb
Show file tree

Hide file tree

Showing 1,118 changed files with 63,843 additions and 57,850 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,12 +1,13 @@
 # Changelog
 
 ## 2.6.0
+
 ##### Released: 17 October 2016
 
 ### New Features
 
 * Initial release to Public Maven Repositories
-* DockerHub Release 
+* DockerHub Release
 * Introduce Personal Spaces
 
 ### Bugfixes
@@ -19,6 +20,7 @@
 * EPMRPP-20926 - Don't allow to create project with type = Personal
 
 ## 2.7.0
+
 ##### Released: XXX October 2016
 
 ### New Features
@@ -36,24 +38,24 @@
 * EPMRPP-22340 - API: It's possible to delete project with entryType=UPSA
 * reportportal/reportportal#9 - Fix incorrect statistics calculation
 
-
 ## 2.7.1
+
 ##### Released: 21 October 2016
 
 ### Bugfixes
 
 * Minor fix: incorrect project version in /info endpoint
 
-
 ## 2.7.2
+
 ##### Released: 22 October 2016
 
 ### Bugfixes
 
-* Expose JVM args as ENV variable to make it configurable 
-
+* Expose JVM args as ENV variable to make it configurable
 
 ## 3.0.0
+
 ##### Released: Apr 20, 2017
 
 ### New Features
@@ -64,35 +66,44 @@
 
 ### Bugfixes
 
-* EPMRPP-23001 - In case login contains only underscore symbol the unclassified error (500) is returned 
-* EPMRPP-23342 - Permissions: User with not admin account role is able to get list of users of PR if using search
-* EPMRPP-23541 - Widget is not become shared after changing option of the dashboard via Edit Dashboard.
+* EPMRPP-23001 - In case login contains only underscore symbol the unclassified error (500) is
+  returned
+* EPMRPP-23342 - Permissions: User with not admin account role is able to get list of users of PR if
+  using search
+* EPMRPP-23541 - Widget is not become shared after changing option of the dashboard via Edit
+  Dashboard.
 * EPMRPP-23564 - GET shared dashboard request does not contain 'description' parameter
 * EPMRPP-23197 - Email server: Add field for Sender address on server settings
-* EPMRPP-23601 - 'External-system' parameter is missed for GET ticket request of posted bug in case launches were merged
-* EPMRPP-23651 - The sender of email of user invitation is not the same as was set on Email server settings
-* EPMRPP-23664 - Sender of email notification of finish launch is not the same as mentioned on Project settings page
+* EPMRPP-23601 - 'External-system' parameter is missed for GET ticket request of posted bug in case
+  launches were merged
+* EPMRPP-23651 - The sender of email of user invitation is not the same as was set on Email server
+  settings
+* EPMRPP-23664 - Sender of email notification of finish launch is not the same as mentioned on
+  Project settings page
 * EPMRPP-23459 - test log item time after parent item's start time
 * reportportal#64 - Please correct limitation: test log item time after parent item's start time
-* EPMRPP-23679 - Unclassified error for long search string by digital filter such as Total, Passed, Faled, etc.
+* EPMRPP-23679 - Unclassified error for long search string by digital filter such as Total, Passed,
+  Faled, etc.
 * EPMRPP-23680 - Unclassified error when searching for launch with special sybmols in its name
 * EPMRPP-21270 - Launches: Sorting for defect statistics works incorrectly
 * EPMRPP-23641 - DASHBOARDS: Description disappeared after refresh
 * EPMRPP-23691 - WS: To combine 'from' using sender name and email when the notification is sending
 * EPMRPP-20425 - Allow to delete Not own dashboards/widgets/filters by PM
 * EPMRPP-23076 - Script for deleting favorite dashboards with new implementation
 * EPMRPP-23741 - Update the info from RP notification center in invitation email
-* EPMRPP-23744 - Error handling: When the invitation is sent to email address that already available in DB, ws returns incorrect error message
+* EPMRPP-23744 - Error handling: When the invitation is sent to email address that already available
+  in DB, ws returns incorrect error message
 * EPMRPP-23468 - User with project role=PM/LEAD unable to invite user
 * EPMRPP-23692 - Demo data: fix line-breakes in Logs
 * EPMRPP-24570 - Status statistics is not updated in case user deletes items from SUITE/TEST levels
 * EPMRPP-24914 - Item with investigated defect type only is not included in scope of analysis
 * reportportal#31/EPMRPP-25006 - Widget limits clear up
 * reportportal#92 - Fixed negative value int the "TI" counter for merged launch
-* EPMRPP-25255 - Personal projects have TEST_BASES calculation strategy (BDD) instead of STEP_BASED (regular)
-
+* EPMRPP-25255 - Personal projects have TEST_BASES calculation strategy (BDD) instead of
+  STEP_BASED (regular)
 
 ## 3.1.0
+
 ##### Released: Aug 5, 2017
 
 ### New Features
@@ -103,7 +114,8 @@
 * EPMRPP-26010 - Implemented Passing rate widget
 * EPMRPP-26021 - Hash for unique instance generating by server
 * reportportal#176/EPMRPP-26045 - Split Test Description and test parameters
-* EPMRPP-26131 - Unique id for test item based on items' names, parameters, launch name, project name
+* EPMRPP-26131 - Unique id for test item based on items' names, parameters, launch name, project
+  name
 * EPMRPP-26263 - Implemented latest launches view
 * EPMRPP-26394 - JIRA issue with empty set of fields for post bug form
 
@@ -112,14 +124,16 @@
 * EPMRPP-25346 - WS: No email notification letter on launch finished is received
 * EPMRPP-24913 - Demo Data Postfix should be unique
 * EPMRPP-25247 - Admin is not able to modify his own role on the project
-* EPMRPP-23287 - Issues (defects) statistics should be calculated by Tests and not by Steps with test based strategy
+* EPMRPP-23287 - Issues (defects) statistics should be calculated by Tests and not by Steps with
+  test based strategy
 * EPMRPP-25434 - Remove attached logo from email letter
 * EPMRPP-25424 - Request with empty recipients are sent to server with set notifications to OFF
 * EPMRPP-25433 - No custom defects are in the email letter on launch finished.
 * EPMRPP-25474 - Filtering: No suggestion results for search by owner of the launch
 * EPMRPP-25506 - Deleted by PM not own shared widget is not removed from the system (DB)
-* EPMRPP-25320 - Internal project is replaced with personal in case they have identical names 
-* EPMRPP-24853 - Next, Previous items should contain only visible items in case 'Collapse precondition methods' functionality is ON
+* EPMRPP-25320 - Internal project is replaced with personal in case they have identical names
+* EPMRPP-24853 - Next, Previous items should contain only visible items in case 'Collapse
+  precondition methods' functionality is ON
 * EPMRPP-25526 - Unshared filter is still shown on Launches page for not owner
 * EPMRPP-25476 - WS: Make 'share'/'isShare' parameter written the same for POST and GET methods
 * EPMRPP-25782 - Share option of widgets is not saved on backend
@@ -133,8 +147,8 @@
 * reportportal/reportportal#12, reportportal/reportportal#65 - LDAP Authorization
 * reportportal/reportportal#208 - Introduce new project role that is not allowed to report
 
-
 ## 3.2
+
 ##### Released: Aug XXX, 2017
 
 ### New Features
@@ -144,19 +158,22 @@
 * EPMRPP-29222 - Introduce LDAP authorization
 
 ### Bugfixes
+
 * reportportal/reportportal#170 - Test run breaks with unclassified error (jbehave) #170
 * reportportal#176/EPMRPP-26045 - Split Test Description and test parameters
 * EPMRPP-29167 - Statistics for deleted elements with custom defect types are still present
-* EPMRPP-26131 - Unique id for test item based on items' names, parameters, launch name, project name
+* EPMRPP-26131 - Unique id for test item based on items' names, parameters, launch name, project
+  name
 * EPMRPP-26263 - Implemented latest launches view
 * EPMRPP-26394 - JIRA issue with empty set of fields for post bug form
 * EPMRPP-29345 - Description and tags are not applied in case they were specified on finishLaunch
 * EPMRPP-29361 - No content of "Overall statistics" widget created for Demo DataUI
 * EPMRPP-29405 - WS: Widget with Latest launches=ON includes launches from Debug
-* EPMRPP-29337 - Widgets with Latest Launches ON include statistics for launches with In Progress status
-
+* EPMRPP-29337 - Widgets with Latest Launches ON include statistics for launches with In Progress
+  status
 
 ## 3.3
+
 ##### Released: XXX XX, 2017
 
 ### New Features
@@ -174,6 +191,7 @@
 * EPMRPP-29701: Tag prefix should be fully matched with the searched value
 
 ## 4.0
+
 ##### Released: Mar 01, 2018
 
 ### New Features
@@ -185,7 +203,7 @@
 * EPMRPP-31220 - WS: removing Match issue
 * EPMRPP-31189 - Update script for widgets.
 * EPMRPP-31780 - Set the name of analyzer which made the changes
-* EPMRPP-33154 - [IMPORT] Better handling of importing exceptions 
+* EPMRPP-33154 - [IMPORT] Better handling of importing exceptions
 * reportportal/reportportal#322 - Make startTestItemRQ in API 4.x case insensitive
 
 ### Bugfixes
@@ -194,15 +212,19 @@
 * EPMRPP-31020 - Launches reported to DEFAULT mode are present in DEBUG as well
 * reportportal/reportportal#245 - Default TestNG xml report isn't compatible with RP report
 * EPMRPP-29701 - Tag prefix should be fully matched with the searched value
-* EPMRPP-31156 - Different launches comparison chart should include only last 2 launches despite sorting of the filter
+* EPMRPP-31156 - Different launches comparison chart should include only last 2 launches despite
+  sorting of the filter
 * EPMRPP-31039 - WS: Results of items on history table do not have sorting in Asc order
 * EPMRPP-31188 - [DASHBOARD] Owner can't see owned widgets in "SELECT SHARED WIDGET" list #213
-* EPMRPP-31308 - [WS]: Add a constrain for a notification [GITHUB] Notifications rule for rp.mode #249
-* EPMRPP-31233 - [IMPORT] Status is always PASSED if parent item has children with different statuses
+* EPMRPP-31308 - [WS]: Add a constrain for a notification [GITHUB] Notifications rule for rp.mode
+  #249
+* EPMRPP-31233 - [IMPORT] Status is always PASSED if parent item has children with different
+  statuses
 * EPMRPP-29375 - WS: Search for user with dot symbol in login name returns the error
 * EPMRPP-31211 - In case the tag contains (") symbols, the filter does not have got any value
 * EPMRPP-31208 - Email template: Tags with special symbols do not work correctly
-* EPMRPP-25622 - Posted bug is not added to item activity in case it was submitted via bulk operation
+* EPMRPP-25622 - Posted bug is not added to item activity in case it was submitted via bulk
+  operation
 * EPMRPP-31435 - 'Ignore in AA' should not to be set for test item in launch on DEBUG level
 * EPMRPP-31447 - WS: "ActionType"=analyze_item for action that was done by user
 * Fix issue with incorrect importing of some junit files
@@ -218,6 +240,7 @@
 * EPMRPP-33492 - WS: Logs of launches from DEBUG level are get to ES during the indexing
 
 ## 4.2
+
 ##### Released: Jun 14, 2018
 
 ### Bugfixes
@@ -228,45 +251,54 @@
 * EPMRPP-34212 - WS: No action for linked issue by ML on History of actions
 
 ## 4.3
+
 ##### Released: 2018
 
 ### Bugfixes
 
 * EPMRPP-35134 - Registration is passed for user with login already in use
-* EPMRPP-35289 - WS: GET item controller returns items that do not belong to specified project, but all available items
+* EPMRPP-35289 - WS: GET item controller returns items that do not belong to specified project, but
+  all available items
 * EPMRPP-35291 - WS: Unclassified error on get latest launches. Improve latest query performance
 
 ## 5.0.0
 
 ### Bugfixes
 
-
 * EPMRPP-39590 - Restore of the password. Incorrect response from repeated request.
-* EPMRPP-40298 - API. Global Email server integration (Gmail). Incorrect number of emails deliver to the mailbox.
-* EPMRPP-41407 - Filters page. Unclassified error occurs when user turns filters On/Off in 2 browser tabs
-* EPMRPP-41361 - All launches and not 'Launches with the same name' applied in case of automatic analysis on finish launch
+* EPMRPP-40298 - API. Global Email server integration (Gmail). Incorrect number of emails deliver to
+  the mailbox.
+* EPMRPP-41407 - Filters page. Unclassified error occurs when user turns filters On/Off in 2 browser
+  tabs
+* EPMRPP-41361 - All launches and not 'Launches with the same name' applied in case of automatic
+  analysis on finish launch
 * EPMRPP-41388 - Level of log is not sent to 'Edit defect type' modal.
 * EPMRPP-41569 - 'AA' flag is shown on UI despite the last action (link issue) was done by user
-* EPMRPP-41055 - Incorrect response (500 error) in Launches page after deleting member from the project.
+* EPMRPP-41055 - Incorrect response (500 error) in Launches page after deleting member from the
+  project.
 * EPMRPP-39525 - Launches and filters are not shown for project if it has name 'project'
 * EPMRPP-40808 - Dashboard. PM can't delete not own shared dashboard
 * EPMRPP-41168 - Admin can't delete test items on the Step View
 * EPMRPP-42274 - History of actions. No actions are displayed for items analyzed manually.
 * EPMRPP-35338 - Add number of run to launch name on widgets for the particular launch.
 * EPMRPP-42004 - Step level. Unclassified error on bulk post/link issue
-* EPMRPP-42337 - 'Issue' parameter should not be available for items without issue type or items have status Passed.
-
+* EPMRPP-42337 - 'Issue' parameter should not be available for items without issue type or items
+  have status Passed.
 
 ## 5.3.0
+
 ##### Released: XXX
 
 ### New Features
+
 * EPMRPP-52161 - Parent line recalculation (Topliner)
 
 ### Bugfixes
+
 * reportportal/reportportal#773 - Service-API errors when user does not have a photo
 * EPMRPP-50276 - Auto-test issue/ When delete user - ACL cache should be cleaned up
-* EPMRPP-49121 - Validation message should appear in case global integration of email server is not setup
+* EPMRPP-49121 - Validation message should appear in case global integration of email server is not
+  setup
 * EPMRPP-52660 - Add launches filtering by items with issues inside
 * reportportal/reportportal#995 - Timezone unsynced between "LAUNCHES" and "DASHBOARD"
 
diff --git a/Jenkinsfile-candidate b/Jenkinsfile-candidate
@@ -1,6 +1,6 @@
 #!groovy
 properties([
-        parameters ([
+        parameters([
                 string(
                         name: "VERSION",
                         defaultValue: "",

diff --git a/Jenkinsfile.k8s.groovy b/Jenkinsfile.k8s.groovy
@@ -20,9 +20,9 @@ def setupJob(branch = 'develop', pollScm = "H/10 * * * *") {
 def String readSecretsDirectory(String dirName) {
     echo "Reading directory ${dirName}"
     String fileNames = sh(script: "ls -1 $dirName", returnStdout: true)
-    String [] fileList = fileNames.split('\\n')
+    String[] fileList = fileNames.split('\\n')
     return fileList.collect {
-        it + '='+ sh(script: "cat $dirName/$it", returnStdout: true).trim()
+        it + '=' + sh(script: "cat $dirName/$it", returnStdout: true).trim()
     }
 }
 
@@ -101,7 +101,7 @@ podTemplate(
                 }
             }
         }, 'Download Sealights': {
-            stage('Download Sealights'){
+            stage('Download Sealights') {
                 dir(sealightsDir) {
                     sh "wget ${sealightsAgentUrl}"
                     unzip sealightsAgentArchive
@@ -122,8 +122,8 @@ podTemplate(
 
         def sealightsToken = util.execStdout("cat $sealightsTokenPath")
         def sealightsSession = "";
-        stage ('Init Sealights') {
-            if(enableSealights) {
+        stage('Init Sealights') {
+            if (enableSealights) {
                 dir(sealightsDir) {
                     container('jre') {
                         echo "Generating Sealights build session ID"
@@ -146,16 +146,17 @@ podTemplate(
         }
 
         def jvmArgs = params.get('JVM_ARGS')
-        if(jvmArgs == null || jvmArgs.trim().isEmpty()){
+        if (jvmArgs == null || jvmArgs.trim().isEmpty()) {
             jvmArgs = '-Xms2G -Xmx3g -DLOG_FILE=app.log -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=/tmp'
         }
 
-        if(enableSealights){
+        if (enableSealights) {
             jvmArgs = jvmArgs + ' -javaagent:./plugins/sl-test-listener.jar -Dsl.tokenFile=sealights-token.txt -Dsl.buildSessionIdFile=sealights-session.txt -Dsl.filesStorage=/tmp'
         }
 
         stage('Deploy to Dev Environment') {
-            helm.deploy("$k8sDir/reportportal/v5", ["serviceapi.repository": srvRepo, "serviceapi.tag": srvVersion, "serviceapi.jvmArgs" : "\"$jvmArgs\""], false) // without wait
+            helm.deploy("$k8sDir/reportportal/v5", ["serviceapi.repository": srvRepo, "serviceapi.tag": srvVersion, "serviceapi.jvmArgs": "\"$jvmArgs\""], false)
+            // without wait
         }
 
         stage('Execute DVT Tests') {

diff --git a/README.md b/README.md
@@ -1,4 +1,5 @@
 # ReportPortal. Main API Module
+
 [![Docker Pulls](https://img.shields.io/docker/pulls/reportportal/service-api.svg?maxAge=159200)](https://hub.docker.com/r/reportportal/service-api/)
 [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0)
 [![stackoverflow](https://img.shields.io/badge/reportportal-stackoverflow-orange.svg?style=flat)](http://stackoverflow.com/questions/tagged/reportportal)
@@ -10,5 +11,6 @@
 [![Total alerts](https://img.shields.io/lgtm/alerts/g/reportportal/service-api.svg?logo=lgtm&logoWidth=18)](https://lgtm.com/projects/g/reportportal/service-api/alerts/)
 
 ## Copyright Notice
+
 Licensed under the [Apache 2.0](https://www.apache.org/licenses/LICENSE-2.0)
 license (see the LICENSE file).
diff --git a/docker/docker-compose.yml b/docker/docker-compose.yml
@@ -70,7 +70,7 @@ services:
       MINIO_SECRET_KEY: minio123
     command: server /data
     healthcheck:
-      test: ["CMD", "curl", "-f", "http://localhost:9000/minio/health/live"]
+      test: [ "CMD", "curl", "-f", "http://localhost:9000/minio/health/live" ]
       interval: 30s
       timeout: 20s
       retries: 3