fix: trim whitespace when parsing numeric properties #2760
Annotations
3 errors and 1 warning
Test:
src\core\src\test\kotlin\org\apache\jmeter\threads\ThreadGroupTest.kt#L1
Replace 61 lines 1..59 with
/*
* Licensed to the Apache Software Foundation (ASF) under one or more
* contributor license agreements. See the NOTICE file distributed with
* this work for additional information regarding copyright ownership.
* The ASF licenses this file to you under the Apache License, Version 2.0
* (the "License"); you may not use this file except in compliance with
* the License. You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package org.apache.jmeter.threads
import org.apache.jmeter.control.LoopController
...41 more lines
|
Test:
task ':src:core:autostyleKotlinCheck'#L1
Execution failed for task ':src:core:autostyleKotlinCheck':
See 'What went wrong' below
|
Test
Execution failed for task ':src:core:autostyleKotlinCheck'.
> The following files have format violations:
src\test\kotlin\org\apache\jmeter\threads\ThreadGroupTest.kt
@@ -1,59 +1,61 @@
-/*\r\n
- * Licensed to the Apache Software Foundation (ASF) under one or more\r\n
- * contributor license agreements. See the NOTICE file distributed with\r\n
- * this work for additional information regarding copyright ownership.\r\n
- * The ASF licenses this file to you under the Apache License, Version 2.0\r\n
- * (the "License"); you may not use this file except in compliance with\r\n
- * the License. You may obtain a copy of the License at\r\n
- *\r\n
- * http://www.apache.org/licenses/LICENSE-2.0\r\n
- *\r\n
- * Unless required by applicable law or agreed to in writing, software\r\n
- * distributed under the License is distributed on an "AS IS" BASIS,\r\n
- * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\r\n
- * See the License for the specific language governing permissions and\r\n
- * limitations under the License.\r\n
- */\r\n
-\r\n
-package org.apache.jmeter.threads\r\n
-\r\n
-import org.apache.jmeter.control.LoopController\r\n
-import org.apache.jmeter.junit.JMeterTestCase\r\n
-import org.apache.jmeter.test.assertions.executePlanAndCollectEvents\r\n
-import org.apache.jmeter.test.samplers.ThreadSleep\r\n
-import org.apache.jmeter.treebuilder.TreeBuilder\r\n
-import org.junit.jupiter.api.Assertions.assertEquals\r\n
-import org.junit.jupiter.api.Test\r\n
-import kotlin.time.Duration.Companion.seconds\r\n
-\r\n
-class ThreadGroupTest : JMeterTestCase() {\r\n
- @test\r\n
- fun `threadNum with trailing whitespace`() {\r\n
- val events = executePlanAndCollectEvents(10.seconds, fun TreeBuilder.() {\r\n
- ThreadGroup::class {\r\n
- props {\r\n
- it[numThreads] = "1 "\r\n
- }\r\n
- rampUp = 0\r\n
- scheduler = true\r\n
- delay = 0\r\n
- duration = 1\r\n
- setSamplerController(\r\n
- LoopController().apply {\r\n
- loops = 1\r\n
- setContinueForever(false)\r\n
- }\r\n
- )\r\n
-\r\n
- ThreadSleep::class {\r\n
... (72 more lines that didn't fit)
You might want to adjust -PmaxCheckMessageLines=50
-PmaxFilesToList=10
-PminLinesPerFile=4
to see more violations
Run './gradlew autostyleApply' to fix the violations.
|
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: burrunan/gradle-cache-action@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
Loading