[Merged by Bors] - Handle __proto__ fields in object literals

[raskad]

This Pull Request changes the following:

• Handle __proto__ fields in object literals

[github-actions]

Test262 conformance changes

Test result	main count	PR count	difference
Total	93,815	93,815	0
Passed	69,528	69,540	+12
Ignored	18,422	18,422	0
Failed	5,865	5,853	-12
Panics	0	0	0
Conformance	74.11%	74.12%	+0.01%

Fixed tests (12):

test/language/expressions/object/__proto__-value-null.js [strict mode] (previously Failed)
test/language/expressions/object/__proto__-value-null.js (previously Failed)
test/language/expressions/object/__proto__-poisoned-object-prototype.js [strict mode] (previously Failed)
test/language/expressions/object/__proto__-poisoned-object-prototype.js (previously Failed)
test/language/expressions/object/__proto__-value-non-object.js [strict mode] (previously Failed)
test/language/expressions/object/__proto__-value-non-object.js (previously Failed)
test/language/expressions/object/__proto__-duplicate.js [strict mode] (previously Failed)
test/language/expressions/object/__proto__-duplicate.js (previously Failed)
test/language/expressions/object/__proto__-value-obj.js [strict mode] (previously Failed)
test/language/expressions/object/__proto__-value-obj.js (previously Failed)
test/language/expressions/object/__proto__-duplicate-computed.js [strict mode] (previously Failed)
test/language/expressions/object/__proto__-duplicate-computed.js (previously Failed)

[HalidOdat]

Looks good to me! :)

[RageKnify]

Just have the question about the comma, everything else looks good

[Razican]

Just a small nitpick. Mostly about documentation of the poco de. We should probably add more references to the spec in general in opcodes.

I'm happy to merge it as is too, though, great work :)

[Razican]

Here it says that this cannot fail per the spec, but then I'm missing some link to the spec, or the spec text that says so, right?

[raskad]

I will open is issue for this as it is currently a problem with all opcodes.

[codecov]

Codecov Report

Merging #2423 (4c5e93a) into main (2ffae5b) will increase coverage by 0.17%.
The diff coverage is 43.10%.

@@            Coverage Diff             @@
##             main    #2423      +/-   ##
==========================================
+ Coverage   38.69%   38.86%   +0.17%     
==========================================
  Files         314      316       +2     
  Lines       23887    24131     +244     
==========================================
+ Hits         9242     9378     +136     
- Misses      14645    14753     +108     

Impacted Files	Coverage Δ
boa_engine/src/bytecompiler/function.rs	45.33% <ø> (ø)
boa_engine/src/vm/code_block.rs	33.42% <ø> (ø)
boa_engine/src/vm/opcode/mod.rs	66.66% <ø> (ø)
boa_interner/src/sym.rs	25.00% <ø> (ø)
boa_parser/src/parser/mod.rs	36.84% <0.00%> (-1.00%)	⬇️
boa_engine/src/bytecompiler/mod.rs	29.39% <20.00%> (+0.17%)	⬆️
boa_parser/src/parser/cursor/mod.rs	38.38% <25.00%> (-0.57%)	⬇️
...arser/expression/primary/object_initializer/mod.rs	41.06% <39.13%> (-0.28%)	⬇️
boa_engine/src/context/mod.rs	34.78% <50.00%> (+2.31%)	⬆️
boa_engine/src/vm/opcode/set/prototype.rs	75.00% <75.00%> (ø)
... and 22 more

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

[raskad]

bors r+

[bors]

Pull request successfully merged into main.

Build succeeded:

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