refactor(TeacherDataService): Move functions to OneWorkgroupPerRowExport

[hirokiterashima]

Changes

The functions were only used in OneWorkgroupPerRowExport, and didn't need to be in TeacherDataService.

This pull request includes several changes to the OneWorkgroupPerRowDataExportStrategy class in the src/assets/wise5/classroomMonitor/dataExport/strategies/OneWorkgroupPerRowDataExportStrategy.ts file. The changes primarily focus on code formatting for better readability and the addition of a new method to retrieve the latest branch path taken event. Additionally, some redundant methods have been removed from the TeacherDataService class.

Improvements to code readability:

• Reformatted lines to improve readability by breaking long lines into multiple lines. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10]

Codebase simplification:

• Added a new method getLatestBranchPathTakenEvent to the OneWorkgroupPerRowDataExportStrategy class to encapsulate the logic for retrieving the latest branch path taken event.
• Removed the getLatestEventByWorkgroupIdAndNodeIdAndType and isEventMatchingNodeIdEventType methods from the TeacherDataService class as they are no longer needed.

Test

• OneWorkgroupPerRow export, with branch path taken.

[hirokiterashima]

🎉 This issue has been resolved in version 5.163.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀