Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: add missing assertion for a test #4701

Merged
merged 4 commits into from Jul 8, 2023
Merged

test: add missing assertion for a test #4701

merged 4 commits into from Jul 8, 2023

Conversation

kibertoad
Copy link
Member

Follow-up to #4698

Checklist

@kibertoad kibertoad requested a review from Eomm April 25, 2023 20:58
@github-actions github-actions bot added the test Issue or pr related to our testing infrastructure. label Apr 25, 2023
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

test/server.test.js Outdated Show resolved Hide resolved
@Fdawgs Fdawgs changed the title Add missing assertion for a test test: add missing assertion for a test Jul 2, 2023
@Uzlopak
Copy link
Contributor

Uzlopak commented Jul 8, 2023

@Eomm
PTAL

@Eomm Eomm merged commit 93000c6 into main Jul 8, 2023
23 checks passed
@Eomm Eomm deleted the fix/listentext-test branch July 8, 2023 11:30
renovate bot added a commit to tomacheese/telcheck that referenced this pull request Jul 17, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [fastify](https://www.fastify.io/)
([source](https://togithub.com/fastify/fastify)) | [`4.19.2` ->
`4.20.0`](https://renovatebot.com/diffs/npm/fastify/4.19.2/4.20.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.19.2/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.19.2/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify (fastify)</summary>

###
[`v4.20.0`](https://togithub.com/fastify/fastify/releases/tag/v4.20.0)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.19.2...v4.20.0)

#### What's Changed

- build(deps-dev): Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.28.20 to 0.29.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[fastify/fastify#4877
- Update Prototype-Poisoning.md by
[@&#8203;ed-henrique](https://togithub.com/ed-henrique) in
[fastify/fastify#4879
- docs: adjust line for linter by
[@&#8203;metcoder95](https://togithub.com/metcoder95) in
[fastify/fastify#4882
- Fixed type inference on .code().send() by
[@&#8203;aadito123](https://togithub.com/aadito123) in
[fastify/fastify#4880
- chore: refactor Error-typings and tests by
[@&#8203;svrnwnsch](https://togithub.com/svrnwnsch) in
[fastify/fastify#4824
- test: add missing assertion for a test by
[@&#8203;kibertoad](https://togithub.com/kibertoad) in
[fastify/fastify#4701
- chore: tests for genReqIdFactory and minor changes by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4783
- Added type definition for allowUnsafeRegex by
[@&#8203;xijdk](https://togithub.com/xijdk) in
[fastify/fastify#4792
- docs(security): update policy on ci/cd reports by
[@&#8203;jsumners](https://togithub.com/jsumners) in
[fastify/fastify#4890
- improve hooks.validate by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4804
- ci: enable caching; split jobs by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4889
- docs(plugins): note side-effect when using `await` in
`fastify.register()` by
[@&#8203;hateablestream](https://togithub.com/hateablestream) in
[fastify/fastify#4846
- \[hooks] Refine hook runners by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4805
- docs: fix wrong header size for FST_ERR_ASYNC_CONSTRAINT in Errors.md
by [@&#8203;graphiteisaac](https://togithub.com/graphiteisaac) in
[fastify/fastify#4893
- fix: rework FastifyErrors, ensure documentation completeness by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4892
- feat: add childLoggerFactory config by
[@&#8203;brettwillis](https://togithub.com/brettwillis) in
[fastify/fastify#4760
- Fix setNotFoundHandler handler type by
[@&#8203;Ethan-Arrowood](https://togithub.com/Ethan-Arrowood) in
[fastify/fastify#4897
- restore code coverage by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[fastify/fastify#4841
- fix: handle abort signal before server is ready by
[@&#8203;shayff](https://togithub.com/shayff) in
[fastify/fastify#4886
- docs(client-aborted): remove deprecated function by
[@&#8203;JekRock](https://togithub.com/JekRock) in
[fastify/fastify#4898
- Revert "docs(client-aborted): remove deprecated function
([#&#8203;4898](https://togithub.com/fastify/fastify/issues/4898))" by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[fastify/fastify#4901
- docs(logging): fix typo by
[@&#8203;yamanidev](https://togithub.com/yamanidev) in
[fastify/fastify#4905
- Support IPv6 ::1 in listeningOrigin by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[fastify/fastify#4902
- fix: extend isCustomValidatorCompiler from parent controller by
[@&#8203;tinchoz49](https://togithub.com/tinchoz49) in
[fastify/fastify#4903
- fix: allow uppercase requestdHeader by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4906

#### New Contributors

- [@&#8203;ed-henrique](https://togithub.com/ed-henrique) made their
first contribution in
[fastify/fastify#4879
- [@&#8203;svrnwnsch](https://togithub.com/svrnwnsch) made their first
contribution in
[fastify/fastify#4824
- [@&#8203;xijdk](https://togithub.com/xijdk) made their first
contribution in
[fastify/fastify#4792
- [@&#8203;hateablestream](https://togithub.com/hateablestream) made
their first contribution in
[fastify/fastify#4846
- [@&#8203;graphiteisaac](https://togithub.com/graphiteisaac) made their
first contribution in
[fastify/fastify#4893
- [@&#8203;shayff](https://togithub.com/shayff) made their first
contribution in
[fastify/fastify#4886
- [@&#8203;JekRock](https://togithub.com/JekRock) made their first
contribution in
[fastify/fastify#4898
- [@&#8203;yamanidev](https://togithub.com/yamanidev) made their first
contribution in
[fastify/fastify#4905
- [@&#8203;tinchoz49](https://togithub.com/tinchoz49) made their first
contribution in
[fastify/fastify#4903

**Full Changelog**:
fastify/fastify@v4.19.2...v4.20.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/tomacheese/telcheck).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi44LjExIiwidXBkYXRlZEluVmVyIjoiMzYuOC4xMSIsInRhcmdldEJyYW5jaCI6Im1hc3RlciJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
renovate bot added a commit to redwoodjs/redwood that referenced this pull request Jul 18, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [fastify](https://www.fastify.io/)
([source](https://togithub.com/fastify/fastify)) | [`4.19.2` ->
`4.20.0`](https://renovatebot.com/diffs/npm/fastify/4.19.2/4.20.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.19.2/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.19.2/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify (fastify)</summary>

###
[`v4.20.0`](https://togithub.com/fastify/fastify/releases/tag/v4.20.0)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.19.2...v4.20.0)

#### What's Changed

- build(deps-dev): Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.28.20 to 0.29.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[fastify/fastify#4877
- Update Prototype-Poisoning.md by
[@&#8203;ed-henrique](https://togithub.com/ed-henrique) in
[fastify/fastify#4879
- docs: adjust line for linter by
[@&#8203;metcoder95](https://togithub.com/metcoder95) in
[fastify/fastify#4882
- Fixed type inference on .code().send() by
[@&#8203;aadito123](https://togithub.com/aadito123) in
[fastify/fastify#4880
- chore: refactor Error-typings and tests by
[@&#8203;svrnwnsch](https://togithub.com/svrnwnsch) in
[fastify/fastify#4824
- test: add missing assertion for a test by
[@&#8203;kibertoad](https://togithub.com/kibertoad) in
[fastify/fastify#4701
- chore: tests for genReqIdFactory and minor changes by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4783
- Added type definition for allowUnsafeRegex by
[@&#8203;xijdk](https://togithub.com/xijdk) in
[fastify/fastify#4792
- docs(security): update policy on ci/cd reports by
[@&#8203;jsumners](https://togithub.com/jsumners) in
[fastify/fastify#4890
- improve hooks.validate by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4804
- ci: enable caching; split jobs by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4889
- docs(plugins): note side-effect when using `await` in
`fastify.register()` by
[@&#8203;hateablestream](https://togithub.com/hateablestream) in
[fastify/fastify#4846
- \[hooks] Refine hook runners by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4805
- docs: fix wrong header size for FST_ERR_ASYNC_CONSTRAINT in Errors.md
by [@&#8203;graphiteisaac](https://togithub.com/graphiteisaac) in
[fastify/fastify#4893
- fix: rework FastifyErrors, ensure documentation completeness by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4892
- feat: add childLoggerFactory config by
[@&#8203;brettwillis](https://togithub.com/brettwillis) in
[fastify/fastify#4760
- Fix setNotFoundHandler handler type by
[@&#8203;Ethan-Arrowood](https://togithub.com/Ethan-Arrowood) in
[fastify/fastify#4897
- restore code coverage by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[fastify/fastify#4841
- fix: handle abort signal before server is ready by
[@&#8203;shayff](https://togithub.com/shayff) in
[fastify/fastify#4886
- docs(client-aborted): remove deprecated function by
[@&#8203;JekRock](https://togithub.com/JekRock) in
[fastify/fastify#4898
- Revert "docs(client-aborted): remove deprecated function
([#&#8203;4898](https://togithub.com/fastify/fastify/issues/4898))" by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[fastify/fastify#4901
- docs(logging): fix typo by
[@&#8203;yamanidev](https://togithub.com/yamanidev) in
[fastify/fastify#4905
- Support IPv6 ::1 in listeningOrigin by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[fastify/fastify#4902
- fix: extend isCustomValidatorCompiler from parent controller by
[@&#8203;tinchoz49](https://togithub.com/tinchoz49) in
[fastify/fastify#4903
- fix: allow uppercase requestdHeader by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4906

#### New Contributors

- [@&#8203;ed-henrique](https://togithub.com/ed-henrique) made their
first contribution in
[fastify/fastify#4879
- [@&#8203;svrnwnsch](https://togithub.com/svrnwnsch) made their first
contribution in
[fastify/fastify#4824
- [@&#8203;xijdk](https://togithub.com/xijdk) made their first
contribution in
[fastify/fastify#4792
- [@&#8203;hateablestream](https://togithub.com/hateablestream) made
their first contribution in
[fastify/fastify#4846
- [@&#8203;graphiteisaac](https://togithub.com/graphiteisaac) made their
first contribution in
[fastify/fastify#4893
- [@&#8203;shayff](https://togithub.com/shayff) made their first
contribution in
[fastify/fastify#4886
- [@&#8203;JekRock](https://togithub.com/JekRock) made their first
contribution in
[fastify/fastify#4898
- [@&#8203;yamanidev](https://togithub.com/yamanidev) made their first
contribution in
[fastify/fastify#4905
- [@&#8203;tinchoz49](https://togithub.com/tinchoz49) made their first
contribution in
[fastify/fastify#4903

**Full Changelog**:
fastify/fastify@v4.19.2...v4.20.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/redwoodjs/redwood).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi44LjExIiwidXBkYXRlZEluVmVyIjoiMzYuOC4xMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
jtoar pushed a commit to redwoodjs/redwood that referenced this pull request Jul 19, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [fastify](https://www.fastify.io/)
([source](https://togithub.com/fastify/fastify)) | [`4.19.2` ->
`4.20.0`](https://renovatebot.com/diffs/npm/fastify/4.19.2/4.20.0) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/fastify/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/fastify/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/fastify/4.19.2/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/fastify/4.19.2/4.20.0?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>fastify/fastify (fastify)</summary>

###
[`v4.20.0`](https://togithub.com/fastify/fastify/releases/tag/v4.20.0)

[Compare
Source](https://togithub.com/fastify/fastify/compare/v4.19.2...v4.20.0)

#### What's Changed

- build(deps-dev): Bump
[@&#8203;sinclair/typebox](https://togithub.com/sinclair/typebox) from
0.28.20 to 0.29.1 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[fastify/fastify#4877
- Update Prototype-Poisoning.md by
[@&#8203;ed-henrique](https://togithub.com/ed-henrique) in
[fastify/fastify#4879
- docs: adjust line for linter by
[@&#8203;metcoder95](https://togithub.com/metcoder95) in
[fastify/fastify#4882
- Fixed type inference on .code().send() by
[@&#8203;aadito123](https://togithub.com/aadito123) in
[fastify/fastify#4880
- chore: refactor Error-typings and tests by
[@&#8203;svrnwnsch](https://togithub.com/svrnwnsch) in
[fastify/fastify#4824
- test: add missing assertion for a test by
[@&#8203;kibertoad](https://togithub.com/kibertoad) in
[fastify/fastify#4701
- chore: tests for genReqIdFactory and minor changes by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4783
- Added type definition for allowUnsafeRegex by
[@&#8203;xijdk](https://togithub.com/xijdk) in
[fastify/fastify#4792
- docs(security): update policy on ci/cd reports by
[@&#8203;jsumners](https://togithub.com/jsumners) in
[fastify/fastify#4890
- improve hooks.validate by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4804
- ci: enable caching; split jobs by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4889
- docs(plugins): note side-effect when using `await` in
`fastify.register()` by
[@&#8203;hateablestream](https://togithub.com/hateablestream) in
[fastify/fastify#4846
- \[hooks] Refine hook runners by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4805
- docs: fix wrong header size for FST_ERR_ASYNC_CONSTRAINT in Errors.md
by [@&#8203;graphiteisaac](https://togithub.com/graphiteisaac) in
[fastify/fastify#4893
- fix: rework FastifyErrors, ensure documentation completeness by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4892
- feat: add childLoggerFactory config by
[@&#8203;brettwillis](https://togithub.com/brettwillis) in
[fastify/fastify#4760
- Fix setNotFoundHandler handler type by
[@&#8203;Ethan-Arrowood](https://togithub.com/Ethan-Arrowood) in
[fastify/fastify#4897
- restore code coverage by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[fastify/fastify#4841
- fix: handle abort signal before server is ready by
[@&#8203;shayff](https://togithub.com/shayff) in
[fastify/fastify#4886
- docs(client-aborted): remove deprecated function by
[@&#8203;JekRock](https://togithub.com/JekRock) in
[fastify/fastify#4898
- Revert "docs(client-aborted): remove deprecated function
([#&#8203;4898](https://togithub.com/fastify/fastify/issues/4898))" by
[@&#8203;climba03003](https://togithub.com/climba03003) in
[fastify/fastify#4901
- docs(logging): fix typo by
[@&#8203;yamanidev](https://togithub.com/yamanidev) in
[fastify/fastify#4905
- Support IPv6 ::1 in listeningOrigin by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[fastify/fastify#4902
- fix: extend isCustomValidatorCompiler from parent controller by
[@&#8203;tinchoz49](https://togithub.com/tinchoz49) in
[fastify/fastify#4903
- fix: allow uppercase requestdHeader by
[@&#8203;Uzlopak](https://togithub.com/Uzlopak) in
[fastify/fastify#4906

#### New Contributors

- [@&#8203;ed-henrique](https://togithub.com/ed-henrique) made their
first contribution in
[fastify/fastify#4879
- [@&#8203;svrnwnsch](https://togithub.com/svrnwnsch) made their first
contribution in
[fastify/fastify#4824
- [@&#8203;xijdk](https://togithub.com/xijdk) made their first
contribution in
[fastify/fastify#4792
- [@&#8203;hateablestream](https://togithub.com/hateablestream) made
their first contribution in
[fastify/fastify#4846
- [@&#8203;graphiteisaac](https://togithub.com/graphiteisaac) made their
first contribution in
[fastify/fastify#4893
- [@&#8203;shayff](https://togithub.com/shayff) made their first
contribution in
[fastify/fastify#4886
- [@&#8203;JekRock](https://togithub.com/JekRock) made their first
contribution in
[fastify/fastify#4898
- [@&#8203;yamanidev](https://togithub.com/yamanidev) made their first
contribution in
[fastify/fastify#4905
- [@&#8203;tinchoz49](https://togithub.com/tinchoz49) made their first
contribution in
[fastify/fastify#4903

**Full Changelog**:
fastify/fastify@v4.19.2...v4.20.0

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/redwoodjs/redwood).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi44LjExIiwidXBkYXRlZEluVmVyIjoiMzYuOC4xMSIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Issue or pr related to our testing infrastructure.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants