Add a npx pepr dev test around the WebhookConfig that is generated
#1723
Comments
This was referenced Feb 12, 2025
github-merge-queue bot
pushed a commit
that referenced
this issue
Feb 12, 2025
## Description We identified a testing gap that exists when a webhook is generated from an instance of the `Assets` object that has `Assets.host` defined. This test case addresses that gap. **NOTE**: There's some tight coupling with the `Assets` class and the use of a massive JSON string to create our test `assets` object in this file. Resolving those issue are outside the scope of this PR but do need to be addressed by future work. See #1816 and #1817 ## Related Issue Fixes #1723 ## Type of change - [ ] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [x] Other (security config, docs update, etc) ## Checklist before merging - [x] Unit, [Journey](https://github.com/defenseunicorns/pepr/tree/main/journey), [E2E Tests](https://github.com/defenseunicorns/pepr-excellent-examples), [docs](https://github.com/defenseunicorns/pepr/tree/main/docs), [adr](https://github.com/defenseunicorns/pepr/tree/main/adr) added or updated as needed - [x] [Contributor Guide Steps](https://docs.pepr.dev/main/contribute/#submitting-a-pull-request) followed
github-project-automation
bot
moved this from 📋 Backlog
to ✅ Done
in Pepr Project Board
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe what should be investigated or refactored
Ensure the webhooks generated in Pepr dev are pointing to the localhost k3d address.
The text was updated successfully, but these errors were encountered: