Skip to content

Adding suite check API & performance improvements #783

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

Merged
merged 34 commits into from
Nov 18, 2018

Conversation

jgebal
Copy link
Member

@jgebal jgebal commented Nov 17, 2018

Added suite level cache as a complementary cache on top of annotations cache.
Improved performance of suite-builder by using IN OUT NOCOPY and avoiding large data variable copying/assignments.
Minimized the amount of data scanned after package compilation.

We now join to all_objects only when executing user doesn't have execute any and is not owner of test packages.

Resolves #778

Reworked output buffer and reporters for performance (we now do bulk inserts into buffer where applicable) - resolves #777

jgebal added 24 commits November 4, 2018 01:36
…he infamous ORA-00600

`ORA-00600: internal error code, arguments: [pfrobj.c: invalid RTTI for Object], [], [], [], [], [], [], [], [], [], [], []``
Refactored lots of code for suite parsing/building.
(Acceptance) tests need rework due to changes in test execution ordering.
One problem to be solved. Oracle nested tables are **unordered!** and we need an ordered list of before/after items that contain CLOB column, so cannot be varray.
Added `seq_no`, numbering before storage and sorting after retrieval.
Added static schema name rather than bind variable.
Adding new function to return list of suites and tests.
Changed functions to procedures to avoid copying of large memmory segments.
…ormation based on info suite cache.

Suite cache is refreshed when calling procedure (as needed).
…s not used uin prior versions.

Added functions `is_test`, `is_suite`, `has_suite` to perform checks for test/suite existance at prcedure, package, schema level.
The script will now automatically copy `*.jar` files into `/utPLSQL-cli/lib` directory.
@jgebal jgebal added this to the v3.1.3 milestone Nov 17, 2018
Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 6

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  2. Code Smell Code Smell: The procedure "run" should be refactored to take at most 10 parameters instead of 11 as currently. (more)
  3. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  4. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 6

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: The procedure "run" should be refactored to take at most 10 parameters instead of 11 as currently. (more)
  2. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  3. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  4. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 6

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: The procedure "run" should be refactored to take at most 10 parameters instead of 11 as currently. (more)
  2. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  3. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  4. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 6

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  2. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  3. Code Smell Code Smell: The procedure "run" should be refactored to take at most 10 parameters instead of 11 as currently. (more)
  4. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 5

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  2. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  3. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 5

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  2. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  3. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 5

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  2. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  3. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Copy link

@utplsql-travis utplsql-travis left a comment

Choose a reason for hiding this comment

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

SonarQube analysis found issues:
Bug Bugs: 0
Vulnerability Vulnerabilities: 0
Code Smell Code Smells: 5

Including the following issue(s) which could not be reported in line:

  1. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  2. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)
  3. Code Smell Code Smell: Complete the task associated to this "TODO" comment. (more)

See all issues in SonarCloud

Added additional tests to:
- cover functionality of reporting warnings on transaction invalidation with sub-suites.
- cover functionality of failing a test in sub-suite when beforeall fails in parent suite with logical suites in between.
Refactored `ut_teamcity_reporter` a bit.
@jgebal jgebal force-pushed the feature/suite_query_api branch from 26eb3ca to 74ec9a1 Compare November 18, 2018 15:03
@jgebal jgebal merged commit 3a1227e into develop Nov 18, 2018
@jgebal jgebal deleted the feature/suite_query_api branch November 18, 2018 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants