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

Added new command 'summary' #1127

Merged
merged 12 commits into from
Jun 10, 2022
Merged

Added new command 'summary' #1127

merged 12 commits into from
Jun 10, 2022

Commits on Jun 7, 2022

  1. Added command summary 

    * Added command named summary
* Added to_summary function to ReportTask
* Added tests for both 'summary' command and 'to_summary' function
    @shashigharti
    shashigharti committed Jun 7, 2022
    Configuration menu
    Copy the full SHA
    65d04a2 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'main' into 1095/adds-default-cli-commands

    @shashigharti
    shashigharti authored Jun 7, 2022
    Configuration menu
    Copy the full SHA
    efc6aba View commit details
    Browse the repository at this point in the history

  3. fixes for failing tests

    @shashigharti
    shashigharti committed Jun 7, 2022
    Configuration menu
    Copy the full SHA
    cde50b5 View commit details
    Browse the repository at this point in the history

  4. Revised code 

    * Rearranged code and placed it in the relevant places "separation of concerns"
* Made changes to tests and added new tests
* Revised summary command code
    @shashigharti
    shashigharti committed Jun 7, 2022
    Configuration menu
    Copy the full SHA
    25e6ee4 View commit details
    Browse the repository at this point in the history

  5. added test for file not found error

    @shashigharti
    shashigharti committed Jun 7, 2022
    Configuration menu
    Copy the full SHA
    b4fc472 View commit details
    Browse the repository at this point in the history

Commits on Jun 8, 2022

  1. refactor validate command 

    * replaced the report table display code with new to_summary function
* fixed failing tests
    @shashigharti
    shashigharti committed Jun 8, 2022
    Configuration menu
    Copy the full SHA
    e936c42 View commit details
    Browse the repository at this point in the history

  2. removed print statement

    @shashigharti
    shashigharti committed Jun 8, 2022
    Configuration menu
    Copy the full SHA
    4a98056 View commit details
    Browse the repository at this point in the history

  3. made validation_summary part of the report class

    @shashigharti
    shashigharti committed Jun 8, 2022
    Configuration menu
    Copy the full SHA
    6a881ec View commit details
    Browse the repository at this point in the history

  4. fixes to failing test 

    * simplified test output for multiline text
    @shashigharti
    shashigharti committed Jun 8, 2022
    Configuration menu
    Copy the full SHA
    b6dd72a View commit details
    Browse the repository at this point in the history

Commits on Jun 9, 2022

  1. * Refactored display summary code in report.py class/file. Added vali… 

    …dation_summary as to_summary to report task class

* Changed the code of report.to_summary to use reporttask summary function
* Removed unrelated tests code added in previous commit and added new ones for schema, resource, reporttask and report
* Organized tests files for summary feature
    @shashigharti
    shashigharti committed Jun 9, 2022
    Configuration menu
    Copy the full SHA
    16a05cf View commit details
    Browse the repository at this point in the history

  2. fixes for failing tests

    @shashigharti
    shashigharti committed Jun 9, 2022
    Configuration menu
    Copy the full SHA
    2b62dc1 View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into 1095/adds-default-cli-commands

    @shashigharti
    shashigharti authored Jun 9, 2022
    Configuration menu
    Copy the full SHA
    84612b6 View commit details
    Browse the repository at this point in the history