Testing/CI #21

Merged

jp merged 40 commits from testing into master 2022-12-11 16:27:22 +01:00

Conversation 2 Commits 40 Files Changed 16

+883 -527

mark commented 2022-11-14 16:31:28 +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 7 commits 2022-11-14 16:31:28 +01:00

eerste opzet testje 3c8d34acdd

refactors en echte tests ofzo f89889fc48

adds requirements for pip 67bb071305

adds drone config ... 7e9e88eee7

nog wel de vraag hoe het gaat zonder ini en hoe we die er in krijgen,
maar dat moeten we maar even zien

alleen builden op push, niet op PR fc0d59964d

fix domme bug waar we niet voor testen da7262307d

vervangt ini file met dotenv ... 2c69b13f3d

Maakt het makkelijker om te testen, en is sowieso betere manier denk ik

mark added 1 commit 2022-11-16 12:58:31 +01:00

get secrets from drone b0901cb2e9

mark added 1 commit 2022-12-02 19:36:09 +01:00

Expected failure for server down ... 3ef817c6aa

Was even een les om te leren. Nu checkt hij eerst voor een 5xx response.

mark added 1 commit 2022-12-02 21:08:13 +01:00

tests voor aww 54b3dfd290

mark added 1 commit 2022-12-02 21:50:38 +01:00

Test voor kopieerpasta 009d64ef16

mark added 1 commit 2022-12-04 15:10:54 +01:00

Verwijder gif functie ... 074545f965

Als het het niet doet en we het niet testen, dan moet het weg.

mark added 4 commits 2022-12-04 19:25:51 +01:00

Verwijder reboot functie ... b376191cd8

is alleen maar gewoon gezeik, en restarten kan ook vanuit de server

Verwijder tex ... b08933b9f1

Gebruikt niemand, en vergt 10 gigabyte installs om lokaal te testen

Test voor huge emoji ... f3dfca14c2

Test alleen de parsing van de emoji en dat er een embed wordt gemaakt.

Verwijdert enkele requirements ... 4b5e4cf407

Onder andere youtubedl en ffmpeg voor de gif, en de latex sympy.
Deze changes zijn ook doorgevoerd in de readme.

mark added 1 commit 2022-12-04 19:33:10 +01:00

Moves tests to module 47ae62afa9

mark self-assigned this 2022-12-04 20:17:09 +01:00

mark added 3 commits 2022-12-04 21:17:23 +01:00

Voegt test voor review toe e8df92be23

Adds test watspelenwe ... 7c29180ca4

Test het martijn en het mark gedeelte los.

Maakt een klein beetje schoon hier en daar ... 39ec30a048

Oude comments, verdwaalde warnings, ongebruikte imports

mark added 1 commit 2022-12-04 23:31:07 +01:00

Fixes error on missing black and white txt 19d71e3b4c

mark added 1 commit 2022-12-05 02:12:23 +01:00

ktdm test ... a9de2ab753

En refactor, want hoe het stond was niet heel testbaar

mark added 4 commits 2022-12-05 14:54:34 +01:00

Adds Dockerfile for easy testing ... fc532086bb

Moet up-to-date worden gehouden met drone. Maar emuleert een pull van de
repo met een .env file erin.

Haalt sympy uit de requirements ... 1cfd3d0dec

Was alleen nodig voor de LaTeX functie, en die is geschrapt

Refactor weer ... 1361d9c083

Omwille van het zo beter te kunnen maken om voor te kunnen testen.

Adds tests for weer 088d86f4dd

mark added 3 commits 2022-12-06 01:40:36 +01:00

more efficient caching of images ba5958fb77

Refactors roll ... 3760d8cb02

En flink ook. Nu met regex en meer pythonic one-liners die niet of
nauwelijks leesbaar zijn. Eindeloos plezier.

Tests roll ... a14d446e82

En vrij grondig ook nog, al zeg ik het zelf

mark added 3 commits 2022-12-07 22:04:30 +01:00

byebye relevantxkcd. gbnf 5ad7512fa8

refactor imgflip calls e1b18661cb

tests imgflips f89418c587

mark added 2 commits 2022-12-07 22:19:31 +01:00

test inspire 90e0dc5bab

Tests giet het al oan ... 09477f3c4d

Vooral om de testcount te padden lmao

mark added 2 commits 2022-12-07 22:37:40 +01:00

Tests corona ... 8395a0b4a6

Beetje slappe test wel, maar je moet iets.

Tests zalgo ... 055f170f62

Soort van. Kijkt iig of de conversie misschien goed gaat.

mark requested review from jp 2022-12-07 22:40:03 +01:00

mark commented 2022-12-08 00:28:18 +01:00

Author

Owner

Copy Link

Copy Source

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.

Dit is op zichzelf deels gebeurd. Vaak blijft het bij kijken of iets wel een string teruggeeft of kijken of bepaalde input wel/niet geaccepteerd wordt. Als zodanig zijn de tests dus niet heel grondig of dekkend, maar zouden ze wel een redelijke indicatie moeten geven van of bepaalde features nog een beetje werken of niet.

Bovendien is van het echt testen met discord modellen niet heel veel terecht gekomen. Ik ben er niet helemaal achter hoe ik daar effectief mocks voor maak en hoe ik op een zinnige manier echt de discord-integratie kan testen.

Maar op zich is ons doel niet de sterren...

> 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. Dit is op zichzelf deels gebeurd. Vaak blijft het bij kijken of iets wel een string teruggeeft of kijken of bepaalde input wel/niet geaccepteerd wordt. Als zodanig zijn de tests dus niet heel grondig of dekkend, maar zouden ze wel een redelijke indicatie moeten geven van of bepaalde features nog een beetje werken of niet. Bovendien is van het echt testen met discord modellen niet heel veel terecht gekomen. Ik ben er niet helemaal achter hoe ik daar effectief mocks voor maak en hoe ik op een zinnige manier echt de discord-integratie kan testen. Maar op zich is ons doel niet de sterren...

mark added 1 commit 2022-12-10 21:21:14 +01:00

example env file 9119ac3171

jp added 1 commit 2022-12-10 22:26:19 +01:00

eerste test voor chatbot c70bb1284f

jp added 2 commits 2022-12-11 16:19:11 +01:00

chatbot and points unittests toegevoegd aan init c4ba63b83b

refactor Points class + tests 25e38b6b5f

jp commented 2022-12-11 16:27:07 +01:00

Collaborator

Copy Link

Copy Source

Commits van collega Mark zien er goed uit.

Ik heb één hele simpele test toegevoegd voor Chatbot.py, maar dat is meer zodat die class ook maar wordt meegenomen. Het chat algoritme unittesten wordt lastig.

Verder heb ik Points.py gerefactord zodat het iets beter te unittesten is. Verdient alsnog geen schoonheidsprijs, maar ik denk dat het eventuele toekomstige aanpassingen wel iets makkelijker maakt.

Commits van collega Mark zien er goed uit. Ik heb één hele simpele test toegevoegd voor Chatbot.py, maar dat is meer zodat die class ook maar wordt meegenomen. Het chat algoritme unittesten wordt lastig. Verder heb ik Points.py gerefactord zodat het iets beter te unittesten is. Verdient alsnog geen schoonheidsprijs, maar ik denk dat het eventuele toekomstige aanpassingen wel iets makkelijker maakt.

jp merged commit 9effd126dd into master 2022-12-11 16:27:22 +01:00

jp referenced this issue from a commit 2022-12-11 16:27:23 +01:00

Merge pull request 'Testing/CI' (#21) from testing into master

Sign in to join this conversation.

Reviewers

No Reviewers

jp

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

Milestone

No items

No Milestone

Assignees

Clear assignees

guus (Guus) jp lourens martijn mark (Mark Hoekveen)

No Assignees mark

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: mark/stroopwafel#21