refactor(StudentDataService, AnnotationService, TeacherDataService): simplify state retrieval logic using Array.findLast()

[hirokiterashima]

Changes

This pull request includes various changes to improve the efficiency and readability of the code in the studentDataService and teacherDataService files. The main changes involve replacing loops with the findLast method and removing unnecessary dependencies and methods.

Codebase simplification and efficiency improvements:

• src/assets/wise5/services/annotationService.ts: Simplified the getLatestAnnotation method by replacing the loop with the findLast method.
• src/assets/wise5/services/studentDataService.ts: Simplified the getLatestComponentStateByNodeIdAndComponentId, getLatestSubmitComponentState, and getLatestNodeEnteredEventNodeIdWithExistingNode methods by replacing loops with the findLast method. [1] [2]
• src/assets/wise5/services/teacherDataService.ts: Simplified the getLatestComponentStateByWorkgroupIdNodeIdAndComponentId and getLatestComponentStateByWorkgroupIdNodeId methods by replacing loops with the findLast method. [1] [2]

Test cleanup:

• src/app/services/studentDataService.spec.ts: Removed the MatDialogModule import and $rootScope variable, and cleaned up related test cases by removing unnecessary $broadcast spy and expectations. [1] [2] [3] [4]

Test

• VLE and CM work as before, especially code that handles loading component states

[hirokiterashima]

🎉 This issue has been resolved in version 5.163.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀