fix(vm/opcode/push,environment,templates,generator,await,new): convert panics to EngineError::Panic using js_expect#5232
Conversation
8a12289 to
8be07af
Compare
Test262 conformance changes
Tested main commit: |
…t panics to EngineError::Panic using js_expect
8be07af to
0042a0b
Compare
|
To this pr i tried one new thing let Some(value) = some_option else {
return context.handle_error(PanicError::new("msg").into());
};This ensures panics are still converted to |
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #5232 +/- ##
===========================================
+ Coverage 47.24% 59.85% +12.60%
===========================================
Files 476 582 +106
Lines 46892 63478 +16586
===========================================
+ Hits 22154 37992 +15838
- Misses 24738 25486 +748 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Part of #3241.
Converts all convertible
.expect()panics to.js_expect()?in:opcode/push/array.rs(4)opcode/push/environment.rs(2)opcode/push/class/mod.rs(1)opcode/push/class/field.rs(8)opcode/push/class/private.rs(12)opcode/environment/mod.rs(12)opcode/templates/mod.rs(7)opcode/generator/mod.rs(6)opcode/generator/yield_stm.rs(2)opcode/await/mod.rs(5)opcode/new/mod.rs(2)Total: 61 panics converted.