Skip to content

Fix misleading error message for not-equal predicate#5073

Merged
hurl-bot merged 1 commit into
Orange-OpenSource:masterfrom
eyupcanakman:fix/not-equal-error-message-2841
Jun 9, 2026
Merged

Fix misleading error message for not-equal predicate#5073
hurl-bot merged 1 commit into
Orange-OpenSource:masterfrom
eyupcanakman:fix/not-equal-error-message-2841

Conversation

@eyupcanakman

Copy link
Copy Markdown
Contributor

A failing != assert printed the same value for actual and expected, so the message did not show what the response should have matched.

The expected line now shows not <value>, matching the output already used for not ==.

Fixes #2841

@jcamiel

jcamiel commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the PR @eyupcanakman we're looking into it asap

@fabricereix

Copy link
Copy Markdown
Collaborator

Thanks @eyupcanakman

@fabricereix

Copy link
Copy Markdown
Collaborator

/accept

@hurl-bot

hurl-bot commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

🕗 /accept is running, please wait for completion.

@hurl-bot

hurl-bot commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

🔨 Auto rebase from Orange-OpenSource/hurl/master succeeds, eyupcanakman/hurl/fix/not-equal-error-message-2841 now embeds these commits:

@hurl-bot

hurl-bot commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

🕗 /accept is still running, please wait for completion.

@hurl-bot hurl-bot force-pushed the fix/not-equal-error-message-2841 branch from 4bd2ac1 to 23ed61e Compare June 9, 2026 08:35
@hurl-bot

hurl-bot commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

✅ Pull request merged with fast forward by fabricereix..

# List of commits merged from eyupcanakman/hurl/fix/not-equal-error-message-2841 branch into Orange-OpenSource/hurl/master branch:

  • 23ed61e Fix misleading error message for not-equal predicate

@hurl-bot hurl-bot merged commit 23ed61e into Orange-OpenSource:master Jun 9, 2026
25 checks passed
@Thomasdezeeuw

Copy link
Copy Markdown

Thanks @eyupcanakman

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Misleading error message using != (not equal) operator

5 participants