From 847611aed49a76959f484537cbd6cd5f4a1362cf Mon Sep 17 00:00:00 2001 From: Simon Laux Date: Sat, 7 Jan 2023 23:37:54 +0100 Subject: [PATCH] improve error of node tests (#3928) remove unused imports and fail without revealing the token --- node/test/test.js | 20 ++++++++------------ 1 file changed, 8 insertions(+), 12 deletions(-) diff --git a/node/test/test.js b/node/test/test.js index 3d6c93de9..d76566b04 100644 --- a/node/test/test.js +++ b/node/test/test.js @@ -1,33 +1,29 @@ // @ts-check -import DeltaChat, { Message } from '../dist' -import binding from '../binding' +import DeltaChat from '../dist' -import { deepEqual, deepStrictEqual, strictEqual } from 'assert' +import { deepStrictEqual, strictEqual } from 'assert' import chai, { expect } from 'chai' import chaiAsPromised from 'chai-as-promised' import { EventId2EventName, C } from '../dist/constants' import { join } from 'path' -import { mkdtempSync, statSync } from 'fs' -import { tmpdir } from 'os' +import { statSync } from 'fs' import { Context } from '../dist/context' +import fetch from 'node-fetch' chai.use(chaiAsPromised) -chai.config.truncateThreshold = 0; // Do not truncate assertion errors. +chai.config.truncateThreshold = 0 // Do not truncate assertion errors. async function createTempUser(url) { - const fetch = require('node-fetch') - async function postData(url = '') { // Default options are marked with * const response = await fetch(url, { method: 'POST', // *GET, POST, PUT, DELETE, etc. - mode: 'cors', // no-cors, *cors, same-origin - cache: 'no-cache', // *default, no-cache, reload, force-cache, only-if-cached - credentials: 'same-origin', // include, *same-origin, omit headers: { 'cache-control': 'no-cache', }, - referrerPolicy: 'no-referrer', // no-referrer, *client }) + if (!response.ok) { + throw new Error('request failed: ' + response.body.read()) + } return response.json() // parses JSON response into native JavaScript objects }