Skip to content

Running ide/projectui tests in the CI checks - #9603

Open
jtulach wants to merge 1 commit into
apache:masterfrom
jtulach:jtulach/EnableProjectUiTestsInCi
Open

Running ide/projectui tests in the CI checks#9603
jtulach wants to merge 1 commit into
apache:masterfrom
jtulach:jtulach/EnableProjectUiTestsInCi

Conversation

@jtulach

@jtulach jtulach commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

@jtulach
jtulach requested a review from mbien September 5, 2026 02:39
@jtulach jtulach self-assigned this Sep 5, 2026
boolean ok;

@Override
public void publish(LogRecord record) {

@jtulach jtulach Sep 5, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Logging handler to check whether something is happening deep in the code
    • and it is not happening
  • I guess we can just delete the test...
    • it must be some hardcore regression test
    • when it is using the log Handler in such an advanced way
    • maybe I wrote, it but I don't remember what it was supposed to do
  • anyway we lived without that tests for some time, we can live without it in the future

@jtulach
jtulach requested a review from lahodaj September 5, 2026 05:29
@mbien mbien added tests CI continuous integration changes labels Sep 5, 2026
@jtulach jtulach added the ci:all-tests [ci] enable all tests label Sep 6, 2026
@jtulach

jtulach commented Sep 6, 2026

Copy link
Copy Markdown
Contributor Author

I squash my commits and integrate.

@jtulach jtulach changed the title projectui tests are missing from CI checks ide/projectui tests are missing from CI checks Sep 6, 2026
@jtulach jtulach changed the title ide/projectui tests are missing from CI checks Running ide/projectui tests in the CI checks Sep 6, 2026
@jtulach
jtulach force-pushed the jtulach/EnableProjectUiTestsInCi branch from 7e218c0 to de131b2 Compare September 6, 2026 06:22
@jtulach
jtulach force-pushed the jtulach/EnableProjectUiTestsInCi branch from de131b2 to 288b53d Compare September 6, 2026 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:all-tests [ci] enable all tests CI continuous integration changes tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants