Skip to content
GitLab
Menu
Projects
Groups
Snippets
Help
Projects
Groups
Snippets
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Sign in
Toggle navigation
Menu
Open sidebar
loongten
metersphere
Commits
31b25364
Commit
31b25364
authored
3 years ago
by
song-tianyang
Committed by
fit2-zhao
3 years ago
Browse files
Options
Download
Email Patches
Plain Diff
fix(接口测试): 修复请求中包含误报断言和失败断言时最终用例执行结果失败的缺陷
修复请求中包含误报断言和失败断言时最终用例执行结果失败的缺陷
parent
2e29e163
Changes
3
Hide whitespace changes
Inline
Side-by-side
Showing
3 changed files
backend/src/main/java/io/metersphere/api/dto/definition/parse/JMeterScriptUtil.java
+4
-4
...etersphere/api/dto/definition/parse/JMeterScriptUtil.java
backend/src/main/java/io/metersphere/commons/utils/ErrorReportLibraryUtil.java
+3
-1
.../io/metersphere/commons/utils/ErrorReportLibraryUtil.java
backend/src/main/java/io/metersphere/commons/utils/ResponseUtil.java
+3
-2
.../main/java/io/metersphere/commons/utils/ResponseUtil.java
with
10 additions
and
7 deletions
+10
-7
backend/src/main/java/io/metersphere/api/dto/definition/parse/JMeterScriptUtil.java
+
4
-
4
View file @
31b25364
...
...
@@ -74,17 +74,17 @@ public class JMeterScriptUtil {
boolean
globalPreScriptIsFilter
=
JMeterScriptUtil
.
isScriptFilter
(
preFilterProtocal
,
protocal
);
boolean
globalPostScriptIsFilter
=
JMeterScriptUtil
.
isScriptFilter
(
postFilterProtocal
,
protocal
);
if
(
isAfterPrivateScript
)
{
if
(
isPreScriptExecAfterPrivateScript
&&
!
globalPreScriptIsFilter
)
{
if
(
isPreScriptExecAfterPrivateScript
&&
!
globalPreScriptIsFilter
&&
StringUtils
.
isNotEmpty
(
preProcessor
.
getScript
())
)
{
addItemHashTree
(
preProcessor
,
samplerHashTree
,
config
,
environmentId
);
}
if
(
isPostScriptExecAfterPrivateScript
&&
!
globalPostScriptIsFilter
)
{
if
(
isPostScriptExecAfterPrivateScript
&&
!
globalPostScriptIsFilter
&&
StringUtils
.
isNotEmpty
(
postProcessor
.
getScript
())
)
{
addItemHashTree
(
postProcessor
,
samplerHashTree
,
config
,
environmentId
);
}
}
else
{
if
(!
isPreScriptExecAfterPrivateScript
&&
!
globalPreScriptIsFilter
)
{
if
(!
isPreScriptExecAfterPrivateScript
&&
!
globalPreScriptIsFilter
&&
StringUtils
.
isNotEmpty
(
preProcessor
.
getScript
())
)
{
addItemHashTree
(
preProcessor
,
samplerHashTree
,
config
,
environmentId
);
}
if
(!
isPostScriptExecAfterPrivateScript
&&
!
globalPostScriptIsFilter
)
{
if
(!
isPostScriptExecAfterPrivateScript
&&
!
globalPostScriptIsFilter
&&
StringUtils
.
isNotEmpty
(
postProcessor
.
getScript
())
)
{
addItemHashTree
(
postProcessor
,
samplerHashTree
,
config
,
environmentId
);
}
}
...
...
This diff is collapsed.
Click to expand it.
backend/src/main/java/io/metersphere/commons/utils/ErrorReportLibraryUtil.java
+
3
-
1
View file @
31b25364
...
...
@@ -21,12 +21,14 @@ public class ErrorReportLibraryUtil {
if
(
result
!=
null
&&
result
.
getResponseResult
()
!=
null
&&
CollectionUtils
.
isNotEmpty
(
result
.
getResponseResult
().
getAssertions
()))
{
List
<
ResponseAssertionResult
>
errorReportAssertionList
=
new
ArrayList
<>();
boolean
hasOtherErrorAssertion
=
false
;
int
otherAssertionCount
=
0
;
for
(
ResponseAssertionResult
assertion
:
result
.
getResponseResult
().
getAssertions
())
{
if
(
StringUtils
.
startsWith
(
assertion
.
getContent
(),
ERROR_CODE_START
))
{
errorReportAssertionList
.
add
(
assertion
);
}
else
{
if
(!
assertion
.
isPass
()){
hasOtherErrorAssertion
=
true
;
otherAssertionCount
++;
}
}
}
...
...
@@ -46,7 +48,7 @@ public class ErrorReportLibraryUtil {
if
(
result
.
getResponseResult
()
!=
null
&&
StringUtils
.
equalsIgnoreCase
(
result
.
getResponseResult
().
getResponseCode
(),
"200"
)
&&
result
.
getError
()
>
0
)
{
if
(!
hasOtherErrorAssertion
){
if
(
otherAssertionCount
==
0
||
!
hasOtherErrorAssertion
){
result
.
setError
(
result
.
getError
()
-
1
);
}
}
...
...
This diff is collapsed.
Click to expand it.
backend/src/main/java/io/metersphere/commons/utils/ResponseUtil.java
+
3
-
2
View file @
31b25364
...
...
@@ -15,11 +15,12 @@ import java.util.Map;
public
class
ResponseUtil
{
public
static
RequestResultExpandDTO
parseByRequestResult
(
RequestResult
requestResult
)
{
//解析是否含有误报库信息
ErrorReportLibraryParseDTO
errorCodeDTO
=
ErrorReportLibraryUtil
.
parseAssertions
(
requestResult
);
RequestResultExpandDTO
expandDTO
=
new
RequestResultExpandDTO
();
BeanUtils
.
copyBean
(
expandDTO
,
requestResult
);
//解析是否含有误报库信息
ErrorReportLibraryParseDTO
errorCodeDTO
=
ErrorReportLibraryUtil
.
parseAssertions
(
requestResult
);
if
(
CollectionUtils
.
isNotEmpty
(
errorCodeDTO
.
getErrorCodeList
())){
Map
<
String
,
String
>
expandMap
=
new
HashMap
<>();
expandDTO
.
setStatus
(
ExecuteResult
.
errorReportResult
.
name
());
...
...
This diff is collapsed.
Click to expand it.
Write
Preview
Supports
Markdown
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment
Menu
Projects
Groups
Snippets
Help