f

Author: gregorriegler

webapp/src/RoleSheet.spec.tsx

@@ -57,6 +57,17 @@ describe('RoleSheet', () => {
  expect(mockScorePoints).toHaveBeenCalledWith("Talking", 2);
  })
 
+ it(`after scoring points todo checkboxes are reset`, async () => {
+ render(<RoleSheet role="Talking" position="Talking" player={new Player("Roger")} scorePoints={jest.fn()} />);
+ await checkTodo('Talking-0');
+ await checkTodo('Talking-1');
+
+ await clickEarnPoints();
+ 
+ expectUnchecked('Talking-0');
+ expectUnchecked('Talking-1');
+ })
+
  it(`earn button stays when no todos are marked`, async () => {
  const mockScorePoints = jest.fn();
  render(<RoleSheet role="Typing" position="Typing" player={new Player("Roger")} scorePoints={mockScorePoints} />);
@@ -101,6 +112,11 @@ async function clickEarnPoints() {
  });
 }
 
+function expectUnchecked(todoId) {
+ const checkbox = document.getElementById(todoId) as HTMLInputElement;
+ expect(checkbox.checked).toBe(false);
+}
+
 // TODO if canEarnPoints for role, show skills of role
 // cannot rotate until players have filled the form
 // TODO use it.each ??? but maybe its too clever?

webapp/src/RoleSheet.tsx

@@ -26,6 +26,7 @@ export function RoleSheet({
  if(amount > 0) {
  scorePoints(role, amount);
  setPointsScored(true);
+ checkboxes.forEach(checkbox => checkbox.checked = false);
  }
  }