Seems there is a duplicate in test file

[talatkuyuk]

Initial checklist

• I read the support docs
• I read the contributing guide
• I agree to follow the code of conduct
• I searched issues and discussions and couldn’t find anything (or linked relevant results below)

Affected package

rehype-highlight@latest

Steps to reproduce

I realized that there is a duplicate in the test file.
Both have the same name should reprocess exacthere and here

I would open a PR, but couldn't decide which one should be deleted; but then, realized also that the content in the first test is a little different. It doesn't produce the exact code:

• <span class="hljs-built_in">console</span> becomes <span class="hljs-variable language_">console</span>
• .log(<span class="hljs-string">"Hello, "</span> becomes .<span class="hljs-title function_">log</span>(<span class="hljs-string">"Hello, "</span>

I couldn't understand whether the first one is designed intentionally (since the input and output are not the same exact) or not, may be a bug causes that.

Another issue in the tests is the last test here. When I delete the settings , {languages: {...common, test: testLang}}, the result doesn't change. Registration of a language should cause a change as far as I understand from the name of the test.

Actual behavior

Actually it is not a bug, but maybe... explained in the previous section

Expected behavior

Actually it is not a bug, but maybe...explained in the previous section

Runtime

node@20

Package manager

npm@11

Operating system

macos

Build and bundle tools

No response

[github-actions]

Hi! This was closed. Team: If this was fixed, please add phase/solved. Otherwise, please add one of the no/* labels.

[wooorm]

Hi! Good eyes! 👀

Looks like it got accidentally copied in at beed585. The earlier one is above that diff in the test file. I assume I copied the last existing test to make a new test for this feature, and was thinking of adding one more.

I guess hljs-built_in was the value then. Later, it became hljs-variable. But that wasn’t changed in the 1st of the duplicates.

[talatkuyuk]

Thank you. I am the type of person who pays attention to details. :)

What about for that:

Another issue in the tests is the last test here. When I delete the settings , {languages: {...common, test: testLang}}, the result doesn't change. Registration of a language should cause a change as far as I understand from the name of the test.

[wooorm]

It seems that was added in #8.
A test, that didn’t actually test, the new behavior. Thanks, fixed!