Skip to content

Comments

Activate skipped test that can run in batch mode#867

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

Activate skipped test that can run in batch mode#867
matsl merged 1 commit intomasterfrom
pr_activate_batch_tests

Conversation

@matsl
Copy link
Collaborator

@matsl matsl commented Feb 16, 2026

What

Activate skipped test that can run in batch mode.

  • test/demo-tests.el:
  • test/hmouse-drv-tests.el:
  • test/hui-select-tests.el:
  • test/hui-tests.el:
  • test/hyrolo-tests.el:
  • test/kotl-mode-tests.el: Activate skipped test that can run in batch mode.
    • In addition to activating the tests some use of
      hy-test-helpers:consume-input-events has been removed where it is not
      needed.
    • A few tests were blocked by checking quoted strings where the quotes
      differ between batch and interactive. They have been modified to not verify
      the part of the message that contains the quotes.

Why

While cleanup some tests I noticed that I could get them to run in
batch mode by just activating them. Not sure why and when they started
to work in batch mode but they do now. We'll see what CI/CD says!

And having more tests work in batch mode is a good thing.

* test/demo-tests.el:
* test/hmouse-drv-tests.el:
* test/hui-select-tests.el:
* test/hui-tests.el:
* test/hyrolo-tests.el:
* test/kotl-mode-tests.el: Activate skipped test that can run in batch mode.
  * In addition to activating the tests some use of
    hy-test-helpers:consume-input-events has been removed where it is not
    needed.
  * A few tests were blocked by checking quoted strings where the quotes
    differ between batch and interactive. They have been modified to not verify
    the part of the message that contains the quotes.
@matsl matsl requested a review from rswgnu February 16, 2026 22:53
@matsl matsl merged commit ebd53c0 into master Feb 17, 2026
4 checks passed
@matsl matsl deleted the pr_activate_batch_tests branch February 17, 2026 09:20
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