3482 Add support for relative IDs in the config content

[Nic-Ma]

Task step 12 of #3482 .

Description

According to the task plan, this PR added support for relative reference ID, like: "test": "@###data#1".

Status

Ready

Types of changes

• Non-breaking change (fix or new feature that would not break existing functionality).
• Breaking change (fix or new feature that would cause existing functionality to change).
• New tests added to cover the changes.
• Integration tests passed locally by running ./runtests.sh -f -u --net --coverage.
• Quick tests passed locally by running ./runtests.sh --quick --unittests --disttests.
• In-line docstrings updated.
• Documentation updated, tested make html command in the docs/ folder.

[Nic-Ma]

/black

[Nic-Ma]

/build

[Nic-Ma]

/build

[wyli]

I think this makes it difficult to distinguish the empty id "" and the parent reference?

[Nic-Ma]

I think this makes it difficult to distinguish the empty id "" and the parent reference?

Hi @wyli ,

The regex in this PR is re.compile(rf"{ref}{sep}+"), which only match the relative IDs at beginning of the string.

Thanks.

[Nic-Ma]

/black

[Nic-Ma]

/build

[Nic-Ma]

/build

[Nic-Ma]

/black

[Nic-Ma]

/build

[Nic-Ma]

Hi @wyli @ericspod ,

Thanks for your review and suggestions.
I revisited the feature, actually, the relative id conversion is kind of "macro replacement" and we need to support it for both the macro token %XXX case and reference @XXX case. So I changed the logic to ConfigParser and no need to call it twice now.
Could you please help review it again?

Thanks in advance.