Skip to content

added tests for JsSymbol#4733

Open
psg-19 wants to merge 2 commits intoboa-dev:mainfrom
psg-19:JsSymbol_tests
Open

added tests for JsSymbol#4733
psg-19 wants to merge 2 commits intoboa-dev:mainfrom
psg-19:JsSymbol_tests

Conversation

@psg-19
Copy link

@psg-19 psg-19 commented Feb 26, 2026

This Pull Request fixes/closes #3489 .

Summary:-
This PR adds comprehensive test coverage for JsSymbol.

  • Rust-level tests (symbol.rs) — new(), description(),fn_name(), descriptive_string().
  • Clone, Display, Debug, PartialEq, Ord and Hash trait impls
  • JS-level tests (tests.rs) - Symbol.for() / Symbol.keyFor(), Symbol.prototype.description getter, valueOf(), toString(), @@toPrimitive, new Symbol() throws TypeError

@psg-19 psg-19 requested a review from a team as a code owner February 26, 2026 06:58
@github-actions
Copy link

github-actions bot commented Feb 26, 2026

Test262 conformance changes

Test result main count PR count difference
Total 52,862 52,862 0
Passed 49,504 49,504 0
Ignored 2,262 2,262 0
Failed 1,096 1,096 0
Panics 0 0 0
Conformance 93.65% 93.65% 0.00%

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.

Add tests for JsSymbol

1 participant