biblioverse/biblioteca

Can we remove kepubify installation from test suite?

Closed this issue · 4 comments

Can we remove kepubify installation from test suite?

You can, but then you need to remove the phpunit tests that uses Kepubify or mock the class so it doesn't have to call the kepubify binary.

@ragusa87 but the binary is already in the image, so why download it a second time?

Well, I would say that I added it in the pipeline as it's wasn't in the image at the time, or because the path was misconfigured. Or because the docker image was not used. I don't know.

I can indeed have a fresh look

I tried and realised that we are indeed not using the image. it's closed, thanks