Skip to content

Commit

Permalink
🚧 change CHECK statement.
Browse files Browse the repository at this point in the history
  • Loading branch information
Drewniok committed Jul 20, 2023
1 parent 5986de8 commit 9b6ca3f
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions test/algorithms/simulation/sidb/random_layout_generator.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -12,9 +12,6 @@
#include <fiction/layouts/clocked_layout.hpp>
#include <fiction/technology/cell_technologies.hpp>

#include <sstream>
#include <vector>

using namespace fiction;

TEST_CASE("Random cube::coord_t layout generation", "[generate_random_layout]")

Check warning

Code scanning / CodeQL

Poorly documented large function Warning test

Poorly documented function: fewer than 2% comments for a function of 175 lines.
Expand Down Expand Up @@ -435,8 +432,7 @@ TEST_CASE("Random siqad::coord_t layout generation", "[generate_random_layout]")
CHECK(cell.x > 0);
CHECK(cell.y < 8);
CHECK(cell.y > 0);
CHECK(cell.z < 21);
CHECK(cell.z >= 0);
CHECK(cell.z <= 1);
});
}

Expand Down Expand Up @@ -512,7 +508,7 @@ TEST_CASE("Random siqad::coord_t layout generation", "[generate_random_layout]")
SECTION("Check uniqueness of two layouts")
{
const generate_random_layout_params<sidb_cell_clk_lyt> params{
{{0, 0, 1}, {9, 9, 0}}, 10, positive_charges::FORBIDDEN, 2, static_cast<uint64_t>(10E6), 2};
{{0, 0, 1}, {20, 20, 0}}, 10, positive_charges::FORBIDDEN, 2, static_cast<uint64_t>(10E6), 2};
const auto result_lyts = generate_multiple_random_layouts<sidb_cell_clk_lyt>(sidb_cell_clk_lyt{}, params);
REQUIRE(result_lyts.size() == 2);

Expand Down

0 comments on commit 9b6ca3f

Please sign in to comment.