coveralls
commented
2 months ago Pull Request Test Coverage Report for Build 8612816890
Details
- 123 of 139 (88.49%) changed or added relevant lines in 18 files are covered.
- 7 unchanged lines in 4 files lost coverage.
- Overall coverage increased (+0.06%) to 35.32%
| Changes Missing Coverage | Covered Lines | Changed/Added Lines | % | ||
|---|---|---|---|---|---|
| src/commons/utils/JsSlangHelper.ts | 0 | 1 | 0.0% | ||
| src/features/cseMachine/components/values/PrimitiveValue.tsx | 6 | 7 | 85.71% | ||
| src/features/cseMachine/CseMachineUtils.ts | 31 | 36 | 86.11% | ||
| src/features/cseMachine/CseMachineLayout.tsx | 30 | 39 | 76.92% | ||
| <!-- | Total: | 123 | 139 | 88.49% | --> |
| Files with Coverage Reduction | New Missed Lines | % | ||
|---|---|---|---|---|
| src/features/cseMachine/CseMachineLayout.tsx | 1 | 63.71% | ||
| src/features/cseMachine/components/arrows/ArrowFromFn.tsx | 1 | 71.43% | ||
| src/features/cseMachine/CseMachineUtils.ts | 1 | 70.23% | ||
| src/features/cseMachine/components/Binding.tsx | 4 | 82.35% | ||
| <!-- | Total: | 7 | --> |
| Totals | |
|---|---|
| Change from base Build 8611712543: | 0.06% |
| Covered Lines: | 5194 |
| Relevant Lines: | 13742 |
Description
See: https://github.com/source-academy/js-slang/pull/1645 Fixes source-academy/js-slang#1641
Changes
functionNamefrom a closure will result in a global functionstreamnow have a special typeStreamFnenvironment, and they have their implementation hiddenapply_in_underlying_javascriptnow works correctly with the environmentSourceObject, and should be displayed as a string, e.g.<Rune>, similar to how the REPL and the stepper displays it