Port unit tests to Google Test by daljit46 · Pull Request #3116 · MRtrix3/mrtrix3

daljit46 · 2025-06-25T08:00:18Z

This PR ports all of our existing tests in testing/unit_tests to use the Google Test, a widely adopted unit testing framework in the C++ world. All tests have been rewritten to leverage the Google Test API, replacing our previous in-house solution. All tests should provide the same coverage as before. The exception is the tests for our dynamic MR::BitSet class, where the new set of tests should be slightly more comprehensive than the previous ones.
On the CMake side, the only needed change is the addition of Google Test in cmake/Dependencies.cmake and very minimal changes to the CMake logic in unit_tests/CMakeLists.txt.

github-actions

clang-tidy made some suggestions

There were too many comments to post at once. Showing the first 25 out of 69. Check the log or trigger a new build to see more.

github-actions · 2025-06-25T08:04:46Z

testing/unit_tests/bitset_tests.cpp

+ * For more details, see http://www.mrtrix.org/.
+ */
+
+#include "gtest/gtest.h"


warning: 'gtest/gtest.h' file not found [clang-diagnostic-error]

#include "gtest/gtest.h" ^

github-actions · 2025-06-25T08:04:46Z

testing/unit_tests/bitset_tests.cpp

+  bool initial_fill_value;
+};
+
+TEST_P(BitSetParamTest, ConstructorAndInitialState) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_P(BitSetParamTest, ConstructorAndInitialState) {

TEST_P(BitSetParamTest /*unused*/, ConstructorAndInitialState /*unused*/) {

github-actions · 2025-06-25T08:04:46Z

testing/unit_tests/bitset_tests.cpp

+  }
+}
+
+TEST_P(BitSetParamTest, CopyConstructorAndEquality) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_P(BitSetParamTest, CopyConstructorAndEquality) {

TEST_P(BitSetParamTest /*unused*/, CopyConstructorAndEquality /*unused*/) {

github-actions · 2025-06-25T08:04:46Z

testing/unit_tests/bitset_tests.cpp

+  }
+}
+
+TEST_P(BitSetParamTest, AssignmentOperator) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_P(BitSetParamTest, AssignmentOperator) {

TEST_P(BitSetParamTest /*unused*/, AssignmentOperator /*unused*/) {

github-actions · 2025-06-25T08:04:46Z

testing/unit_tests/bitset_tests.cpp

+TEST_P(BitSetParamTest, AssignmentOperator) {
+  for (size_t num_bits = 0; num_bits < 16; ++num_bits) {
+    const MR::BitSet source(num_bits, initial_fill_value);
+    const bool other_fill_value = !initial_fill_value;


warning: variable 'other_fill_value' is not initialized [cppcoreguidelines-init-variables]

Suggested change

const bool other_fill_value = !initial_fill_value;

const bool other_fill_value = false = !initial_fill_value;

github-actions · 2025-06-25T08:04:48Z

testing/unit_tests/ordered_queue_tests.cpp

+TEST_F(OrderedQueueTest, Regular2Stage) {
+  const MR::Timer timer;
+
+  SourceFunctor source;


warning: variable 'source' of type 'SourceFunctor' can be declared 'const' [misc-const-correctness]

Suggested change

SourceFunctor source;

SourceFunctor const source;

github-actions · 2025-06-25T08:04:49Z

testing/unit_tests/ordered_queue_tests.cpp

+  const MR::Timer timer;
+
+  SourceFunctor source;
+  SinkFunctor sink;


warning: variable 'sink' of type 'SinkFunctor' can be declared 'const' [misc-const-correctness]

Suggested change

SinkFunctor sink;

SinkFunctor const sink;

github-actions · 2025-06-25T08:04:49Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::Enforce);
+}
+
+TEST_F(OrderedQueueTest, Batched2Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, Batched2Stage) {

TEST_F(OrderedQueueTest /*unused*/, Batched2Stage /*unused*/) {

github-actions · 2025-06-25T08:04:49Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::Enforce);
+}
+
+TEST_F(OrderedQueueTest, Regular3Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, Regular3Stage) {

TEST_F(OrderedQueueTest /*unused*/, Regular3Stage /*unused*/) {

github-actions · 2025-06-25T08:04:49Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::DoNotEnforce);
+}
+
+TEST_F(OrderedQueueTest, BatchedUnbatched3Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, BatchedUnbatched3Stage) {

TEST_F(OrderedQueueTest /*unused*/, BatchedUnbatched3Stage /*unused*/) {

- Runs every combination of bit-set size in [0-15] and initial fill value. - New scenarios: - assignment-operator equivalence - resize-shrink paths and round-trip through size 0 - explicit mixed-state sanity checks

github-actions

clang-tidy made some suggestions

There were too many comments to post at once. Showing the first 25 out of 44. Check the log or trigger a new build to see more.

github-actions · 2025-06-25T11:25:47Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::DoNotEnforce);
+}
+
+TEST_F(OrderedQueueTest, UnbatchedBatched3Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, UnbatchedBatched3Stage) {

TEST_F(OrderedQueueTest /*unused*/, UnbatchedBatched3Stage /*unused*/) {

github-actions · 2025-06-25T11:25:47Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::DoNotEnforce);
+}
+
+TEST_F(OrderedQueueTest, BatchedBatchedRegular3Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, BatchedBatchedRegular3Stage) {

TEST_F(OrderedQueueTest /*unused*/, BatchedBatchedRegular3Stage /*unused*/) {

github-actions · 2025-06-25T11:25:47Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::DoNotEnforce);
+}
+
+TEST_F(OrderedQueueTest, Regular4Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, Regular4Stage) {

TEST_F(OrderedQueueTest /*unused*/, Regular4Stage /*unused*/) {

github-actions · 2025-06-25T11:25:47Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::DoNotEnforce);
+}
+
+TEST_F(OrderedQueueTest, BatchedUnbatchedUnbatched4Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, BatchedUnbatchedUnbatched4Stage) {

TEST_F(OrderedQueueTest /*unused*/, BatchedUnbatchedUnbatched4Stage /*unused*/) {

github-actions · 2025-06-25T11:25:48Z

testing/unit_tests/ordered_queue_tests.cpp

+  PerformChecksAndLog(timer, sink, OrderEnforcement::DoNotEnforce);
+}
+
+TEST_F(OrderedQueueTest, UnbatchedBatchedUnbatched4Stage) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(OrderedQueueTest, UnbatchedBatchedUnbatched4Stage) {

TEST_F(OrderedQueueTest /*unused*/, UnbatchedBatchedUnbatched4Stage /*unused*/) {

github-actions · 2025-06-25T11:25:50Z

testing/unit_tests/roi_tests.cpp

+  }
+}
+
+TEST_F(ROITests, ThreeOrderedROIsCorrectOrder) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ROITests, ThreeOrderedROIsCorrectOrder) {

TEST_F(ROITests /*unused*/, ThreeOrderedROIsCorrectOrder /*unused*/) {

github-actions · 2025-06-25T11:25:50Z

testing/unit_tests/roi_tests.cpp

+  }
+}
+
+TEST_F(ROITests, ThreeOrderedROIsIllegalABA) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ROITests, ThreeOrderedROIsIllegalABA) {

TEST_F(ROITests /*unused*/, ThreeOrderedROIsIllegalABA /*unused*/) {

github-actions · 2025-06-25T11:25:50Z

testing/unit_tests/roi_tests.cpp

+  }
+}
+
+TEST_F(ROITests, FourOrderedROIsIllegalABCAD) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ROITests, FourOrderedROIsIllegalABCAD) {

TEST_F(ROITests /*unused*/, FourOrderedROIsIllegalABCAD /*unused*/) {

github-actions · 2025-06-25T11:25:50Z

testing/unit_tests/roi_tests.cpp

+  }
+}
+
+TEST_F(ROITests, CombinationOrderedAndUnorderedROIs) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ROITests, CombinationOrderedAndUnorderedROIs) {

TEST_F(ROITests /*unused*/, CombinationOrderedAndUnorderedROIs /*unused*/) {

github-actions · 2025-06-25T11:25:50Z

testing/unit_tests/shuffle_tests.cpp

+
+#include <Eigen/Core>
+#include <algorithm>
+#include <gtest/gtest.h>


warning: 'gtest/gtest.h' file not found [clang-diagnostic-error]

#include <gtest/gtest.h> ^

Using CTAD runs into deep fold expressions and exceed's template nesting limits on both Clang and GCC.

github-actions

clang-tidy made some suggestions

github-actions · 2025-06-25T11:37:55Z

testing/unit_tests/shuffle_tests.cpp

+void PrintTo(const ShufflerParams &params, ::std::ostream *os) { *os << "{" << params.name << "}"; }
+
+std::vector<ShufflerParams> GetShufflerTestParams() {
+  std::vector<ShufflerParams> all_params;


warning: variable 'all_params' is not initialized [cppcoreguidelines-init-variables]

Suggested change

std::vector<ShufflerParams> all_params;

std::vector<ShufflerParams> all_params = 0;

github-actions · 2025-06-25T11:37:56Z

testing/unit_tests/shuffle_tests.cpp

+}
+} // namespace
+
+class ShufflerTest : public ::testing::Test {


warning: constructor does not initialize these fields: blocks [cppcoreguidelines-pro-type-member-init]

testing/unit_tests/shuffle_tests.cpp:208:

- std::vector<std::set<size_t>> blocks; + std::vector<std::set<size_t>> blocks{};

github-actions · 2025-06-25T11:37:56Z

testing/unit_tests/shuffle_tests.cpp

+      blocks[block_indices[i]].insert(i);
+  }
+
+  void TestPermutationWithin(Shuffler &in, const std::string &fail_msg) {


warning: method 'TestPermutationWithin' can be made static [readability-convert-member-functions-to-static]

Suggested change

void TestPermutationWithin(Shuffler &in, const std::string &fail_msg) {

static void TestPermutationWithin(Shuffler &in, const std::string &fail_msg) {

github-actions · 2025-06-25T11:37:56Z

testing/unit_tests/shuffle_tests.cpp

+  void TestPermutationWithin(Shuffler &in, const std::string &fail_msg) {
+    in.reset();
+    Shuffle shuffle;
+    Eigen::Array<int, Eigen::Dynamic, 1> shuffled_data;


warning: variable 'shuffled_data' is not initialized [cppcoreguidelines-init-variables]

Suggested change

Eigen::Array<int, Eigen::Dynamic, 1> shuffled_data;

Eigen::Array<int, Eigen::Dynamic, 1> shuffled_data = 0;

github-actions · 2025-06-25T11:37:56Z

testing/unit_tests/shuffle_tests.cpp

+    }
+  }
+
+  void TestSignflipWhole(Shuffler &in, const std::string &fail_msg) {


warning: method 'TestSignflipWhole' can be made static [readability-convert-member-functions-to-static]

Suggested change

void TestSignflipWhole(Shuffler &in, const std::string &fail_msg) {

static void TestSignflipWhole(Shuffler &in, const std::string &fail_msg) {

github-actions · 2025-06-25T11:37:58Z

testing/unit_tests/to_tests.cpp

+ * For more details, see http://www.mrtrix.org/.
+ */
+
+#include "gtest/gtest.h"


warning: 'gtest/gtest.h' file not found [clang-diagnostic-error]

#include "gtest/gtest.h" ^

github-actions · 2025-06-25T11:37:58Z

testing/unit_tests/to_tests.cpp

+
+class ToBoolTest : public ::testing::Test {};
+
+TEST_F(ToBoolTest, StringToBoolConversion) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ToBoolTest, StringToBoolConversion) {

TEST_F(ToBoolTest /*unused*/, StringToBoolConversion /*unused*/) {

github-actions · 2025-06-25T11:37:58Z

testing/unit_tests/to_tests.cpp

+
+class ToIntTest : public ::testing::Test {};
+
+TEST_F(ToIntTest, StringToIntConversion) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ToIntTest, StringToIntConversion) {

TEST_F(ToIntTest /*unused*/, StringToIntConversion /*unused*/) {

github-actions · 2025-06-25T11:37:58Z

testing/unit_tests/to_tests.cpp

+
+class ToFloatTest : public ::testing::Test {};
+
+TEST_F(ToFloatTest, StringToFloatConversion) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ToFloatTest, StringToFloatConversion) {

TEST_F(ToFloatTest /*unused*/, StringToFloatConversion /*unused*/) {

github-actions · 2025-06-25T11:37:58Z

testing/unit_tests/to_tests.cpp

+
+class ToComplexFloatTest : public ::testing::Test {};
+
+TEST_F(ToComplexFloatTest, StringToComplexFloatConversion) {


warning: all parameters should be named in a function [readability-named-parameter]

Suggested change

TEST_F(ToComplexFloatTest, StringToComplexFloatConversion) {

TEST_F(ToComplexFloatTest /*unused*/, StringToComplexFloatConversion /*unused*/) {

daljit46 self-assigned this Jun 25, 2025

daljit46 added CI/tests code quality labels Jun 25, 2025

daljit46 requested a review from a team June 25, 2025 08:01

github-actions bot reviewed Jun 25, 2025

View reviewed changes

daljit46 removed the request for review from a team June 25, 2025 08:37

daljit46 added 11 commits June 25, 2025 12:03

Start migrating unit tests to Google Test

20db969

Migrate BitSet unit tests to GTest and expand coverage

c6c40f7

- Runs every combination of bit-set size in [0-15] and initial fill value. - New scenarios: - assignment-operator equivalence - resize-shrink paths and round-trip through size 0 - explicit mixed-state sanity checks

Make tests for BitSet less verbose

840e381

Port erfinv to Google Test

827133c

Port parse_ints tests to Google Test

2a90885

Port tests for MR::to to Google Test

2fedc0f

Port sh_computer tests to Google Test

6a26e44

Add more tests

44995db

Add missing C++ tests

593357d

Add Google Test dependency only when testing is enabled

86f364d

Make shuffle tests less verbose

7cb1956

daljit46 force-pushed the google_test branch from ccba644 to 404e6cc Compare June 25, 2025 11:21

github-actions bot reviewed Jun 25, 2025

View reviewed changes

daljit46 added 2 commits June 25, 2025 12:33

Apply clang-format to bitset_tests.cpp

4165ae2

Use std::vector in icls_tests.cpp for large arrays

4c6976f

Using CTAD runs into deep fold expressions and exceed's template nesting limits on both Clang and GCC.

daljit46 force-pushed the google_test branch from 404e6cc to 4c6976f Compare June 25, 2025 11:33

github-actions bot reviewed Jun 25, 2025

View reviewed changes

daljit46 added 7 commits June 25, 2025 12:55

Address some clang-tidy warnings

c5bee9e

Link against pthread for GTest executable

81d6b98

Remove extra parenthesis

10af8c9

Don't link GTest twice

41d865b

Relocate comment to appropriate location

a79d7e1

Avoid ABI mismatches with gtest with STL debug mode

4838ce9

Format icls_tests.cpp

61cfc53

Run unit tests in parallel

2916ca4

daljit46 force-pushed the google_test branch from 25ad4fa to 2916ca4 Compare June 25, 2025 14:56

daljit46 requested a review from a team June 25, 2025 15:02

Lestropie force-pushed the google_test branch from 2916ca4 to e8c2669 Compare August 26, 2025 07:47

Lestropie force-pushed the dev branch 3 times, most recently from 70031c3 to 6bf4cec Compare August 26, 2025 08:11

Lestropie force-pushed the google_test branch from e8c2669 to 2916ca4 Compare August 29, 2025 06:10

Lestropie mentioned this pull request Aug 29, 2025

Reversion of erroneous force pushes & branch stocktake #3167

Open

daljit46 merged commit 052ed0c into dev Sep 22, 2025
15 of 17 checks passed

daljit46 deleted the google_test branch September 22, 2025 10:47

Lestropie mentioned this pull request Sep 24, 2025

Remove MR::BitSet class #3182

Merged

Lestropie mentioned this pull request Nov 11, 2025

API: Replace .type_various() #3074

Merged

	TEST_P(BitSetParamTest, ConstructorAndInitialState) {
	TEST_P(BitSetParamTest /unused/, ConstructorAndInitialState /unused/) {

	TEST_P(BitSetParamTest, CopyConstructorAndEquality) {
	TEST_P(BitSetParamTest /unused/, CopyConstructorAndEquality /unused/) {

	TEST_P(BitSetParamTest, AssignmentOperator) {
	TEST_P(BitSetParamTest /unused/, AssignmentOperator /unused/) {

	const bool other_fill_value = !initial_fill_value;
	const bool other_fill_value = false = !initial_fill_value;

	TEST_F(OrderedQueueTest, Batched2Stage) {
	TEST_F(OrderedQueueTest /unused/, Batched2Stage /unused/) {

	TEST_F(OrderedQueueTest, Regular3Stage) {
	TEST_F(OrderedQueueTest /unused/, Regular3Stage /unused/) {

	TEST_F(OrderedQueueTest, BatchedUnbatched3Stage) {
	TEST_F(OrderedQueueTest /unused/, BatchedUnbatched3Stage /unused/) {

	TEST_F(OrderedQueueTest, UnbatchedBatched3Stage) {
	TEST_F(OrderedQueueTest /unused/, UnbatchedBatched3Stage /unused/) {

	TEST_F(OrderedQueueTest, BatchedBatchedRegular3Stage) {
	TEST_F(OrderedQueueTest /unused/, BatchedBatchedRegular3Stage /unused/) {

	TEST_F(OrderedQueueTest, Regular4Stage) {
	TEST_F(OrderedQueueTest /unused/, Regular4Stage /unused/) {

	TEST_F(ROITests, ThreeOrderedROIsCorrectOrder) {
	TEST_F(ROITests /unused/, ThreeOrderedROIsCorrectOrder /unused/) {

	TEST_F(ROITests, ThreeOrderedROIsIllegalABA) {
	TEST_F(ROITests /unused/, ThreeOrderedROIsIllegalABA /unused/) {

	TEST_F(ROITests, FourOrderedROIsIllegalABCAD) {
	TEST_F(ROITests /unused/, FourOrderedROIsIllegalABCAD /unused/) {

	TEST_F(ROITests, CombinationOrderedAndUnorderedROIs) {
	TEST_F(ROITests /unused/, CombinationOrderedAndUnorderedROIs /unused/) {

	std::vector<ShufflerParams> all_params;
	std::vector<ShufflerParams> all_params = 0;

	void TestPermutationWithin(Shuffler &in, const std::string &fail_msg) {
	static void TestPermutationWithin(Shuffler &in, const std::string &fail_msg) {

	Eigen::Array<int, Eigen::Dynamic, 1> shuffled_data;
	Eigen::Array<int, Eigen::Dynamic, 1> shuffled_data = 0;

	void TestSignflipWhole(Shuffler &in, const std::string &fail_msg) {
	static void TestSignflipWhole(Shuffler &in, const std::string &fail_msg) {


		class ToBoolTest : public ::testing::Test {};

		TEST_F(ToBoolTest, StringToBoolConversion) {

	TEST_F(ToBoolTest, StringToBoolConversion) {
	TEST_F(ToBoolTest /unused/, StringToBoolConversion /unused/) {

Conversation

daljit46 commented Jun 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

Choose a reason for hiding this comment

Uh oh!

github-actions bot Jun 25, 2025

daljit46 commented Jun 25, 2025 •

edited

Loading


		class ToIntTest : public ::testing::Test {};

		TEST_F(ToIntTest, StringToIntConversion) {

	TEST_F(ToIntTest, StringToIntConversion) {
	TEST_F(ToIntTest /unused/, StringToIntConversion /unused/) {


		class ToFloatTest : public ::testing::Test {};

		TEST_F(ToFloatTest, StringToFloatConversion) {

	TEST_F(ToFloatTest, StringToFloatConversion) {
	TEST_F(ToFloatTest /unused/, StringToFloatConversion /unused/) {


		class ToComplexFloatTest : public ::testing::Test {};

		TEST_F(ToComplexFloatTest, StringToComplexFloatConversion) {

	TEST_F(ToComplexFloatTest, StringToComplexFloatConversion) {
	TEST_F(ToComplexFloatTest /unused/, StringToComplexFloatConversion /unused/) {