Logging for the default assert library #217
Open
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
Replaces `arr.map()` with a for-loop to avoid a "Max call stack size exceeded" error for fixtures with *many* lines to dedent. Closes lukeed#115
* docs: added information for assert.match * docs: added information for assert.not.match * docs: added more information per requested change
* chore: add esbuild example * chore: remove `.gitignore` file Co-authored-by: Luke Edwards <luke.edwards05@gmail.com>
- Similar to "typescript" example, but uses `"type": "module"` and source files only include ".js" files with ESM syntax. This matches the scenario(s) presented in lukeed#123 Also includes two approaches for solving this problem; see npm "scripts" section. Closes lukeed#123
* chore: add "solidjs" example * chore: lock devdependency versions Co-authored-by: Luke Edwards <luke.edwards05@gmail.com>
- drops / inlines totalist dependency - Closes lukeed#111
* Update api.assert.md Adding a reference on the idiom of how to assert an async function throws an error: lukeed#35 (comment) * Apply suggestions from code review * Update docs/api.assert.md Co-authored-by: Luke Edwards <luke.edwards05@gmail.com>
* Fix missing assertion error on array type mismatch * chore: add mirror test * nit: code style Co-authored-by: Luke Edwards <luke.edwards05@gmail.com>
- do not build `uvu/parse` from source since it needs different contents, like uvu/run - rely on `require` directly for loading file paths (revert 4cedf40)
- ensures `uvu` initializes with shared ENV/global context
* Detect BigInt within circular() and properly convert them within stringify * narrower bigint check * update test expectants * fix: avoid bigint syntax errors in Node8 Co-authored-by: Luke Edwards <luke.edwards05@gmail.com>
…ises (lukeed#207) * A test that exits early should fail Fixes lukeed#206 Also fixes the case where a test calls process.exit(0) itself, as that's essentially the same thing. * Apply suggestions from code review * wrap exit listener; merge test files Co-authored-by: Luke Edwards <luke.edwards05@gmail.com>
You can choose what to do with it later too.
|
Last commit I ran prettier with the .editorconfig settings, because I was copy pasting stuff fast, that's where some of the changes come from. |
Closed
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.
This adds toggleable logging for the default library.
TODO