Skip to content

Commit faee5f8

Browse files
author
MarcoFalke
committed
test: Create fresh CDataStream each time
Can be reviewed with --ignore-all-space
1 parent fa71114 commit faee5f8

File tree

1 file changed

+15
-14
lines changed

1 file changed

+15
-14
lines changed

src/test/streams_tests.cpp

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -164,34 +164,35 @@ BOOST_AUTO_TEST_CASE(bitstream_reader_writer)
164164
BOOST_AUTO_TEST_CASE(streams_serializedata_xor)
165165
{
166166
std::vector<std::byte> in;
167-
CDataStream ds(in, 0, 0);
168167

169168
// Degenerate case
170-
ds.Xor({0x00, 0x00});
171-
BOOST_CHECK_EQUAL(""s, ds.str());
169+
{
170+
CDataStream ds{in, 0, 0};
171+
ds.Xor({0x00, 0x00});
172+
BOOST_CHECK_EQUAL(""s, ds.str());
173+
}
172174

173175
in.push_back(std::byte{0x0f});
174176
in.push_back(std::byte{0xf0});
175177

176178
// Single character key
177-
178-
ds.clear();
179-
ds.insert(ds.begin(), in.begin(), in.end());
180-
181-
ds.Xor({0xff});
182-
BOOST_CHECK_EQUAL("\xf0\x0f"s, ds.str());
179+
{
180+
CDataStream ds{in, 0, 0};
181+
ds.Xor({0xff});
182+
BOOST_CHECK_EQUAL("\xf0\x0f"s, ds.str());
183+
}
183184

184185
// Multi character key
185186

186187
in.clear();
187188
in.push_back(std::byte{0xf0});
188189
in.push_back(std::byte{0x0f});
189190

190-
ds.clear();
191-
ds.insert(ds.begin(), in.begin(), in.end());
192-
193-
ds.Xor({0xff, 0x0f});
194-
BOOST_CHECK_EQUAL("\x0f\x00"s, ds.str());
191+
{
192+
CDataStream ds{in, 0, 0};
193+
ds.Xor({0xff, 0x0f});
194+
BOOST_CHECK_EQUAL("\x0f\x00"s, ds.str());
195+
}
195196
}
196197

197198
BOOST_AUTO_TEST_CASE(streams_buffered_file)

0 commit comments

Comments
 (0)