Skip to content

Add a unit test for the ConnectionRouter #1561

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

HackerFoo
Copy link
Contributor

@HackerFoo HackerFoo commented Sep 22, 2020

Adds a unit test for the ConnectionRouter.

Description

This unit test is both to verify functionality, and as a starting point to experiment with the router.

Related Issue

Motivation and Context

Unit tests allow quicker iteration because they run quickly and make it easier to locate bugs. Furthermore, they serve as documentation on to do something, and a starting point for changes.

Specifically, I'm working on parallel routing algorithms, so this is a starting point for that.

How Has This Been Tested?

The unit test passes in make test.

Types of changes

  • Bug fix (change which fixes an issue)
  • New feature (change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

  • My change requires a change to the documentation
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed

@probot-autolabeler probot-autolabeler bot added lang-cpp C/C++ code tests VPR VPR FPGA Placement & Routing Tool labels Sep 22, 2020
@HackerFoo HackerFoo force-pushed the connection_router_unit_test branch 3 times, most recently from 2abe08d to d5956a6 Compare September 22, 2020 23:59
@HackerFoo HackerFoo marked this pull request as ready for review September 23, 2020 00:01
@HackerFoo HackerFoo requested a review from litghost September 23, 2020 00:01
@HackerFoo HackerFoo force-pushed the connection_router_unit_test branch from d5956a6 to 42d484d Compare September 29, 2020 05:07
@HackerFoo HackerFoo merged commit ae5584f into verilog-to-routing:master Sep 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang-cpp C/C++ code tests VPR VPR FPGA Placement & Routing Tool
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants