Skip to content

Commit b548118

Browse files
committed
Fix up configuration tests
Signed-off-by: Lakira Ashley <[email protected]>
1 parent 4e92b35 commit b548118

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

lib/xilinx/tests/spartan6/configuration_test.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,8 @@ TEST(ConfigurationTest, DISABLED_CheckForPaddingAfterIOBFrame) {
153153
frames.getFrames(), test_part);
154154
// createType2ConfigurationPacketData should add a 16-bit pad word after
155155
// after the IOB frame
156-
EXPECT_EQ(packet_data.size(), 3 * 65 + 1);
156+
EXPECT_EQ(packet_data.frames.size(), 1);
157+
EXPECT_EQ(packet_data.frames[0].data.size(), 3 * 65 + 1);
157158

158159
std::vector<uint32_t> idcode{0x1234};
159160
std::vector<uint32_t> cmd{0x0001};
@@ -182,7 +183,7 @@ TEST(ConfigurationTest, DISABLED_CheckForPaddingAfterIOBFrame) {
182183
static_cast<unsigned int>(0x1),
183184
ConfigurationPacket<Spartan6::ConfRegType>::Opcode::Write,
184185
Spartan6::ConfRegType::FDRI,
185-
absl::MakeSpan(packet_data),
186+
absl::MakeSpan(packet_data.frames[0].data),
186187
},
187188
};
188189

lib/xilinx/tests/xc7series/configuration_test.cc

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -278,7 +278,8 @@ TEST(ConfigurationTest, CheckForPaddingFrames) {
278278
// extra padding frames are added at the end of the creation of the data
279279
// overall this gives us: 5(real frames) + 4*2 + 2 = 15 frames, which is
280280
// 15 * 101 = 1515 words
281-
EXPECT_EQ(packet_data.size(), 15 * 101);
281+
EXPECT_EQ(packet_data.frames.size(), 1);
282+
EXPECT_EQ(packet_data.frames[0].data.size(), 15 * 101);
282283

283284
std::vector<uint32_t> idcode{0x1234};
284285
std::vector<uint32_t> cmd{0x0001};
@@ -307,7 +308,7 @@ TEST(ConfigurationTest, CheckForPaddingFrames) {
307308
static_cast<unsigned int>(0x1),
308309
ConfigurationPacket<Series7::ConfRegType>::Opcode::Write,
309310
Series7::ConfRegType::FDRI,
310-
absl::MakeSpan(packet_data),
311+
absl::MakeSpan(packet_data.frames[0].data),
311312
},
312313
};
313314

0 commit comments

Comments
 (0)