inteligr8/SearchServices
1
0
Fork 0
mirror of https://github.com/Alfresco/SearchServices.git synced 2026-09-16 18:12:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

REPO-2112 / MNT-17545: update test with bug.status.Fixed to not be skipped.

Browse Source 
Also, removed double quotes from test as a new issue was raised: MNT-17594 RegEx for filename fails / Regular Expression constraints not checked properly
This commit is contained in:
Mihaela Popa
2017-03-14 15:46:25 +02:00
parent dd44737711
commit c14396f967
1 changed files with 3 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
e2e-test/java/org/alfresco/rest/nodes/NodesContentTests.java
+3 -3
View File
@@ -44,7 +44,7 @@ public class NodesContentTests extends RestTest
restClient.assertHeaderValueContains("Content-Disposition", String.format("filename=\"%s\"", file1.getName()));
}
@Bug(id="REPO-2112")
@Bug(id="MNT-17545", description = "HTTP Header Injection in ContentStreamer", status = Bug.Status.FIXED)
@TestRail(section = { TestGroup.REST_API,TestGroup.NODES }, executionType = ExecutionType.REGRESSION,
description = "Verify file name with special chars is escaped in Content-Disposition header")
@Test(groups = { TestGroup.REST_API, TestGroup.NODES, TestGroup.CORE})
@@ -53,10 +53,10 @@ public class NodesContentTests extends RestTest
char c1 = 127;
char c2 = 31;
char c3 = 256;
FileModel file = dataContent.usingUser(user2).usingSite(site2).createContent(new FileModel("\ntest\"" + c1 + c2 + c3, FileType.TEXT_PLAIN));
FileModel file = dataContent.usingUser(user2).usingSite(site2).createContent(new FileModel("\ntest" + c1 + c2 + c3, FileType.TEXT_PLAIN));
restClient.authenticateUser(user2).withCoreAPI().usingNode(file).usingParams("attachment=false").getNodeContent();
restClient.assertStatusCodeIs(HttpStatus.OK);
restClient.assertHeaderValueContains("Content-Disposition","filename=\" test .txt\"");
restClient.assertHeaderValueContains("Content-Disposition","filename=\" test .txt\"");
}
}