Added unit tests for format

[ashishajr]

Description

Added unit tests for hurl::runner::filter::format

Related Issue

Add unit tests on hurl::runner::filter

[lepapareil]

/rebase

[hurl-bot]

🕗 /rebase is running, please wait for completion.

[hurl-bot]

🔨 Auto rebase from Orange-OpenSource/hurl/master succeeds, ashishajr/hurl/filter-format now embeds these commits:

• 07e2fc7 Disable Zizmor

[jcamiel]

Thanks for the PR @ashishajr . We should add another test unit when the format specifier is not valid for instance format "%%%". The code is panicking with this input so the new test should be marked with #[should_panic].

I'm creating a new bug issue so we'll fix this panic! and have a good error message.

[jcamiel]

Looks good thanks @ashishajr !

[ashishajr]

Thanks for all the help @jcamiel :)

[jcamiel]

/rebase

[hurl-bot]

🕗 /rebase is running, please wait for completion.

[hurl-bot]

🔨 Auto rebase from Orange-OpenSource/hurl/master succeeds, ashishajr/hurl/filter-format now embeds these commits:

• 0a6ee2f Added unit tests for decode
• cc30851 Update crates
• 198f72c Add variables to header_option integration test

[ashishajr]

@jcamiel, could you please hold off on accepting this PR, I made a mistake, the newest test is panicking for the wrong reason.

It's panicking because I'm calling unwrap on the Err instead of panicking because of invalid format. I'm incorrectly passing String instead of Date

fn eval_filter_format_ko_invalid_format() {
    let variables = VariableSet::new();

    let filter = new_format_filter("%%");
    eval_filter(
        &filter,
        &Value::String("01/01/2025".to_string()), // <- Should be Value::Date
        &variables,
        false,
    )
    .unwrap(); // <- Test is panicking here
}

My apologies, let me quickly fix this

[jcamiel]

No worries, good catch!

[ashishajr]

Should be fixed now!

[jcamiel]

/rebase

[hurl-bot]

🕗 /rebase is running, please wait for completion.

[hurl-bot]

🔨 Auto rebase from Orange-OpenSource/hurl/master succeeds, ashishajr/hurl/filter-format now embeds these commits:

• be6fbf9 Remove hex-literal crate dependency

[jcamiel]

/accept

[hurl-bot]

🕗 /accept is running, please wait for completion.

[hurl-bot]

✅ Pull request merged with fast forward by jcamiel..

# List of commits merged from ashishajr/hurl/filter-format branch into Orange-OpenSource/hurl/master branch:

• 96b80af Fixed incorrect panic unit test
• baddf60 Added panic unit test for format
• a9de5f6 Added unit tests for format