source-academy / frontend / 5219065445

40%

Build Type

push

github

Committed by web-flow

Commit Message

Refactor state handlers memoization (#2492)

* Replace props with propsRef.current

As propsRef.current is memoized and mutable, this eliminates the need
for complex dependency arrays.

* Use useTypedSelector for some state variables

* Fix tests

* Fix formatting

* Revert "Use useTypedSelector for some state variables"

This reverts commit a52bf9415.

* Revert "Replace props with propsRef.current"

This reverts commit 153a9b38c.

* Revert "Fix tests"

This reverts commit e2c7fc788.

* Simplify Playground handlers memoization

* Use named imports for React hooks in Playground

* Update test snapshots

* Simplify Sourcecast handlers memoization

* Simplify GitHubAssessmentWorkspace handlers memoization

* Remove completed TODO comment

* Use Role enum instead of string in route check

* Replace unnecessary template literals with strings.

* Remove unnecessary fragment

* Make tags self-closing where applicable

Improves code cleanliness.

* Move constants out of component

This prevents them from being unnecessarily recreated on rerenders.

* Remove more unnecessary fragments

Run Details

2014 of 6194 branches covered (32.52%)

Branch coverage included in aggregate %.

47 of 98 new or added lines in 5 files covered. (47.96%)

1 existing line in 1 file now uncovered.

5349 of 13243 relevant lines covered (40.39%)

17.96 hits per line