Bug 26700 - Remove unused C4/SIP/t directory
Summary: Remove unused C4/SIP/t directory
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: master
Hardware: All All
: P5 - low minor (vote)
Assignee: David Cook
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-10-16 00:22 UTC by David Cook
Modified: 2023-07-18 10:51 UTC (History)
4 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00,23.05.02


Attachments
Bug 26700: Remove unused C4/SIP/t directory (59.50 KB, patch)
2023-06-02 02:19 UTC, David Cook
Details | Diff | Splinter Review
Bug 26700: Remove unused C4/SIP/t directory (59.60 KB, patch)
2023-06-02 05:24 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 26700: Remove occurrences in Makefile.PL (983 bytes, patch)
2023-06-02 05:24 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 26700: Remove unused C4/SIP/t directory (59.67 KB, patch)
2023-06-13 14:25 UTC, Marcel de Rooy
Details | Diff | Splinter Review
Bug 26700: Remove occurrences in Makefile.PL (1.05 KB, patch)
2023-06-13 14:25 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description David Cook 2020-10-16 00:22:28 UTC
It doesn't seem like C4/SIP/t is used, so we may as well remove it.
Comment 1 David Cook 2020-10-16 00:25:21 UTC
It actually looks like there are quite a few tests there... maybe they should be moved into the t/SIP directory...
Comment 2 Jonathan Druart 2021-01-07 14:26:49 UTC
From bug 9848 comment 9

"""
 They arnt a test suite they are a reminder that one needs to be written
"""
Comment 3 David Cook 2021-01-07 22:24:27 UTC
(In reply to Jonathan Druart from comment #2)
> From bug 9848 comment 9
> 
> """
>  They arnt a test suite they are a reminder that one needs to be written
> """

But then it seems like Adrien was running them at https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9848#c0? 

I wonder if there is anyone at BibLibre who can speak to that...
Comment 4 David Cook 2023-06-02 02:19:27 UTC
Created attachment 151960 [details] [review]
Bug 26700: Remove unused C4/SIP/t directory

This patch removes the unused C4/SIP/t directory.
Comment 5 Jonathan Druart 2023-06-02 05:24:51 UTC
Created attachment 151965 [details] [review]
Bug 26700: Remove unused C4/SIP/t directory

This patch removes the unused C4/SIP/t directory.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Comment 6 Jonathan Druart 2023-06-02 05:24:53 UTC
Created attachment 151966 [details] [review]
Bug 26700: Remove occurrences in Makefile.PL

As well as other 2 non existing paths
Comment 7 Marcel de Rooy 2023-06-02 07:29:19 UTC
(In reply to David Cook from comment #1)
> It actually looks like there are quite a few tests there... maybe they
> should be moved into the t/SIP directory...

You just remove, not move. Did you check if there are some useful tests that we do  not want to loose here? And they probably need quite a bit of editing too if we want to move them..

How did you verify that they are not being used?

@Kyle: Are you aware of people using these tests ?
Comment 8 David Cook 2023-06-04 23:23:39 UTC
(In reply to Marcel de Rooy from comment #7)
> You just remove, not move. Did you check if there are some useful tests that
> we do  not want to loose here? And they probably need quite a bit of editing
> too if we want to move them..
> 
> How did you verify that they are not being used? 

I raised this ticket and linked to existing tickets over 2 years ago, and no one ever replied. 

These tests aren't run by the official Koha test suite. 

If anyone uses them in an unofficial capacity, I'm sure they'll raise a ticket to say so after they're removed. (If they're running them, they're probably also someone with the capacity to do the work of editing them and moving them into the t/ or xt/ directories.)

As Jonathan noted in 2021, Colin also commented in 2015 that he couldn't determine that anyone ran these tests or that anyone felt confident in working on them.

Since we have the git history and it wouldn't affect production installations, I think we can remove these tests, and then we'll really be able to verify if anyone uses them.
Comment 9 Kyle M Hall 2023-06-05 13:19:50 UTC
(In reply to Marcel de Rooy from comment #7)
> (In reply to David Cook from comment #1)
> > It actually looks like there are quite a few tests there... maybe they
> > should be moved into the t/SIP directory...
> 
> You just remove, not move. Did you check if there are some useful tests that
> we do  not want to loose here? And they probably need quite a bit of editing
> too if we want to move them..
> 
> How did you verify that they are not being used?
> 
> @Kyle: Are you aware of people using these tests ?

I am not aware of anyone using these tests. We don't run them and avoid them due to their flakiness. I always choose to add unit tests to /t and avoid these ones altogether.
Comment 10 Marcel de Rooy 2023-06-13 14:25:48 UTC
Created attachment 152295 [details] [review]
Bug 26700: Remove unused C4/SIP/t directory

This patch removes the unused C4/SIP/t directory.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 11 Marcel de Rooy 2023-06-13 14:25:50 UTC
Created attachment 152296 [details] [review]
Bug 26700: Remove occurrences in Makefile.PL

As well as other 2 non existing paths

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 12 Marcel de Rooy 2023-06-13 14:27:42 UTC
(In reply to David Cook from comment #8)
> 
> Since we have the git history and it wouldn't affect production
> installations, I think we can remove these tests, and then we'll really be
> able to verify if anyone uses them.

A few weeks later, we are all ready to go :)
Comment 13 Tomás Cohen Arazi 2023-06-14 12:24:25 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 14 Martin Renvoize 2023-07-17 11:13:38 UTC
Thanks for all the hard work!

Pushed to 23.05.x for the next release
Comment 15 Pedro Amorim 2023-07-18 10:51:59 UTC
Not pushing to 22.11.x