Skip to content

Commit

Permalink
TP fix the integration test failures. (#7792)
Browse files Browse the repository at this point in the history
* Fix TP integration tests

* Readd tests

---------

Co-authored-by: Steve Hamrick <shamrick@apache.org>
  • Loading branch information
shamrickus and shamrickus committed Sep 8, 2023
1 parent fe1aa82 commit 0410344
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 9 deletions.
2 changes: 1 addition & 1 deletion .github/actions/tp-integration-tests/entrypoint.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ onFail() {
if [[ -f "${REPO_DIR}/traffic_ops/traffic_ops_golang/out.log" ]]; then
mv "${REPO_DIR}/traffic_ops/traffic_ops_golang/out.log" Reports/to.log
fi
docker logs $CHROME_CONTAINER > Reports/chromium.log 2>&1;
docker logs $CHROME_CONTAINER > Reports/chrome.log 2>&1;
docker logs $HUB_CONTAINER > Reports/hub.log 2>&1;
echo "Detailed logs produced info Reports artifact"
exit 1
Expand Down
5 changes: 2 additions & 3 deletions traffic_portal/test/integration/Data/deliveryservices.ts
Original file line number Diff line number Diff line change
Expand Up @@ -121,8 +121,7 @@ export const deliveryservices = {
name: "location",
value: "/a/b/c/d",
configFile: "remap.config",
secure: false,
noRandomize: true
secure: false
}
]
},
Expand Down Expand Up @@ -159,7 +158,7 @@ export const deliveryservices = {
qstringIgnore: 0,
rangeRequestHandling: 0,
regionalGeoBlocking: false,
requiredCapabilities: ["DSTestCap"],
requiredCapabilities: [],
tenantId: 0,
typeId: 1,
xmlId: "dstestro1",
Expand Down
8 changes: 4 additions & 4 deletions traffic_portal/test/integration/Data/topologies.ts
Original file line number Diff line number Diff line change
Expand Up @@ -315,7 +315,7 @@ export const topologies = {
data: [
{
cacheGroupID: 0,
cdnID: 2,
cdnID: 0,
domainName: "test.net",
hostName: "topologieserver1",
httpsPort: 443,
Expand Down Expand Up @@ -350,7 +350,7 @@ export const topologies = {
mgmtIpGateway: "",
mgmtIpNetmask: "",
offlineReason: "",
physLocationID: 2,
physicalLocationID: 2,
profiles: ["TopTestPf"],
routerHostName: "",
routerPortName: "",
Expand Down Expand Up @@ -381,7 +381,7 @@ export const topologies = {
},
{
cacheGroupID: 0,
cdnID: 2,
cdnID: 0,
domainName: "test.net",
hostName: "topologieserver3",
httpsPort: 443,
Expand Down Expand Up @@ -434,7 +434,7 @@ export const topologies = {
route: "/phys_locations",
queryKey: "name",
queryValue: "TopTestPhys",
replace: "physLocationID"
replace: "physicalLocationID"
},
{
route: "/cdns",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ export class DeliveryServicePage extends BasePage {
element(by.name("active")).sendKeys("Active"),
element(by.id("type")).sendKeys(type),
element(by.name("tenantId")).click().then(() => element(by.name(tenant)).click()),
element(by.name("cdn")).sendKeys("dummycdn")
element(by.name("cdn")).sendKeys("dummycdn"+randomize)
);

await Promise.all(ps);
Expand Down

0 comments on commit 0410344

Please sign in to comment.