Skip to content

Comments

Use default value for face test flag#868

Merged
matsl merged 1 commit intomasterfrom
pr_hywiki_test_cleanup
Feb 17, 2026
Merged

Use default value for face test flag#868
matsl merged 1 commit intomasterfrom
pr_hywiki_test_cleanup

Conversation

@matsl
Copy link
Collaborator

@matsl matsl commented Feb 17, 2026

What

Use default value for face test flag.

  • test/hywiki-tests.el (hywiki-tests--verify-removal-of-delimiter-updates-face):
    (hywiki-tests--wikiword-yanked-with-extra-words):
    (hywiki-tests--create-wikiword-file-highlights-wikiword): Use
    hywiki-tests--with-face-test default value.
  • test/hywiki-yki-tests.el (hywiki--verify-get-buffer-text-with-point-and-highlight-compact):
    Use hywiki-tests--with-face-test default value. Let
    hywiki-tests--preserve-hywiki-mode handle wikiword file and hywiki-directory.

Why

The face test flag is per default t so we should only set it to nil in
the cases where we don't want to verify the face. This is a left over
from the initial test for faces where the highlighting did not work
properly.

The hywiki-tests--preserve-hywiki-mode macro handles the removal of
the hywiki-directory and also creates the first WikiWord for the
tests. Test should just use that when applicable.

* test/hywiki-tests.el (hywiki-tests--verify-removal-of-delimiter-updates-face):
    (hywiki-tests--wikiword-yanked-with-extra-words):
    (hywiki-tests--create-wikiword-file-highlights-wikiword): Use
    hywiki-tests--with-face-test default value.
* test/hywiki-yki-tests.el (hywiki--verify-get-buffer-text-with-point-and-highlight-compact):
    Use hywiki-tests--with-face-test default value. Let
    hywiki-tests--preserve-hywiki-mode handle wikiword file and hywiki-directory.
@matsl matsl requested a review from rswgnu February 17, 2026 09:38
@matsl matsl merged commit 96350ca into master Feb 17, 2026
4 checks passed
@matsl matsl deleted the pr_hywiki_test_cleanup branch February 17, 2026 14:51
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.

2 participants