Add integration test for discovering functions from a project repository. #5214
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
|
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
taeold
force-pushed
the
dl-cf3-discover-test
branch
from
7c666f6
to
7de7f09
Compare
Codecov ReportBase: 56.27% // Head: 56.27% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## dl-fn-discovery-cmd #5214 +/- ##
====================================================
Coverage 56.27% 56.27%
====================================================
Files 309 309
Lines 20838 20838
Branches 4227 4227
====================================================
Hits 11726 11726
Misses 8100 8100
Partials 1012 1012 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
taeold
force-pushed
the
dl-cf3-discover-test
branch
2 times, most recently
from
960f1b4
to
1d21dee
Compare
taeold
force-pushed
the
dl-fn-discovery-cmd
branch
from
0b62447
to
0d6179a
Compare
taeold
force-pushed
the
dl-cf3-discover-test
branch
from
1d21dee
to
ff2647b
Compare
TheIronDev
approved these changes
Nov 7, 2022
colerogers
approved these changes
Nov 7, 2022
taeold
force-pushed
the
dl-fn-discovery-cmd
branch
from
0d6179a
to
6ac68b8
Compare
taeold
force-pushed
the
dl-cf3-discover-test
branch
from
ff2647b
to
7d28a88
Compare
taeold
force-pushed
the
dl-fn-discovery-cmd
branch
from
6ac68b8
to
68889a4
Compare
taeold
force-pushed
the
dl-cf3-discover-test
branch
from
7d28a88
to
dc1163b
Compare
taeold
force-pushed
the
dl-fn-discovery-cmd
branch
from
68889a4
to
8144450
Compare
taeold
force-pushed
the
dl-cf3-discover-test
branch
2 times, most recently
from
51ddf7e
to
95a4810
Compare
taeold
force-pushed
the
dl-fn-discovery-cmd
branch
from
eecb3dc
to
15e23ca
Compare
taeold
force-pushed
the
dl-cf3-discover-test
branch
from
95a4810
to
0bdee8d
Compare
taeold
force-pushed
the
dl-cf3-discover-test
branch
from
0bdee8d
to
cce0388
Compare
|
Graphite Merge Job Current status: ✅ Merged This pull request was successfully merged as part of a stack. This comment was auto-generated by Graphite. Job Reference: Uhty8UE38l3MKEIRA9gp |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add new integration test for discovering function triggers in a project.
Here, we test couple of simple project setups:
simple: Plain ol' project withfuncntionsdirectoryesm: Likesimplebut functions are packages as an es module.codebases: Multiple functions spread across codebasesThe test can be run locally with
npm run test:functions-discover.