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

Run PT with JUnit #132

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

ssheikin
Copy link

No description provided.

@cla-bot cla-bot bot added the cla-signed label Nov 12, 2024
@ssheikin ssheikin force-pushed the ssheikin/38/tempto/pt-junit branch 10 times, most recently from f0f0e4a to 789b0f3 Compare November 15, 2024 17:21
Requiremets and method requirements are disabled for all tests for now.
Assume requirements are not needed for the test.

Requirement
CommandRequirement - is not used now
TableRequirement - most probably can be substituted by
@before.., @after..
LdapObjectRequirement - to find out how to deal with it.

As time shows, method level requirements are not used in the new code,
and most probably, were implemented for
`Convention based SQL query tests`
@ssheikin ssheikin force-pushed the ssheikin/38/tempto/pt-junit branch from 789b0f3 to 08b9a5f Compare November 16, 2024 09:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

1 participant