source-academy / js-slang / 1323

77%

push

travis-ci

Change scoping rules for code execution (#269)

* Change scoping rules for code execution

First time code is executed:

const math_PI = 3.14;
const map = <some builtin>;
.
.
.
{
  // user code start
  const con = 1;
  let le = 2;
  // blah blah
  <store con back>
  <store le>
}

Second time code is executed:

const math_PI = 3.14;
const map = <some builtin>;
.
.
.
{
  const con = <retrieve result>
  let le = <retrieve result>
  {
    // another user code start
    const con = 2; // can declare const again here!
    // blah blah
  }
}

* Format

* Change interpreter to match new scoping rules for execution

* Replace outdated test cases

* Update test cases

* Format and revert tslint.json change

544 of 616 branches covered (88.31%)

Branch coverage included in aggregate %.

47 of 47 new or added lines in 4 files covered. (100.0%)

1709 of 1844 relevant lines covered (92.68%)

324923.09 hits per line