website
website copied to clipboard
test: add tests for netlify functions
Description
- Added tests for netlify edge functions.
- Workflow added to verify the tests. Sample run can be found here
Related issue(s) Fixes #842
Deploy Preview for asyncapi-website ready!
Built without sensitive environment variables
| Name | Link |
|---|---|
| Latest commit | 88f0a72ad7eb12d289c81eb0ed8de841a40fd5af |
| Latest deploy log | https://app.netlify.com/sites/asyncapi-website/deploys/664e47bf4d2d62000814f65f |
| Deploy Preview | https://deploy-preview-2493--asyncapi-website.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
⚡️ Lighthouse report for the changes in this PR:
| Category | Score |
|---|---|
| 🔴 Performance | 42 |
| 🟢 Accessibility | 98 |
| 🟢 Best practices | 92 |
| 🟢 SEO | 100 |
| 🟠 PWA | 56 |
Lighthouse ran on https://deploy-preview-2493--asyncapi-website.netlify.app/
@Shurtu-gal In order to clarify the desired coverage of the test, I created the following flowchart explaining the flow the edge function should accomplish.
EDIT: please find it here.
Based on that, I believe we should have a test for each result of every decision (diamonds).
WDYT?
This pull request has been automatically marked as stale because it has not had recent activity :sleeping:
It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.
There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.
Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.
Thank you for your patience :heart:
@Shurtu-gal are you still up on this?
@Shurtu-gal are you still up on this?
Yeah, sorry this totally went out of my mind. Let me look at it again. Might need some time to get what I was doing though 🤣