Skip to content

Commit cda9e9f

Browse files
authored
fix(tests): the full-chain script asserts two endpoints that no longer exist (#3423)
OpenCatalogi retired its CMS (opencatalogi#1422). The `page` and `menu` schemas, both controllers and their 8 routes went with it, because pages and menus are Portaliq's now. This script was one of the four consumers that removal was confirmed against, and it was the one left standing: `step_verify_opencatalogi` called `/api/menus` and `/api/pages/home` and logged an error when they did not answer. They answer 404 by design, so the step reported two failures on a healthy instance. The checks are removed rather than repointed. This step verifies OPENCATALOGI's configuration; asserting against Portaliq's `/api/content/{pages,menus}` here would fail for reasons this script cannot explain, and Portaliq's own suite owns that surface. The publication-register and settings-import checks either side are untouched. `bash -n` clean, and all 75 applicable hydra gates pass. A repo-wide sweep for `page_schema`, `menu_schema` and the two route prefixes finds nothing else.
1 parent 97a0784 commit cda9e9f

1 file changed

Lines changed: 12 additions & 18 deletions

File tree

‎tests/full-chain-test.sh‎

Lines changed: 12 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -181,24 +181,18 @@ step_verify_opencatalogi() {
181181
echo "$import_result"
182182
fi
183183

184-
# Check menus exist
185-
local menus=$(api_call GET "/index.php/apps/opencatalogi/api/menus?_limit=100")
186-
local menu_count=$(echo "$menus" | grep -o '"total":[0-9]*' | grep -o '[0-9]*')
187-
188-
if [ -n "$menu_count" ] && [ "$menu_count" -gt 0 ]; then
189-
log_success "OpenCatalogi menus loaded: $menu_count menus found"
190-
else
191-
log_error "OpenCatalogi menus not loaded"
192-
fi
193-
194-
# Check pages exist
195-
local home_page=$(api_call GET "/index.php/apps/opencatalogi/api/pages/home")
196-
197-
if echo "$home_page" | grep -q '"title"'; then
198-
log_success "OpenCatalogi home page exists"
199-
else
200-
log_error "OpenCatalogi home page not found"
201-
fi
184+
# The menu and page checks that stood here are GONE, not moved.
185+
#
186+
# OpenCatalogi retired its CMS (opencatalogi#1422): the `page` and `menu`
187+
# schemas, both controllers and their 8 routes left with it, and pages and
188+
# menus are Portaliq's now. This script was one of the four consumers that
189+
# removal was confirmed against, so it asserted two endpoints that answer
190+
# 404 by design.
191+
#
192+
# Not repointed at Portaliq's /api/content/{pages,menus} either: this step
193+
# verifies OPENCATALOGI's configuration, and an assertion about a different
194+
# app's content API here would fail for reasons this script cannot explain.
195+
# Portaliq's own suite owns that.
202196

203197
# Check publication register exists
204198
local registers=$(api_call GET "/index.php/apps/openregister/api/registers")

0 commit comments

Comments
 (0)