Bug 26700

Summary: Remove unused C4/SIP/t directory
Product: Koha Reporter: David Cook <dcook>
Component: Architecture, internals, and plumbingAssignee: David Cook <dcook>
Status: RESOLVED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: minor    
Priority: P5 - low CC: jonathan.druart, kyle, m.de.rooy, pedro.amorim
Version: Main   
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15956
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9848
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9845
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9842
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=9838
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
Bug 26700: Remove unused C4/SIP/t directory
Bug 26700: Remove occurrences in Makefile.PL
Bug 26700: Remove unused C4/SIP/t directory
Bug 26700: Remove occurrences in Makefile.PL

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