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

Add Tests for the Bucket Datastructure #27

Merged

Conversation

franziska-wegner
Copy link
Owner

This PR adds initial tests for the bucket data structure.

Changes to be committed:
	modified:   tests/CMakeLists.txt
@franziska-wegner franziska-wegner added the tests Add or update tests to EGOA label Dec 8, 2023
@franziska-wegner franziska-wegner self-assigned this Dec 8, 2023
@franziska-wegner franziska-wegner linked an issue Dec 8, 2023 that may be closed by this pull request
3 tasks
@franziska-wegner franziska-wegner marked this pull request as ready for review December 13, 2023 10:52
@franziska-wegner franziska-wegner enabled auto-merge (squash) December 13, 2023 10:53
@franziska-wegner franziska-wegner merged commit 7c9da13 into main Dec 13, 2023
12 checks passed
@franziska-wegner franziska-wegner deleted the franziska-wegner/2023/december/tests/datastructures/bucket branch December 13, 2023 11:01
github-actions bot added a commit that referenced this pull request Dec 13, 2023
* Enable Bucket tests
Changes to be committed:
	modified:   tests/CMakeLists.txt
* Remove round brackets
* Mark a deprecated function as todo C++17 change
* Comment out some tests
* Add resolved tests
* Switching from `EXPECT_THAT(MatchesRegex(assertionString.c_str()), error.what())` to `EXPECT_THAT(error.what(),MatchesRegex(assertionString.c_str()))`
* Windows differentiates between 'operator <' and 'operator<'
   Why?
   * Whitespaces
* Switch a EXPECT_THAT statement from assertionString to assertionString2
* Change operator[] to operator.*[] due to Windows whitespace issue 7c9da13
github-actions bot added a commit that referenced this pull request Dec 13, 2023
* Enable Bucket tests
Changes to be committed:
	modified:   tests/CMakeLists.txt
* Remove round brackets
* Mark a deprecated function as todo C++17 change
* Comment out some tests
* Add resolved tests
* Switching from `EXPECT_THAT(MatchesRegex(assertionString.c_str()), error.what())` to `EXPECT_THAT(error.what(),MatchesRegex(assertionString.c_str()))`
* Windows differentiates between 'operator <' and 'operator<'
   Why?
   * Whitespaces
* Switch a EXPECT_THAT statement from assertionString to assertionString2
* Change operator[] to operator.*[] due to Windows whitespace issue 7c9da13
github-actions bot added a commit that referenced this pull request Dec 13, 2023
* Enable Bucket tests
Changes to be committed:
	modified:   tests/CMakeLists.txt
* Remove round brackets
* Mark a deprecated function as todo C++17 change
* Comment out some tests
* Add resolved tests
* Switching from `EXPECT_THAT(MatchesRegex(assertionString.c_str()), error.what())` to `EXPECT_THAT(error.what(),MatchesRegex(assertionString.c_str()))`
* Windows differentiates between 'operator <' and 'operator<'
   Why?
   * Whitespaces
* Switch a EXPECT_THAT statement from assertionString to assertionString2
* Change operator[] to operator.*[] due to Windows whitespace issue 7c9da13
github-actions bot added a commit that referenced this pull request Dec 13, 2023
* Enable Bucket tests
Changes to be committed:
	modified:   tests/CMakeLists.txt
* Remove round brackets
* Mark a deprecated function as todo C++17 change
* Comment out some tests
* Add resolved tests
* Switching from `EXPECT_THAT(MatchesRegex(assertionString.c_str()), error.what())` to `EXPECT_THAT(error.what(),MatchesRegex(assertionString.c_str()))`
* Windows differentiates between 'operator <' and 'operator<'
   Why?
   * Whitespaces
* Switch a EXPECT_THAT statement from assertionString to assertionString2
* Change operator[] to operator.*[] due to Windows whitespace issue 7c9da13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Add or update tests to EGOA
Projects
Development

Successfully merging this pull request may close these issues.

Test Bucket Datastructure
1 participant