Bug 32248 - t/00-checkdatabase-version.t should be removed
Summary: t/00-checkdatabase-version.t should be removed
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: Architecture, internals, and plumbing (show other bugs)
Version: unspecified
Hardware: All All
: P5 - low normal (vote)
Assignee: Jonathan Druart
QA Contact: Testopia
URL:
Keywords:
Depends on: 25078
Blocks:
  Show dependency treegraph
 
Reported: 2022-11-17 13:52 UTC by Jonathan Druart
Modified: 2023-06-08 22:28 UTC (History)
1 user (show)

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


Attachments
Bug 32248: Remove t/00-checkdatabase-version.t (2.54 KB, patch)
2022-11-17 13:53 UTC, Jonathan Druart
Details | Diff | Splinter Review
Bug 32248: Remove t/00-checkdatabase-version.t (2.59 KB, patch)
2022-11-17 16:47 UTC, Tomás Cohen Arazi
Details | Diff | Splinter Review
Bug 32248: Remove t/00-checkdatabase-version.t (2.64 KB, patch)
2022-11-17 16:55 UTC, Katrin Fischer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Jonathan Druart 2022-11-17 13:52:16 UTC

    
Comment 1 Jonathan Druart 2022-11-17 13:53:34 UTC
Created attachment 144000 [details] [review]
Bug 32248: Remove t/00-checkdatabase-version.t

Since bug 25078 we do no longer use updatedatabase.pl. This test was
there to make sure the RM(aints) was replacing the XXX correctly.
Comment 2 Tomás Cohen Arazi 2022-11-17 16:47:05 UTC
Created attachment 144012 [details] [review]
Bug 32248: Remove t/00-checkdatabase-version.t

Since bug 25078 we do no longer use updatedatabase.pl. This test was
there to make sure the RM(aints) was replacing the XXX correctly.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 3 Katrin Fischer 2022-11-17 16:55:04 UTC
Created attachment 144013 [details] [review]
Bug 32248: Remove t/00-checkdatabase-version.t

Since bug 25078 we do no longer use updatedatabase.pl. This test was
there to make sure the RM(aints) was replacing the XXX correctly.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Tomás Cohen Arazi 2022-11-17 17:41:19 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!