search

eclipse-platform / eclipse.platform.text

8 stars 45 forks source link

Adjust ContextInformationTest#testContextInfo_hide_focusOut #168

Closed laeubi closed 1 year ago

laeubi commented 1 year ago

WDYT @akurtakov @iloveeclipse ?

iloveeclipse commented 1 year ago

If that works, it is better as disabling the test completely. Please note, build fails with ECA check.

laeubi commented 1 year ago

If that works, it is better as disabling the test completely.

I'll first check that the build succeeds then I'll enable centos-8 ... this is just a guess as I have no centos to test with.

laeubi commented 1 year ago

Did not help :-\ If I run the test local it also works, and the error indicates that focus was transferred,..

vogella commented 1 year ago

I will disable the test "temporary" with https://github.com/eclipse-platform/eclipse.platform.text/pull/162/files#diff-f15f98a8a22343cd1678b0fb9dc2206bbcd5b2e2e47d2ee279602fbcd2302dc5R96, I assume this should not prevent you from enabling it again if you find a working solution.

vogella commented 1 year ago

Disablement of test and migration to latest Cent OS https://github.com/eclipse-platform/eclipse.platform.text/pull/162