feat: Draft implementation to store and expose integration errors

This commit is contained in:
Faruk AYDIN
2022-09-19 12:13:36 +03:00
parent d0bcf997fb
commit 8826125845
3 changed files with 47 additions and 21 deletions

View File

@@ -19,7 +19,13 @@ export default class SearchTweets {
};
let response;
const tweets: IJSONObject[] = [];
const tweets: {
data: IJSONObject[];
error: IJSONObject | null;
} = {
data: [],
error: null,
};
do {
const params: IJSONObject = {
@@ -47,10 +53,15 @@ export default class SearchTweets {
headers: { ...authHeader },
});
if (response.automatischError) {
tweets.error = response.automatischError;
return tweets;
}
if (response.data.meta.result_count > 0) {
response.data.data.forEach((tweet: IJSONObject) => {
if (!lastInternalId || Number(tweet.id) > Number(lastInternalId)) {
tweets.push(tweet);
tweets.data.push(tweet);
} else {
return;
}
@@ -58,16 +69,6 @@ export default class SearchTweets {
}
} while (response.data.meta.next_token && lastInternalId);
if (response.data?.errors) {
const errorMessages = response.data.errors
.map((error: IJSONObject) => error.detail)
.join(' ');
throw new Error(
`Error occured while fetching user data: ${errorMessages}`
);
}
return tweets;
}
}

View File

@@ -11,7 +11,12 @@ export default class HttpClient {
}
async get(path: string, options?: IJSONObject) {
return await this.instance.get(path, options);
try {
return await this.instance.get(path, options);
} catch (error) {
error.response.automatischError = error.response.data;
return error.response;
}
}
async post(path: string, body: IJSONObject | string, options?: IJSONObject) {

View File

@@ -33,9 +33,9 @@ class Processor {
const triggerStep = steps.find((step) => step.type === 'trigger');
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
let initialTriggerData = await this.getInitialTriggerData(triggerStep!);
const initialTriggerData = await this.getInitialTriggerData(triggerStep!);
if (initialTriggerData.length === 0) {
if (!initialTriggerData.error && initialTriggerData.data.length === 0) {
const lastInternalId = await this.flow.lastInternalId();
const executionData: Partial<Execution> = {
@@ -52,12 +52,12 @@ class Processor {
return;
}
if (this.testRun) {
initialTriggerData = [initialTriggerData[0]];
if (this.testRun && initialTriggerData.data.length > 0) {
initialTriggerData.data = [initialTriggerData.data[0]];
}
if (initialTriggerData.length > 1) {
initialTriggerData = initialTriggerData.sort(
if (initialTriggerData.data.length > 1) {
initialTriggerData.data = initialTriggerData.data.sort(
(item: IJSONObject, nextItem: IJSONObject) => {
return (item.id as number) - (nextItem.id as number);
}
@@ -66,7 +66,7 @@ class Processor {
const executions: Execution[] = [];
for await (const data of initialTriggerData) {
for await (const data of initialTriggerData.data) {
const execution = await Execution.query().insert({
flowId: this.flow.id,
testRun: this.testRun,
@@ -118,6 +118,22 @@ class Processor {
}
}
if (initialTriggerData.error) {
const executionWithError = await Execution.query().insert({
flowId: this.flow.id,
testRun: this.testRun,
});
executions.push(executionWithError);
await executionWithError.$relatedQuery('executionSteps').insertAndFetch({
stepId: triggerStep.id,
status: 'failure',
dataIn: triggerStep.parameters,
errorDetails: initialTriggerData.error,
});
}
if (!this.testRun) return;
const lastExecutionStepFromFirstExecution = await executions[0]
@@ -125,7 +141,11 @@ class Processor {
.orderBy('created_at', 'desc')
.first();
return lastExecutionStepFromFirstExecution?.dataOut;
if (lastExecutionStepFromFirstExecution.errorDetails) {
return lastExecutionStepFromFirstExecution.errorDetails;
} else {
return lastExecutionStepFromFirstExecution?.dataOut;
}
}
async getInitialTriggerData(step: Step) {