Skip to content

Commit 1dbe022

Browse files
authored
e2e false positives (#1840)
* try/catch login
1 parent be4f691 commit 1dbe022

File tree

1 file changed

+31
-10
lines changed

1 file changed

+31
-10
lines changed

tests/e2e/tutorials/tutorialBase.js

Lines changed: 31 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,13 @@ class TutorialBase {
2323
}
2424

2525
initScreenshoter() {
26-
utils.createScreenshotsDir();
26+
try {
27+
utils.createScreenshotsDir();
28+
}
29+
catch(err) {
30+
console.error("Error creating screenshots directory", err);
31+
throw(err);
32+
}
2733

2834
this.__interval = setInterval(async() => {
2935
await this.takeScreenshot();
@@ -35,12 +41,18 @@ class TutorialBase {
3541
}
3642

3743
async start() {
38-
await this.beforeScript();
39-
await this.goTo();
44+
try {
45+
await this.beforeScript();
46+
await this.goTo();
4047

41-
const needsRegister = await this.registerIfNeeded();
42-
if (!needsRegister) {
43-
await this.login();
48+
const needsRegister = await this.registerIfNeeded();
49+
if (!needsRegister) {
50+
await this.login();
51+
}
52+
}
53+
catch(err) {
54+
console.error("Error starting", err);
55+
throw(err);
4456
}
4557
}
4658

@@ -68,10 +80,9 @@ class TutorialBase {
6880
async openStudyLink(openStudyTimeout = 20000) {
6981
this.__responsesQueue.addResponseListener("open");
7082

71-
await this.goTo();
72-
7383
let resp = null;
7484
try {
85+
await this.goTo();
7586
resp = await this.__responsesQueue.waitUntilResponse("open", openStudyTimeout);
7687
}
7788
catch(err) {
@@ -93,7 +104,15 @@ class TutorialBase {
93104
this.__responsesQueue.addResponseListener("projects?type=template");
94105
this.__responsesQueue.addResponseListener("catalog/dags");
95106
this.__responsesQueue.addResponseListener("services");
96-
await auto.logIn(this.__page, this.__user, this.__pass);
107+
108+
try {
109+
await auto.logIn(this.__page, this.__user, this.__pass);
110+
}
111+
catch(err) {
112+
console.error("Failed logging in", err);
113+
throw(err);
114+
}
115+
97116
try {
98117
const resp = await this.__responsesQueue.waitUntilResponse("projects?type=template");
99118
const templates = resp["data"];
@@ -106,6 +125,7 @@ class TutorialBase {
106125
console.error("Templates could not be fetched", err);
107126
throw(err);
108127
}
128+
109129
try {
110130
const resp = await this.__responsesQueue.waitUntilResponse("catalog/dags");
111131
const dags = resp["data"];
@@ -118,6 +138,7 @@ class TutorialBase {
118138
console.error("DAGs could not be fetched", err);
119139
throw(err);
120140
}
141+
121142
try {
122143
const resp = await this.__responsesQueue.waitUntilResponse("services");
123144
const services = resp["data"];
@@ -246,8 +267,8 @@ class TutorialBase {
246267
await auto.toDashboard(this.__page);
247268
await this.takeScreenshot("dashboardDeleteFirstStudy_before");
248269
this.__responsesQueue.addResponseListener("projects/");
249-
await auto.dashboardDeleteFirstStudy(this.__page, this.__templateName);
250270
try {
271+
await auto.dashboardDeleteFirstStudy(this.__page, this.__templateName);
251272
await this.__responsesQueue.waitUntilResponse("projects/");
252273
}
253274
catch(err) {

0 commit comments

Comments
 (0)