Skip to content

Commit 606e60e

Browse files
dependabot[bot]matz3
authored andcommitted
deps: Bump body-parser from 1.20.3 to 2.2.2
Bumps [body-parser](https://github.com/expressjs/body-parser) from 1.20.3 to 2.2.2. - [Release notes](https://github.com/expressjs/body-parser/releases) - [Changelog](https://github.com/expressjs/body-parser/blob/master/HISTORY.md) - [Commits](expressjs/body-parser@1.20.3...v2.2.2) --- updated-dependencies: - dependency-name: body-parser dependency-version: 2.2.2 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
1 parent 262f7e7 commit 606e60e

File tree

4 files changed

+136
-101
lines changed

4 files changed

+136
-101
lines changed

package-lock.json

Lines changed: 98 additions & 99 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

packages/middleware-code-coverage/lib/middleware.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ export default async function({log, middlewareUtil, options={}, resources}) {
6868
bodyParser.json({type: "application/json", limit: "50mb"}),
6969
async (req, res) => {
7070
const reportData = await reportCoverage(
71-
req.body,
71+
req.body ?? {},
7272
config,
7373
resources,
7474
log

packages/middleware-code-coverage/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@
3333
"node": "^20.11.0 || >=22.0.0"
3434
},
3535
"dependencies": {
36-
"body-parser": "^1.20.3",
36+
"body-parser": "^2.2.2",
3737
"istanbul-lib-coverage": "^3.2.2",
3838
"istanbul-lib-instrument": "^6.0.3",
3939
"istanbul-lib-report": "^3.0.1",

packages/middleware-code-coverage/test/unit/lib/middleware.js

Lines changed: 36 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -170,6 +170,42 @@ test("Coverage report request: no report data", async (t) => {
170170
});
171171
});
172172

173+
test("Coverage report request: no body", async (t) => {
174+
const reportCoverageStub = sinon.stub();
175+
const log = sinon.stub();
176+
const instrumenterMiddleware = await esmock("../../../lib/middleware.js", {
177+
"../../../lib/coverage-reporter.js": reportCoverageStub.returns(undefined)
178+
});
179+
const middleware = await instrumenterMiddleware({log, resources});
180+
181+
t.plan(2);
182+
183+
await new Promise((resolve) => {
184+
const res = {
185+
json() {
186+
t.fail("should not be called.");
187+
resolve();
188+
},
189+
err(message) {
190+
t.is(reportCoverageStub.callCount, 1);
191+
t.is(message, "No report data provided");
192+
resolve();
193+
}
194+
};
195+
const next = () => {
196+
t.fail("should not be called.");
197+
resolve();
198+
};
199+
middleware({
200+
method: "POST",
201+
url: "/.ui5/coverage/report",
202+
headers: {
203+
type: "application/json"
204+
},
205+
}, res, next);
206+
});
207+
});
208+
173209
test("Consume Coverage report request", async (t) => {
174210
const serveStaticStub = sinon.stub();
175211
const instrumenterMiddleware = await esmock.p("../../../lib/middleware.js", {

0 commit comments

Comments
 (0)