[ 
https://issues.apache.org/jira/browse/CASSANDRA-15956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17162034#comment-17162034
 ] 

Berenguer Blasi commented on CASSANDRA-15956:
---------------------------------------------

At the end of the day this one turned out to be shorter than expected. Most 
tools have some seed testing which has been enhanced or added as a result of 
this ticket and CASSANDRA-15942. It looks like no developer is going to have an 
excuse not to add tests, be it unit or dtest, to any PR touching tooling.

Coverage is a different story where tools mostly seem to have only minimal 
testing. Some like nodetool have extensive testing on some commands but none on 
others. Fqltool i.e. has some testing particular to itself, etc.

Overall I consider what we have at this point good and now it's a matter of 
discussing how to best start adding more test cases.

> 15583 - Ensure tooling surface area coverage
> --------------------------------------------
>
>                 Key: CASSANDRA-15956
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-15956
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Test/unit
>            Reporter: Berenguer Blasi
>            Assignee: Berenguer Blasi
>            Priority: Normal
>             Fix For: 4.0-beta
>
>
> Given CASSANDRA-15942 POC seems ok so far. Now we'll extend that to other 
> tools and scenarios in a width first approach to make sure enough surface 
> area is covered. This should suffice to confirm our framework should enable 
> devs to test any tools when needed.
> Tools: https://cassandra.apache.org/doc/latest/tools/index.html
> ||Tool||Easy UT possible||UT Sample||Easy dtest possible||Has 
> dtest||Comments||
> |Nodetool|(/) Added/Enhanced|(/) Added/enhanced|(/)|(/)|Use UT if possible. 
> Otherwise dtests if you need a full multinode env|
> |Cqlsh|(/) Added|(/) Added|(/)|(/)|Easy Cqlsh UT is now possible and an 
> example is included|
> |Cassandra-stress|(/) Added and existing|(/) Added|(/)|(x)|
> |debug-cql|(/)|(x)|(/)|(x)|
> |fqltool|(/) Existing|(/) Existing|(/)|(/)|
> |auditlogviewer|(/) Existing|(/) Existing|(/)|(/)|
> |*Sstable utilities*||||||
> |sstabledump|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstableexpiredblockers|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstablelevelreset|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstableloader|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstablemetadata|(/) enhanced|(/) enhanced|(/)|(!)|Ran in dtests, no 
> dedicated test|
> |sstableofflinerelevel|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstablerepairedset|(/) enhanced|(/) enhanced|(/)|(!)|Ran in dtests, no 
> dedicated test|
> |sstablescrub|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstablesplit|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstableupgrade|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstableutil|(/) enhanced|(/) enhanced|(/)|(/)||
> |sstableverify|(/) enhanced|(/) enhanced|(/)|(/)||



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to