Skip to content

London | 26-ITP-Sep | Hanna Bohlin | Sprint 2 | Practice TDD - #1720

Open
habohlin wants to merge 44 commits into
CodeYourFuture:mainfrom
habohlin:Practice-TDD
Open

London | 26-ITP-Sep | Hanna Bohlin | Sprint 2 | Practice TDD#1720
habohlin wants to merge 44 commits into
CodeYourFuture:mainfrom
habohlin:Practice-TDD

Conversation

@habohlin

@habohlin habohlin commented Oct 16, 2026

Copy link
Copy Markdown

Learners, PR Template

Self checklist

  • I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
  • My changes meet the requirements of the task
  • I have tested my changes
  • My changes follow the style guide

Task code

CYF-1060

Changelist

Completed the three exercises in Practice TDD with one commit for each added test and one commit for each refactor.

habohlin added 30 commits October 14, 2026 22:59
@habohlin
Add test with different string length, it fails
38dbbd8
@habohlin
Refactor countChar to make prev test pass
6b5df27
@habohlin
Add test for varying chars in string, it fails
ffcf1f9
@habohlin
Refactor countChar to make prev test pass
9f80255
@habohlin
Add test for any char that exists in string, it fails
b7fcec6
@habohlin
Refactor countChar to make prev test pass
626f314
@habohlin
Add test for char NOT in string, returns 0, test passes already
b184661
@habohlin
Add test for count is 1, str is 'hello', it fails
e59481e
@habohlin
Refactor repeatStr to pass prev test
ce8ef88
@habohlin
Add test for count is 0, str is 'hello', should return empty string, …
3da27c9 
…test fails
@habohlin
Refactor repeatStr to pass prev test
ece5aaf
@habohlin
Add test for count is 4, test fails
41c6f58
@habohlin
Refactor repeatStr to be dynamic, and pass prev test
c4ed549
@habohlin
Add test for count is negative, should throw error, test fails
dbb63f0
@habohlin
Refactor repeatString to throw error and refactor the test, it passes
d8bc6e0
@habohlin
Add test for dynamic string value, it fails
43522a0
@habohlin
Refactor repeatStr to use dynamic string in output, prev test now passes
54e7f2e
@habohlin
Refactor function to pass the failing tests
5739486
@habohlin
Add tests for numbers ending with 11, they fail
6721f24
@habohlin
Refactor function to pass prev tests
582352f
@habohlin
Add tests for number ending 2 but not 12, they fail
ed6c46c
@habohlin
Refactor function to pass prev tests
8ad33fc
@habohlin
Add tests for numbers ending with 12, they fail
b21915a
@habohlin
Refactor function to pass prev tests
4ef1b43
@habohlin
Add tests for numbers ending 3 but not 13, they fail
8d41b97
@habohlin
Refactor function to pass prev tests
719b29d
@habohlin
Add tests for numbers ending 13, they fail
c4fa2c6
@habohlin
Refactor function to pass prev tests
5c15ff4
@habohlin
Add tests for all other whole numbers, they fail
7b78cdd
@habohlin
Refactor function to pass prev tests
eae160f
habohlin added 14 commits October 16, 2026 11:07
@habohlin
Add test for throwing error when decimal number passed. Test fails
c032f5b
@habohlin
Refactor function to pass prev test
8614766
@habohlin
Add test for missing argument, it passes on the same premise as the l…
8c54303 
…ast test
@habohlin
Add test for a passed string, it also passes thanks to the same check
74f5c33
@habohlin
Add test for undefined, it passes as well due to the same reason
23e9048
@habohlin
Add test for num is null to throw error, it fails
803ce36
@habohlin
Refactor function to pass prev test
5a8d70f
@habohlin
Add test for num is false should throw error, it passes
ceaec78
@habohlin
Add test for num is true to throw error, it fails
63998f9
@habohlin
Refactor function to pass prev test
0aa534c
@habohlin
Add test for num is array to throw error, it fails
1724b9a
@habohlin
Refactor function to pass prev test
330bd08
@habohlin
Add test for num is object to throw error, it passes
9574101
@habohlin
Format files with prettier
c939e57
@habohlin habohlin added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Oct 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@habohlin