8681562984

Committed 14 Apr 2024 11:24PM UTC coverage: 81.988% (+0.02%) from 81.971%

Build # 8681562984

Build Type

push

github

Committed by

web-flow

Commit Message

Unify Runner Execution (#1672)

* Refactor more runner logic into SourceRunner

* Allow source to run from the desktop

* Run format

* Made it so that all runners need to go through sourceFilesRunner

* Use fetch for docs loading exclusively

* Run format

* Add new tests to make sure that runners load modules

* Make repl work with local imports and add tests

* Fix potential issue where repl could return an error

* bumping version

---------

Co-authored-by: Martin Henz <henz@comp.nus.edu.sg>

Run Details

3426 of 4524 branches covered (75.73%)

Branch coverage included in aggregate %.

148 of 166 new or added lines in 12 files covered. (89.16%)

4 existing lines in 2 files now uncovered.

10830 of 12864 relevant lines covered (84.19%)

128276.1 hits per line

Source File
Press 'n' to go to next uncovered line, 'b' for previous

76.47

/src/modules/utils.ts

import * as _ from 'lodash'

import type { RecursivePartial } from '../types'
import type { ImportOptions } from './moduleTypes'
import { defaultAnalysisOptions } from './preprocessor/analyzer'

const exportDefaultStr = 'export default'
export function removeExportDefault(text: string) {
  if (text.startsWith(exportDefaultStr)) {
    text = text.substring(exportDefaultStr.length).trim()
  }

  if (text.endsWith(';')) {
    text = text.slice(0, -1)
  }

  return text
}

export function mergeImportOptions(src?: RecursivePartial<ImportOptions>): ImportOptions {
  const baseOptions = _.cloneDeep(defaultAnalysisOptions)
  return _.merge(baseOptions, src as any)
}

/**
 * Checks if the given string refers to a Source module instead of a local module
 */
export const isSourceModule = (path: string) => !path.startsWith('.') && !path.startsWith('/')

1	import * as _ from 'lodash'	61✔
2
3	import type { RecursivePartial } from '../types'
4	import type { ImportOptions } from './moduleTypes'
5	import { defaultAnalysisOptions } from './preprocessor/analyzer'	61✔
6
7	const exportDefaultStr = 'export default'	61✔
8	export function removeExportDefault(text: string) {	61✔
9	if (text.startsWith(exportDefaultStr)) {	2✔
10	text = text.substring(exportDefaultStr.length).trim()	1✔
11	}
12
13	if (text.endsWith(';')) {	2!
14	text = text.slice(0, -1)	×
15	}
16
17	return text	2✔
18	}
19
20	export function mergeImportOptions(src?: RecursivePartial<ImportOptions>): ImportOptions {	61✔
UNCOV 21	const baseOptions = _.cloneDeep(defaultAnalysisOptions)	×
UNCOV 22	return _.merge(baseOptions, src as any)	×
23	}
24
25	/**
26	* Checks if the given string refers to a Source module instead of a local module
27	*/
28	export const isSourceModule = (path: string) => !path.startsWith('.') && !path.startsWith('/')	381✔

source-academy / js-slang / 8681562984

Source File Press 'n' to go to next uncovered line, 'b' for previous

Source File
Press 'n' to go to next uncovered line, 'b' for previous