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

Test Anotations over encrytped Tests #160

Open
fjsosa opened this issue Feb 18, 2022 · 3 comments
Open

Test Anotations over encrytped Tests #160

fjsosa opened this issue Feb 18, 2022 · 3 comments

Comments

@fjsosa
Copy link

fjsosa commented Feb 18, 2022

Hi, we had some tests marked with --@tSQLt:SkipTest, now this tests are encrypted (with: WITH ENCRYPTION AS)
now it seems that tsqlt is ignoring the annotations, is this an expected behavior? thank so much.

I was checking the CLR code involved and it seems that the only way to detect annotations like --@tSQLt:SkipTest is by analizing the source code of each SP. It seems that is not possible to work with annotations and encrypted test. Could you please confirm this?

If this is the case, could we, in the future, optionally use Extended Properties to add this kind of annotations?

@mbt1
Copy link
Collaborator

mbt1 commented Feb 27, 2022

@fjsosa, I've been thinking about how to address this. There are a few options that could be implemented. But, could you help me understand your use case for encrypted test case stored procedures? To me, a unit test also serves as documentation, so encrypting it seems counterintuitive.

@fjsosa
Copy link
Author

fjsosa commented Mar 2, 2022

Hi, thanks for you answer, for us, some tests goes to the customer db, we use that to monitor / check the product health, but, some of those test can reveal some parts of our code we would like to keep hidden for the customers. We have plans to implement the tSQLt annotations as ExtendedProperties.

@mbt1
Copy link
Collaborator

mbt1 commented Jun 1, 2022

@fiosa, have you had any luck with implementing Extended Properties?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants