Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

👩‍🌾 Disable tests that initialize App on macOS: they're all flaky #184

Merged
merged 2 commits into from
Feb 23, 2021

Conversation

chapulina
Copy link
Contributor

Signed-off-by: Louise Poubel louise@openrobotics.org

🦟 Bug workaround

See #123

Summary

All the tests that initialize an App have been flaky on ign-gui4 even though they consistently pass on ign-gui3 (#123). I've tried a few different things to debug this with no luck.

Since we're not likely to have time to address these issues soon, I'm going to disable those tests so we don't pollute CI results with tests that are known to flake.

Our macOS CI uses a higher version of Qt than our Linux CI. So problems on macOS often let us know of upcoming problems on Linux. So commenting out these tests could just be pushing the problem down the road.

Checklist

  • Signed all commits for DCO
  • Added tests - more like removed 😄
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge


https://github.com/osrf/buildfarmer/issues/156

Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina added bug Something isn't working tests Broken or missing tests / testing infra macOS macOS support labels Feb 19, 2021
@osrf-triage osrf-triage added this to Inbox in Core development Feb 19, 2021
@github-actions github-actions bot added the 🔮 dome Ignition Dome label Feb 19, 2021
Signed-off-by: Louise Poubel <louise@openrobotics.org>
@chapulina chapulina moved this from Inbox to In review in Core development Feb 19, 2021
@codecov
Copy link

codecov bot commented Feb 19, 2021

Codecov Report

Merging #184 (5437444) into ign-gui4 (56bb6b3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           ign-gui4     #184   +/-   ##
=========================================
  Coverage     60.77%   60.77%           
=========================================
  Files            21       21           
  Lines          2840     2840           
=========================================
  Hits           1726     1726           
  Misses         1114     1114           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 56bb6b3...5437444. Read the comment docs.

@j-rivero
Copy link
Contributor

Probably out of the scope of this PR but github actions is failing with:

  [ RUN      ] ApplicationTest.LoadDefaultConfig
  [GUI] [Wrn] [Application.cc:649] [QT] could not connect to display :1.0
  [GUI] [Err] [Application.cc:653] [QT] This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.
  
  Available platform plugins are: eglfs, linuxfb, minimal, minimalegl, offscreen, vnc, xcb.
  
  
        Start  2: check_UNIT_Application_TEST
   2/32 Test  #2: check_UNIT_Application_TEST ............***Failed    0.03 sec

I did not find any issue created for this or if we want to fix it or disable it somehow in github actions.

@chapulina
Copy link
Contributor Author

I thought I had sent this message yesterday, but it was still here 😬

github actions is failing with

Failing? I don't see any failures 😇 I re-ran the tests, those ones are flaky, see #58

@chapulina chapulina merged commit 5d717e6 into ign-gui4 Feb 23, 2021
Core development automation moved this from In review to Done Feb 23, 2021
@chapulina chapulina deleted the chapulina/4/mac_tests branch February 23, 2021 18:47
@j-rivero j-rivero removed this from Done in Core development May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 🔮 dome Ignition Dome macOS macOS support tests Broken or missing tests / testing infra
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants