Skip to content

test: add test for '__repr__' in SpecEvaluator#499

Merged
camilamaia merged 1 commit intomainfrom
test_spec_evaluator_repr
Aug 20, 2021
Merged

test: add test for '__repr__' in SpecEvaluator#499
camilamaia merged 1 commit intomainfrom
test_spec_evaluator_repr

Conversation

@rafaquelhodev
Copy link
Copy Markdown
Contributor

@rafaquelhodev rafaquelhodev commented Aug 20, 2021

Description

Add unit tests for method __repr__ of SpecEvaluator class

Motivation behind this PR?

Closes issue #486

What type of change is this?

Unit testing

Checklist

  • I have added a changelog entry / my PR does not need a new changelog entry. Instructions.
  • I have added/updated unit tests. Instructions.
  • New and existing unit tests pass locally with my changes. Instructions
  • I have self-documented code my changes by adding docstring(s) and comment(s). Instructions
  • Current PR does not significantly decrease the code coverage and docstring coverage.
  • My code follows the style guidelines of this project.
  • I have run ScanAPI locally and manually tested my changes. Instructions.
  • I have squashed my commits. Instructions.

Issue

Closes #486

@rafaquelhodev rafaquelhodev requested review from a team as code owners August 20, 2021 00:04
@github-actions
Copy link
Copy Markdown

github-actions bot commented Aug 20, 2021

@rafaquelhodev your pull request is missing a changelog!

camilamaia
camilamaia approved these changes Aug 20, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@camilamaia camilamaia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⭐️

@camilamaia camilamaia merged commit 92d0b9e into main Aug 20, 2021
@camilamaia camilamaia deleted the test_spec_evaluator_repr branch August 20, 2021 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@camilamaia camilamaia camilamaia approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add unit tests for the __repr__ method of the SpecEvaluator class

2 participants

@rafaquelhodev @camilamaia