? PR-4.4-dev Pending

User tests: Successful: Unsuccessful:

avatar heelc29
heelc29
2 Aug 2023

Summary of Changes

API tests for following endpoints:

  • GET: v1/content/articles/:id
  • GET: v1/content/categories/:id
  • POST: v1/content/categories
  • PATCH: v1/content/categories/:id
  • GET: v1/fields/content/articles/:id
  • GET: v1/fields/content/categories
  • GET: v1/fields/content/categories/:id
  • POST: v1/fields/content/categories
  • PATCH: v1/fields/content/categories/:id
  • DELETE: v1/fields/content/categories/:id
  • GET: v1/fields/groups/content/articles/:id
  • GET: v1/fields/groups/content/categories
  • GET: v1/fields/groups/content/categories/:id
  • POST: v1/fields/groups/content/categories
  • PATCH: v1/fields/groups/content/categories/:id
  • DELETE: v1/fields/groups/content/categories/:id

missing:

asset:

  • DELETE: v1/content/categories/:id

history:

  • GET: v1/content/articles/:id/contenthistory
  • PATCH: v1/content/articles/:id/contenthistory/keep
  • DELETE: v1/content/articles/:id/contenthistory

Testing Instructions

code review; drone execute test

avatar joomla-cms-bot joomla-cms-bot - change - 2 Aug 2023
Category JavaScript Unit Tests
avatar heelc29 heelc29 - open - 2 Aug 2023
avatar heelc29 heelc29 - change - 2 Aug 2023
Status New Pending
avatar laoneo
laoneo - comment - 8 Aug 2023

Pr is looking good and drone is happy too, can we merge it?

avatar heelc29
heelc29 - comment - 8 Aug 2023

Some endpoints are missing ...

Delete category: fails because no asset item is created when insert category direct in database
History: fails because history is not created when insert category direct in database

Maybe an option is an combined test: create and delete category ...

If the pr is already ok then it will be ready to merge

avatar heelc29 heelc29 - change - 8 Aug 2023
Labels Added: ? PR-4.4-dev
avatar laoneo laoneo - change - 9 Aug 2023
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2023-08-09 10:17:15
Closed_By laoneo
avatar laoneo laoneo - close - 9 Aug 2023
avatar laoneo laoneo - merge - 9 Aug 2023
avatar laoneo
laoneo - comment - 9 Aug 2023

Thanks!

Add a Comment

Login with GitHub to post a comment