New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add initial testing suit #53

Merged

AriaMoradi merged 5 commits into Suwayomi:master from Syer10:testing_suit

Apr 3, 2021

Collaborator

Syer10 commented Apr 3, 2021 •

edited

Loading

It goes over all sources, installs/updates/uninstalls extensions when it first starts up
Queries popular manga for each source(a total of 3 times if it errors)
Takes the first manga and queries manga info(a total of 3 times if it errors)
Takes the manga and queries for chapters(a total of 3 times if it errors)
And then lastly queries for the page list(a total of 3 times if it errors)

Additional changes:

~~It changes SourceDataClass.id to a long value(because it should be one, shouldnt have to convert it every use)~~
Implement coroutine testing suit
Reorginize applicationSetup function so that it allows us to change the root dir
Instead of using GlobalConfigManager use KodeinDI for ServerConfig, allows for easy swapping for tests

Notes: It creates files based on the tests results, so you can see what sources/manga info/chapters/page lists, failed to get

Syer10 added 5 commits

April 3, 2021 16:42


          Add initial testing suit

c0df7d3


          Revert source id to long

a211a41


          dirs -> applicationDirs

b2d5354


          Use properties to set rootDir so that ConfigManager can use it

e043cb5


          Allow rootdir to be used as a argument

c11887f

AriaMoradi merged commit 975a3b1 into Suwayomi:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet