Skip to content

test: cover the Tag admin screen - #63

Merged
marevol merged 1 commit into
mainfrom
feat/tagtype
Oct 5, 2026
Merged

marevol merged 1 commit into
mainfrom
feat/tagtype

Conversation

@marevol

@marevol marevol commented Oct 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds a tagtype module covering the Tag admin screen (/admin/tagtype/) that codelibs/fess#3551 adds for per-user tags in Fess 15.9. The module is modelled on the label tests.

  • add: creates a tag with name, owner, paths, permissions and sort order, and checks the list and the details.
  • list/search: searches by name and by owner.
  • validation: rejects a blank name, a blank owner, a whitespace-only name and a name over the default 50-character limit.
  • update: changes the paths and the owner, and checks the details afterwards.
  • delete: deletes the tag and waits until the list no longer shows it.

The module runs right after label in the default order and is skipped on Fess versions older than 15.9.

Testing

  • The suite's unit tests pass (119), all files compile, and the label/message keys exist in all bundles.
  • I ran it against a Fess built from the #3551 code with OpenSearch 3, in an isolated compose project:
    • label and tagtype pass in English, and tagtype passes in Japanese.
    • As a negative control, a deliberately wrong expectation made the run fail.

Until a Fess snapshot image that includes #3551 is published, tagtype will fail against the current snapshot image.

Add a tagtype module for /admin/tagtype/: add (paths, virtual host, sort
order, owner-default permission), list search by name and owner with
reset, validation (blank name, blank owner, whitespace-only name, a name
of ideographic spaces and an over-long name rejected by the tag-name
check, duplicate name and owner), update with an owner change that moves
the owner permission and the id, and delete with a converging absence
check. The module is skipped on Fess older than 15.9.
@marevol marevol self-assigned this Oct 5, 2026
@marevol
marevol merged commit 871bedb into main Oct 5, 2026
8 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant