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

[TEST] Seven commits a top of the current default branch #26

Closed

Conversation

hebasto
Copy link
Owner

@hebasto hebasto commented Sep 15, 2023

No description provided.

@hebasto hebasto deleted the branch 230915-each-commit.MERGED September 19, 2023 16:13
@hebasto hebasto closed this Sep 19, 2023
@hebasto hebasto deleted the 230915-each-commit.TEST.seven-top branch September 26, 2023 08:57
hebasto added a commit that referenced this pull request Jul 22, 2024
4d77971 fixup! cmake: Create test suite for `ctest` (Hennadii Stepanov)
16ac373 fixup! cmake: Build `test_bitcoin` executable (Hennadii Stepanov)

Pull request description:

  CMake allows to report about skipped tests properly instead of passing dummy test cases. For instance, on Windows:
  ```
  > ctest --build-config Release -R compilerbug
  Test project C:/Users/hebasto/source/repos/bitcoin/build
      Start 26: compilerbug_tests
  1/1 Test #26: compilerbug_tests ................***Skipped   0.02 sec

  100% tests passed, 0 tests failed out of 1

  Total Test time (real) =   0.04 sec

  The following tests did not run:
           26 - compilerbug_tests (Skipped)
  ```

  We have 3 such cases:
  - `src/test/compilerbug_tests.cpp`
  - `src/test/raii_event_tests.cpp`
  - `src/test/system_tests.cpp`

  This PR handles the first two cases.

  Once bitcoin#30454 is merged, the following lines can be deleted: https://github.com/hebasto/bitcoin/blob/9b4aa929b0e84e64472402e594ebb13648bac878/src/test/system_tests.cpp#L19-L25

Top commit has no ACKs.

Tree-SHA512: e4be4fd2469163fa4b3a36afb90f6186204f0151f668490305b6a7e632a6922cc4982233a6fc4bf2b1f7049658b42916a20e766197969aedd28f2eb097c4674d
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.

1 participant