Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

Use classes to build a suite #156

Merged
merged 2 commits into from
Jan 14, 2019
Merged

Use classes to build a suite #156

merged 2 commits into from
Jan 14, 2019

Conversation

extsoft
Copy link
Member

@extsoft extsoft commented Jan 14, 2019

Implements #137

The change is made to build more obvious and unambiguous namespace.

The rename was made using deprecation mechanism. This means the old
class is still available, however, a new one is used in the code.

#137
Now there are two suites (SuiteFromClasses and SuiteFromFileSystem)
which can represent a SunshineSuite. SuiteFromClasses allows creation
a test suite based on predefined classes. Any of these classes can be
used while a custom suite (specific for a test runner) is build.

Also, TestFromClass is created as we need to represent a class as an
instance of SunshineTest.

All relevant documentation and unit tests are updated.

#137
@codecov
Copy link

codecov bot commented Jan 14, 2019

Codecov Report

Merging #156 into master will increase coverage by 1.98%.
The diff coverage is 88.46%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #156      +/-   ##
============================================
+ Coverage     70.45%   72.43%   +1.98%     
- Complexity      115      127      +12     
============================================
  Files            47       50       +3     
  Lines           511      526      +15     
  Branches          7        7              
============================================
+ Hits            360      381      +21     
+ Misses          147      141       -6     
  Partials          4        4
Impacted Files Coverage Δ Complexity Δ
...a/org/tatools/sunshine/core/SunshineSuiteBase.java 100% <100%> (+100%) 2 <1> (+2) ⬆️
...n/java/org/tatools/sunshine/testng/TestNGTest.java 75% <100%> (ø) 3 <1> (ø) ⬇️
...va/org/tatools/sunshine/core/SuiteFromClasses.java 100% <100%> (ø) 2 <2> (?)
...n/java/org/tatools/sunshine/core/TestFromFile.java 66.66% <100%> (ø) 4 <1> (?)
...va/org/tatools/sunshine/core/SunshineTestBase.java 71.42% <100%> (ø) 3 <0> (ø) ⬇️
...g/tatools/sunshine/testng/LoadableTestNGSuite.java 72.5% <100%> (ø) 12 <1> (ø) ⬇️
...n/java/org/tatools/sunshine/junit4/JunitSuite.java 60% <100%> (ø) 4 <1> (ø) ⬇️
...org/tatools/sunshine/core/SuiteFromFileSystem.java 75% <75%> (ø) 3 <3> (?)
.../java/org/tatools/sunshine/core/TestFromClass.java 85.71% <85.71%> (ø) 4 <4> (?)
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e33cba4...c62c73b. Read the comment docs.

@extsoft
Copy link
Member Author

extsoft commented Jan 14, 2019

@rultor please merge.

@rultor
Copy link
Collaborator

rultor commented Jan 14, 2019

@rultor please merge.

@extsoft OK, I'll try to merge now. You can check the progress of the merge here

@rultor rultor merged commit c62c73b into master Jan 14, 2019
@rultor
Copy link
Collaborator

rultor commented Jan 14, 2019

@rultor please merge.

@extsoft Done! FYI, the full log is here (took me 3min)

@extsoft extsoft deleted the 137 branch January 14, 2019 08:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants