Skip to content

Commit 62e37ba

Browse files
committed
test: add comprehensive tests for stdio discovery
- Add test fixture for broken syntax error handling - Refactor tests to use unified DiscoveryResult interface - Parameterize tests with nested loops to test all cases with both discovery methods - Add error handling tests for both HTTP and stdio discovery - All stdio discovery tests passing (16/16)
1 parent e0d6028 commit 62e37ba

File tree

4 files changed

+199
-155
lines changed

4 files changed

+199
-155
lines changed
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
const functions = require("firebase-functions");
2+
3+
// This will cause a syntax error
4+
exports.broken = functions.https.onRequest((request, response) => {
5+
response.send("Hello from Firebase!"
6+
}); // Missing closing parenthesis

scripts/bin-test/sources/broken-syntax/package-lock.json

Lines changed: 81 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
{
2+
"name": "broken-syntax",
3+
"main": "index.js",
4+
"dependencies": {
5+
"firebase-functions": "file:../../../.."
6+
}
7+
}

0 commit comments

Comments
 (0)