source-academy / js-slang

Implementations of sublanguages of JavaScript, TypeScript, Scheme and Python

https://source-academy.github.io/source/

Apache License 2.0

66 stars 104 forks source link

CSE Machine: Changes to step count #1582

Closed NhatMinh0208 closed 6 months ago

NhatMinh0208 commented 6 months ago

Description

This PR makes some changes to how step count is updated, so that context.runtime.envStepsTotal represents the number of steps completed by the machine before the program finishes/runs into an error.

This is needed to implement source-academy/frontend#2847.

Changes

[X] context.runtime.envStepsTotal is updated during evaluation
[X] Step count now starts from 0 and represents the number of steps completed

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 8312839135

Details

4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.887%

Totals
Change from base Build 8229765431:	0.0%
Covered Lines:	10375
Relevant Lines:	12243