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

Drv/TcpClient unit tests intermittently fail on Mac OS #881

Closed
bocchino opened this issue Aug 1, 2021 · 6 comments
Closed

Drv/TcpClient unit tests intermittently fail on Mac OS #881

bocchino opened this issue Aug 1, 2021 · 6 comments
Labels

Comments

@bocchino
Copy link
Collaborator

bocchino commented Aug 1, 2021

F´ Version release/v3.0.0
Affected Component Drv/TcpClient

Problem Description

Intermittently, when running the unit tests for Drv/TcpClient, I see errors like this:

1: [ RUN      ] Nominal.BasicMessaging
1: /Users/bocchino/JPL/Tools/FPrime/fprime/Drv/TcpClient/test/ut/Tester.cpp:42: Failure
1: Expected equality of these values:
1:   serverStat
1:     Which is: -9
1:   SOCK_SUCCESS
1:     Which is: 0
1: [  FAILED  ] Nominal.BasicMessaging (3 ms)
1: [ RUN      ] Nominal.BasicReceiveThread
1: /Users/bocchino/JPL/Tools/FPrime/fprime/Drv/TcpClient/test/ut/Tester.cpp:42: Failure
1: Expected equality of these values:
1:   serverStat
1:     Which is: -9
1:   SOCK_SUCCESS
1:     Which is: 0
1: [  FAILED  ] Nominal.BasicReceiveThread (1 ms)

How to Reproduce

Run the unit tests as described above. I am running on a MacBook Pro with Mac OS 10.15.17.

Expected Behavior

Unit tests should pass.

@bocchino bocchino added the bug label Aug 1, 2021
@Joshua-Anderson
Copy link
Contributor

@LeStarch I seem to remember you said that you found the issue causing this. What was the fix?

@LeStarch
Copy link
Collaborator

LeStarch commented Aug 2, 2021

This should work and it doesn't....so apparently I did not fix it.

@LeStarch
Copy link
Collaborator

LeStarch commented Aug 2, 2021

...wait this is on the v3.0.0 branch. We should update to the latest devel and see if it fails there.

@LeStarch
Copy link
Collaborator

Not seen this in a few months on v3.0.0 w/ macOS CI. Closing.

@JohanBertrand
Copy link
Contributor

Out of curiosity, what is the reason behind this error?

It is also randomly happening for me on a docker

[ RUN      ] Nominal.BasicMessaging
/builds/Fprime/Drv/TcpServer/test/ut/TcpServerTester.cpp:44: Failure
Expected equality of these values:
  serverStat
    Which is: -9
  SOCK_SUCCESS
    Which is: 0
/builds/Fprime/Drv/TcpServer/test/ut/TcpServerTester.cpp:46: Failure
Value of: component.getSocketHandler().isStarted()
  Actual: false
Expected: true
/builds/build-fprime-automatic-native-ut/F-Prime/Drv/TcpServer/TcpServerGTestBase.cpp:87: Failure
Expected equality of these values:
  size
    Which is: 1
  this->fromPortHistorySize_ready
    Which is: 0
/builds/Fprime/Drv/TcpServer/test/ut/TcpServerTester.cpp:100
  Value:    Size of history for ready
  Expected: 1
  Actual:   0
[  FAILED  ] Nominal.BasicMessaging (0 ms)

@JohanBertrand
Copy link
Contributor

For the record, it is still happening on the pipeline, but not only on macos: https://github.com/nasa/fprime/actions/runs/8839296509/job/24272334136

[ RUN      ] Reconnect.MultiMessaging
/home/runner/work/fprime/fprime/Drv/TcpServer/test/ut/TcpServerTester.cpp:44: Failure
Expected equality of these values:
  serverStat
    Which is: -9
  SOCK_SUCCESS
    Which is: 0
/home/runner/work/fprime/fprime/Drv/TcpServer/test/ut/TcpServerTester.cpp:46: Failure
Value of: component.getSocketHandler().isStarted()
  Actual: false
Expected: true
/home/runner/work/fprime/fprime/build-fprime-automatic-native-ut/F-Prime/Drv/TcpServer/TcpServerGTestBase.cpp:87: Failure
Expected equality of these values:
  size
    Which is: 100
  this->fromPortHistorySize_ready
    Which is: 0

/home/runner/work/fprime/fprime/Drv/TcpServer/test/ut/TcpServerTester.cpp:100
  Value:    Size of history for ready
  Expected: 100
  Actual:   0

[  FAILED  ] Reconnect.MultiMessaging (1 ms)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants
@LeStarch @bocchino @Joshua-Anderson @JohanBertrand and others