Implement isEmpty predicate #1202
Closed
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
Closed
fabricereix
reviewed
Jan 26, 2023
| expected: 0.to_string(), | ||
| type_mismatch: false, | ||
| }), | ||
| _ => Ok(AssertResult { |
There was a problem hiding this comment.
The count filter (https://github.com/Orange-OpenSource/hurl/blob/master/packages/hurl/src/runner/filter.rs#L117) can also take a Bytes or Nodeset value, we could therefore add them in the empty predicate.
|
@Reachip |
Format code Fix integration tests Fix useless use of format! Publish Hurl 2.0.0 to npm. Add get sum to chocolatey contrib section Fix hurl test on chocolatey contrib section Move from docker-hub to github-container-registry Update docker contrib doc Update winget contrib doc Update chocolatey contrib Update winget contrib doc Fix clippy 1.67 Fix servers log artifacts Update crates Update cast poster for README. Precise that --color does not affect HTTP response output. Update FAQ with curl/Hurl features interaction. Update crates Fix some Hurl samples newlines. Fix GraphQL query with variables to HTTP body request Update Rust sample with hurl/hurl_core 2.0.0 Update change log. Fix windows package artifacts Fix push create release commits on ci Update hurl version to 2.0.1-SNAPSHOT Create 2.0.1 release Update hurl version to 2.1.0-SNAPSHOT Add Date value/filters Fix docker contrib readme Update installation with 2.0.1 links. Update installation.md when releasing Update npm publish scripts for 2.0.1 Fix https://www.hurl.dev to https://hurl.dev Add README for Homebrew publication. Replace google.fr by hurl.dev Update crates Update Hurl snippet in FAQ. Update crates Delete Progress and parse Hurl file before execution. Reorganize Hurl output. Fix variables-file example Update crates Finalize cleaning main structure. Use Date for cookie Expires query Fix focal mirror repo for ci Use long options for curl command Update crates Update home with star links. Update crates Remove dead code. Fix Markdown links. Fix visibility of module html. Update crates Add progress bar for tests. Make jsonpath module private. Clean HTTP client fields. Delete `redirect_count`. Make `handle private. Simplify execute HTTP client method. Implements Default trait for RequestSpec. Fix bad renaming. Update crates Add certificate query to grammar. Add Certificate model for HTTP response. Add Default impl for http::Response Add associated function to create a Header. Update crates Fix report module visibility. Fix import name for path utilities. isEmpty can take Bytes and Nodeset value Update bytes.py
|
I'm closing this PR we'll reopen a new PR once the implementation is done. |
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 PR implement the "isEmpty" predicate.
Today, we are able to say:
We would like to be able to write:
This predicate could work for collections or strings.