Skip to content

Implement a pause button in TypeScript and Jest project#5

Open
ryanmab wants to merge 4 commits intomainfrom
feature/pause-button-typescript-example
Open

Implement a pause button in TypeScript and Jest project#5
ryanmab wants to merge 4 commits intomainfrom
feature/pause-button-typescript-example

Conversation

@ryanmab
Copy link
Copy Markdown
Member

@ryanmab ryanmab commented Feb 27, 2024

Description

image

This is a simple PR which adds a Pause button to the TypeScript and Jest project.

@coverage-robot
Copy link
Copy Markdown

coverage-robot bot commented Feb 27, 2024

Coverage Report

Merging #5 will not change the total coverage (compared to 8ca0aa2)

Total Coverage Diff Coverage
72.88% 64.29%
Tags
Tag Lines Covered Partial Uncovered Coverage
typescript-jest 118 84 2 32 72.88%
Impacted Files
File Diff Coverage
examples/typescript-jest/src/App.tsx 0%
examples/typescript-jest/src/hooks/useTimer.ts 66.67%
examples/typescript-jest/src/components/PauseButton/PauseButton.tsx 100%
examples/typescript-jest/src/components/StatsCard/StatsCard.tsx 100%
examples/typescript-jest/src/hooks/useGameManager.ts 100%

Last update to a620a7e at 00:45am UTC

cols={cols}
onTilePressed={(row: number, col: number) =>
row === activeTile?.row && col === activeTile?.col && incrementScore()
!isPaused &&
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

None of these branches are covered by tests.

incrementScore()
}
activeTile={activeTile}
activeTile={!isPaused ? activeTile : undefined}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

None of these branches are covered by tests.

timeRemaining,
isPaused,
toggleTimer,
} = useGameManager(rows, cols, difficulty);
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This line is not covered by any tests.

@ryanmab ryanmab self-assigned this Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant