search

JGCRI / hector

The Hector Simple Climate Model
http://jgcri.github.io/hector/
GNU General Public License v3.0
107 stars 45 forks source link

Investigate why streaming into google test doesn't compile in some cases #665

Open bpbond opened 1 year ago

bpbond commented 1 year ago

In test_dependency_finder.cpp we stream messages into tests, e.g.

  EXPECT_LT( bIt - ordering.begin(), aIt - ordering.begin() ) << "Ordering: " << ordering << endl;

and this doesn't compile with some compiler/googletest combinations — for example clang and googletest 1.12.1_1 (what I'm using currently) — while it does work with e.g. @pralitp 's gcc and 1.11.0-3.

See https://github.com/JGCRI/hector/pull/664#issuecomment-1334679160

I commented out these streamed messages, so as to merge #664 and get to v3, but in 2023 let's revisit.