Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add parity check UI #1829

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Add parity check UI #1829

wants to merge 2 commits into from

Commits on Oct 18, 2024

  1. Add basic UI for parity checks 

    Add a basic UI to kick off a parity check and view the progress/completion
status.

Add `ParityCheckJob` to run parity checks in the background.

Add methods to `ParityCheck` service for tracking progress.

Add `ParityChecksController` and associated views; no details of the check
results are surfaced yet, just the functionality to start and view the overall
status.

DRY up parity check tests with `migration_helper`.
    @ethax-ross
    ethax-ross committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    df2128e View commit details
    Browse the repository at this point in the history

  2. Add completion and details to parity check UI 

    Fleshes out the bare-bones parity check UI with an overview list of the paths
called for each provider on the completion page. If the request
responses/status codes are different you can click on the path to view a
detailed report of the comparison along with a diff of the response bodies.
Also includes a basic response time comparison chart that plots the averages
and min/max for each path.

Add detailed repsonse comparison action/view.

Add `diffy` to compare response bodies.

Update CSRP to allow chart.js from a CDN (as its only used on this page and it
will be deleted shortly I figured this was the best way to include it/ensure
its removed).

Remove `equal?` from `ResponseComparison` as this is a system method it started
giving issues; we can just negate `different?` in its place.
    @ethax-ross
    ethax-ross committed Oct 18, 2024
    Configuration menu
    Copy the full SHA
    f1d65d0 View commit details
    Browse the repository at this point in the history