macros: fix invalid file paths

[senekor]

This regression was introduced in:
88c13fb

[no important files changed]

Co-authored-by: yjhtry [email protected]

[github-actions]

This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested.

If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos.

[no important files changed]

For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping @exercism/maintainers-admin in a comment. Thank you!

[senekor]

based on #2041

I made sure I could actually get some of the invalid tests to fail with an incorrect solution:

test output:

test single ... ok
test no_trailing_comma ... ok
test nested ... ok
test test::macro_out_of_scope ... ok
test empty ... ok
test test::type_not_in_scope ... ok
test trailing_comma ... ok
test type_override ... ok
test compile_fails_comma_sep ... ok
test compile_fails_two_arrows ... ok
test compile_fails_only_comma ... ok
test compile_fails_triple_arguments ... ok
test compile_fails_leading_comma ... ok
test compile_fails_missing_argument ... ok
test compile_fails_single_argument ... ok
test compile_fails_double_commas ... FAILED
test compile_fails_no_comma ... ok
test compile_fails_only_arrow ... ok

source of test double_commas:

fn main() {
    // a single trailing comma is okay, but two is not
    let _hm: HashMap<_, _> = hashmap!('a' => 2, ,);
}

incorrect solution that triggers the failure:

#[macro_export]
macro_rules! hashmap {
    ($($key:expr => $value:expr,)+) => { $crate::hashmap!($($key => $value),+) };
    ($($key:expr => $value:expr),*) => {
        {
            let mut _map = ::std::collections::HashMap::new();
            $(
                _map.insert($key, $value);
            )*
            _map
        }
    };
    ($key:expr => $value:expr , , ) => {
        {
            let mut _map = ::std::collections::HashMap::new();
            _map.insert($key, $value);
            _map
        }
    };
}

[senekor]

@ellnix did you see this? The tests are currently failing no matter what, would be nice to get this merged.

[ellnix]

@senekor I did see it but I didn't know it was that urgent, I'll look at it right now.

[ellnix]

LGTM! This indeed fails for the right reasons, correct values are passed to --bin. I can't see any problems.