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.

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

[Testing] Testing Util #91

Merged
merged 2 commits into from
Jun 26, 2021
Merged

[Testing] Testing Util #91

merged 2 commits into from
Jun 26, 2021

Conversation

hopeman15
Copy link
Contributor

Description

This moves the TestObject to its own module in preparation for #74, so we can update the TestObject accordingly and the tests will run in both the model forge and the additional android project.

Review checklist

  • PR is split into meaningful commits for the ease of reviewing
  • [-] Description contains clear instructions on how to test the feature (where applicable)
  • [-] Tests have been written
  • Appropriate labels have been applied

@codecov-commenter
Copy link

Codecov Report

Merging #91 (68b2c06) into main (5ab99b2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              main     #91   +/-   ##
=======================================
  Coverage     90.9%   90.9%           
  Complexity      55      55           
=======================================
  Files           16      16           
  Lines          111     111           
  Branches        18      18           
=======================================
  Hits           101     101           
  Misses           5       5           
  Partials         5       5           

@hopeman15 hopeman15 merged commit 752ca3f into main Jun 26, 2021
@hopeman15 hopeman15 deleted the testing/testing-module branch June 26, 2021 08:01
hopeman15 added a commit that referenced this pull request Jul 13, 2021
* [Testing] Testing Util (#91)

* [Testing] Testing Util

* ignore test-utils in code coverage reports

* forge-core

* forge-test-utils

* report

* xml report

* local publishing

* set artifact id
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants