source-academy / js-slang / 5704727291

77%

push

github

Further improvements to environment visualizer (#1454)

* Env instruction no longer pushed onto an agenda as the last item

* Avoid unnecessary block statements

* Refactoring context.runtime.envsteps to an option paramter to be in line with the notion that the context shouldn't be updated by the frontend

* * Use the step limit option to allow users to limit the number of steps to the ec-evaluator
* Fixed bug with ec-evaluator: conditional statements now produce undefined if no value is produced since they are value producing.

* Add check for simple functions and bypass to return argument if so

* Add reset instruction only if there is no marker next

* Check for arguments in the application instruction before pushing environment instruction

* Avoid unnecessary break and continue markers

* Add support for rest syntax in ec-evaluator

---------

Co-authored-by: Dick Ong <dickongwd@gmail.com>

3593 of 4711 branches covered (76.27%)

Branch coverage included in aggregate %.

74 of 74 new or added lines in 2 files covered. (100.0%)

10699 of 12388 relevant lines covered (86.37%)

96239.79 hits per line