https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=40173 --- Comment #7 from David Cook <dcook@prosentient.com.au> --- (In reply to Jonathan Druart from comment #6)
I am actually adding test to http-client.js indirectly here, via t/cypress/integration/t/api-client.ts ;)
Indirectly being the key word there.
The more we are going to use it in Cypress tests, the more it is going to be covered by tests.
But not really. It's the difference between a unit test and an end to end test. Despite the word "integration" in "t/cypress/integration/t/api-client.ts", it's really an end to end test, and it's just covering the end effects. It's like saying we don't need Perl unit tests because we have the Cypress end to end tests. -- But that's fine. It's the status quo. I think that we could do better than we are currently, but it doesn't need to be right now. It's just an opportunity to try something better. But the current patches are still a good move forward. -- You are receiving this mail because: You are watching all bug changes.