fix(Authoring Tool): Move first lesson no longer breaks step numbering (

#1601)
WISE-Community · Jan 30, 2024 · f960e84 · f960e84
1 parent 8b0d13b
commit f960e84
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/src/app/services/teacherProjectService.spec.ts b/src/app/services/teacherProjectService.spec.ts
@@ -815,5 +815,11 @@ function removeNodeIdFromTransitions() {
         expect(service.getNodeById('group3').startId).toEqual('');
       });
     });
+    describe('remove the first lesson', () => {
+      it('changes the start id of the root group to be the second lesson', () => {
+        deleteNodeService.deleteNode('group1');
+        expect(service.getNodeById('group0').startId).toEqual('group2');
+      });
+    });
   });
 }
diff --git a/src/assets/wise5/services/teacherProjectService.ts b/src/assets/wise5/services/teacherProjectService.ts
@@ -1753,7 +1753,7 @@ export class TeacherProjectService extends ProjectService {
   private updateParentGroupStartId(nodeId: string): void {
     const parentGroup = this.getParentGroup(nodeId);
     if (parentGroup != null && parentGroup.startId === nodeId) {
-      const transitions = this.getTransitionsFromNode(nodeId);
+      const transitions = this.getTransitionsFromNode(this.getNodeById(nodeId));
       if (transitions.length > 0) {
         for (const transition of transitions) {
           const toNodeId = transition.to;