Opened 7 years ago

Closed 6 years ago

#1689 closed enhancement (worksforme)

Systemtest_rename rasql's test cases homogeneously

Reported by: Bang Pham Huu Owned by: Dimitar Misev
Priority: minor Milestone: Future
Component: systemtest Version: development
Keywords: Cc: Dimitar Misev
Complexity: Medium

Description (last modified by Bang Pham Huu)

In systemtest/testcases_mandatory (e.g: http://rasdaman.org/browser/systemtest/testcases_mandatory/test_select/queries)

the test queries are not homogeneous with systemtest/testcases_services (e.g: http://rasdaman.org/browser/systemtest/testcases_services/test_wcps/queries)

They should have:
+ Test number prefix (e.g: 001-arcosbig.rasql).
+ Test errors needs .error.rasql to recognize (current I need to check oracle file to know this test is error, not only in my system).

@Dragi: The ticket is about checking rasdaman/systemtest with any queries folders and add the prefix with increasing numbers if they don't exist, e.g:

systemtest/testcases_mandatory/test_clipping/queries/cline_bothvertout.rasql

-> 

systemtest/testcases_mandatory/test_clipping/queries/001-cline_bothvertout.rasql

NOTE: if you change test files in queries folder (e.g: test_clipping/), check if test files also listed in test_clipping/known_fails and rename them accordingly.

and this is more frustrating, check folder oracles of test cases without prefix numbers and see which oracles are expected error cases. Add ".error." before the file extension for the corresponding query in queries folder and oracle file in oracle folder, e.g:

Open this file, it is an oracle with expected error from Rasql query, but there is no way to tell it is working query or non working query from its name:

systemtest/testcases_mandatory/test_clipping/oracle/e_clinestr_rg_card1.oracle

rename to (note the prefix increasing number)

systemtest/testcases_mandatory/test_clipping/oracle/001-e_clinestr_rg_card1.error.oracle

and 


systemtest/testcases_mandatory/test_clipping/queries/001-e_clinestr_rg_card1.error.rasql

Change History (7)

comment:1 by Vlad Merticariu, 7 years ago

Milestone: 9.5Future

comment:2 by Bang Pham Huu, 6 years ago

Cc: bbell added
Description: modified (diff)
Milestone: Future9.7
Owner: changed from bbell to dkamov
Status: newassigned

comment:3 by Bang Pham Huu, 6 years ago

Description: modified (diff)

comment:4 by Dimitar Misev, 6 years ago

Milestone: 9.7Future
Owner: changed from dkamov to Dimitar Misev
Priority: majorminor

comment:5 by Dimitar Misev, 6 years ago

Cc: bbell removed

They should have:
+ Test number prefix (e.g: 001-arcosbig.rasql).

I'm not sure if this is a good idea anymore honestly. Without the number prefix it's possible to group queries by prefix (e.g. a bunch of different tests for arccos-big.rasql, arccos-small.rasql, etc). With the numbers we lose this grouping.

What's the advantage of this number prefix? Order of query evaluation is fixed. Although this can be emulated easily within groups where it could matter, e.g.

  • arccos-01-create_coll.rasql
  • arccos-02-insert_data.rasql
  • arccos-03-query.rasql
  • arccos-04-drop_coll.rasql

+ Test errors needs .error.rasql to recognize (current I need to check oracle file to know this test is error, not only in my system).

Not sure why is this important? If the test passes it's all fine, irrelevant of whether the oracle contains an error or not. If it doesn't pass then again you certainly need to inspect the current oracle to check what's the issue with the output.

For test_clipping though I think Brennan encoded these error cases with a e_ prefix (instead of this .error suffix)

Anyway, I'd close this ticket as _wontfix_, ok with you Bang?

Version 0, edited 6 years ago by Dimitar Misev (next)

comment:6 by Bang Pham Huu, 6 years ago

The main benefit is I just need the preifix number when one test is failed, I can open test query or test oraclr or test output quickly.
It is very inconvenient when searching a test with more or less same names.

It is best to make them with patterns and a student can do it.

comment:7 by Dimitar Misev, 6 years ago

Resolution: worksforme
Status: assignedclosed

I'm not convinced by your reason, will close this ticket.

Note: See TracTickets for help on using tickets.