Skip to content
This repository has been archived by the owner on Feb 12, 2025. It is now read-only.

Add unit tests #59

Open
smallketchup82 opened this issue May 26, 2024 · 2 comments
Open

Add unit tests #59

smallketchup82 opened this issue May 26, 2024 · 2 comments
Assignees
Labels
priority:high needs to be worked on asap type:enhancement New feature

Comments

@smallketchup82
Copy link
Member

Quoted from the discord:

i want to investigate creating unit tests to test certain aspects of KetchupBot. im not entirely sure as to how useful this will be, but at the very least it would help us check if ketchupbot properly formats, updates, merges, and sanitizes data. we could use jest and ts-jest for this

@smallketchup82 smallketchup82 self-assigned this May 26, 2024
@smallketchup82 smallketchup82 added the type:enhancement New feature label May 26, 2024
@smallketchup82 smallketchup82 changed the title Investigate making unit tests Add unit tests Aug 10, 2024
@smallketchup82
Copy link
Member Author

again, i'm still not too sure as to what i should necessarily do in unit tests, or what to test, for that matter

i guess i'll write some tests that investigate the end result of an update, and the individual components. idk

@smallketchup82 smallketchup82 added the priority:high needs to be worked on asap label Aug 10, 2024
@smallketchup82
Copy link
Member Author

i can see #102 being a prerequisite to this, but i can unironically access the variables even though ketchupbot-updater isn't a class library

i could get started on defining tests, but it might just be better to address #102 beforehand, to keep things structured

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority:high needs to be worked on asap type:enhancement New feature
Projects
Development

No branches or pull requests

1 participant