mark/stroopwafel
4
2
Fork 0
Code Issues 8 Pull Requests Releases Activity

WIP: Unit tests #19

Closed
mark wants to merge 0 commits from unittests into master
pull from: unittests
merge into: mark:master
Conversation 1 Commits - Files Changed -
+78 -7
mark commented 2022-11-03 23:32:10 +01:00
Owner
Copy Link
Copy Source

Plan van aanpak:

  1. Refactor bestaande code zodanig dat elk command een aparte functie gebruikt die iets returnt voordat het terug wordt gestuurd.
  2. Gebruik unittests om te kijken of die functie nog wel het juiste returnt.

In theorie eenvoudig, in de praktijk toch lastiger. Uiteindelijk moet er ook een hoop gebeuren op het gebied van de bot zelf, en de onderliggende discordpy classes. Ook moet duplicate code zo veel mogelijk worden ontweken, dus is het eigenlijk ideaal als we het testen ook echt met een bot doen (Stroopwafel Test) bijvoorbeeld.

Deze PR houdt de voortgang bij van de branch.

Fixes #18

Plan van aanpak: 1. Refactor bestaande code zodanig dat elk command een aparte functie gebruikt die *iets* returnt voordat het terug wordt gestuurd. 2. Gebruik unittests om te kijken of die functie nog wel het juiste returnt. In theorie eenvoudig, in de praktijk toch lastiger. Uiteindelijk moet er ook een hoop gebeuren op het gebied van de bot zelf, en de onderliggende discordpy classes. Ook moet duplicate code zo veel mogelijk worden ontweken, dus is het eigenlijk ideaal als we het testen ook echt met een bot doen (Stroopwafel Test) bijvoorbeeld. Deze PR houdt de voortgang bij van de branch. Fixes #18
mark added the enhancement label 2022-11-03 23:32:10 +01:00
mark self-assigned this 2022-11-03 23:32:10 +01:00
mark added 1 commit 2022-11-03 23:32:11 +01:00
eerste opzet testje 3c8d34acdd
mark added 1 commit 2022-11-04 23:25:41 +01:00
refactors en echte tests ofzo f89889fc48
mark added 2 commits 2022-11-13 20:00:57 +01:00
adds requirements for pip 67bb071305
adds drone config
continuous-integration/drone/push Build is failing
Details
continuous-integration/drone/pr Build is failing
Details
7e9e88eee7 
nog wel de vraag hoe het gaat zonder ini en hoe we die er in krijgen,
maar dat moeten we maar even zien
mark added 1 commit 2022-11-13 20:02:12 +01:00
alleen builden op push, niet op PR
continuous-integration/drone/push Build is failing
Details
fc0d59964d
mark commented 2022-11-14 16:30:59 +01:00
Author
Owner
Copy Link
Copy Source

branch is verpest omdat ik iets doms heb gedaan met git, komt een vervangend PR voor

branch is verpest omdat ik iets doms heb gedaan met git, komt een vervangend PR voor
mark closed this pull request 2022-11-14 16:30:59 +01:00
Some checks are pending
continuous-integration/drone/push Build is failing
Details

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label enhancement
Milestone
No items
No Milestone
Assignees
Clear assignees
guus (Guus) jp lourens martijn mark (Mark Hoekveen)
No Assignees mark
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: mark/stroopwafel#19
Delete Branch "unittests"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?