[Merged by Bors] - Fix String.prototype.replace() order of ToString execution

[HalidOdat]

Fix order of ToString execution, there are 3 failing tests on .replace() but they seem to be regex related

[github-actions]

Test262 conformance changes

Test result	main count	PR count	difference
Total	94,343	94,343	0
Passed	71,163	71,167	+4
Ignored	17,386	17,386	0
Failed	5,794	5,790	-4
Panics	0	0	0
Conformance	75.43%	75.43%	+0.00%

Fixed tests (4):

test/built-ins/String/prototype/replace/replaceValue-evaluation-order-regexp-object.js [strict mode] (previously Failed)
test/built-ins/String/prototype/replace/replaceValue-evaluation-order-regexp-object.js (previously Failed)
test/built-ins/String/prototype/replace/replaceValue-evaluation-order.js [strict mode] (previously Failed)
test/built-ins/String/prototype/replace/replaceValue-evaluation-order.js (previously Failed)

[codecov]

Codecov Report

Merging #2799 (6597f07) into main (34d6b93) will decrease coverage by 0.01%.
The diff coverage is 70.96%.

@@            Coverage Diff             @@
##             main    #2799      +/-   ##
==========================================
- Coverage   50.57%   50.56%   -0.01%     
==========================================
  Files         415      415              
  Lines       41207    41206       -1     
==========================================
- Hits        20841    20837       -4     
- Misses      20366    20369       +3     

Impacted Files	Coverage Δ
boa_engine/src/builtins/string/mod.rs	62.98% <70.96%> (-0.40%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[raskad]

Nice fix and also nice optimization with the enum.

[jedel1043]

Good work!

[Razican]

Thanks!

[Razican]

bors r+

[bors]

Pull request successfully merged into main.

Build succeeded:

• Build and Test (macos-latest)
• Build and Test (windows-latest)
• Coverage
• Misc