Joomla! Issue Tracker - CMS

Download
Launch
Feeling Lucky

[#44861] - [5.2][system tests] Do not delete tuf metadata

Unit/System Tests PR-5.2-dev Pending

User tests: Successful: Unsuccessful:

avatar heelc29
heelc29
11 Feb 2025

Pull Request for Issue #44859 .

Summary of Changes

Do not delete tuf_metadata in afterEach() cleanupDB

Testing Instructions

  • run specs tests/System/integration/administrator/components/com_joomlaupdate/Update.cy.js
  • check content of tuf_metadata table

Actual result BEFORE applying this Pull Request

table is empty

Expected result AFTER applying this Pull Request

one entry with id=1

Link to documentations

Please select:

  • No documentation changes for docs.joomla.org needed
  • No documentation changes for manual.joomla.org needed
avatar heelc29 heelc29 - open - 11 Feb 2025
avatar heelc29 heelc29 - change - 11 Feb 2025
Status New ⇒ Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Feb 2025
Category ⇒ JavaScript Unit Tests
avatar heelc29 heelc29 - change - 11 Feb 2025
Labels Added: Unit/System Tests PR-5.2-dev
avatar laoneo
laoneo - comment - 12 Feb 2025

Would it not be better to do an update in db_setInvalidTufRoot or db_setValidTufRoot? I would like to try to avoid to not add another dependency in the deleteInsertedItems function.

avatar SniperSister
SniperSister - comment - 12 Feb 2025

See #44866 for an alternate approach following the suggestion of @laoneo

avatar laoneo laoneo - close - 12 Feb 2025
avatar laoneo
laoneo - comment - 12 Feb 2025

Closing it in favor of #44866.

avatar laoneo laoneo - change - 12 Feb 2025
Status Pending ⇒ Closed
Closed_Date 0000-00-00 00:00:00 ⇒ 2025-02-12 08:03:22
Closed_By ⇒ laoneo

Add a Comment

Login with GitHub to post a comment