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

how is this maintained? all this bugs :( #293

Open
jnormen opened this issue Sep 3, 2024 · 3 comments
Open

how is this maintained? all this bugs :( #293

jnormen opened this issue Sep 3, 2024 · 3 comments
Labels
triage/needs-information Indicates an issue needs more information in order to work on it. waitforfeedback

Comments

@jnormen
Copy link

jnormen commented Sep 3, 2024

I have so many issues with this library.
If not old bugs, there are new once.

Sorry I'm so negative about this framework, I throw it out... it had potential, but more issues than it bring values...

But there are bugs I have reported month ago that are not fixed. :/ like the Any Attribute issue.

@fgather
Copy link
Contributor

fgather commented Sep 3, 2024

@jnormen I'm really sorry that you are not happy with the latest release. Could you give some example that helps us to reproduce your issues?
I was only able to find one of your bug reports #257, which was closed in May, as the original description could not be reproduced. As it was never reopened after you provided examples, we probably did not see them. However, I'll answer quickly there, as it was very likely an issue in the way the assembly was loaded.

@fgather fgather added waitforfeedback triage/needs-information Indicates an issue needs more information in order to work on it. labels Sep 3, 2024
@jnormen
Copy link
Author

jnormen commented Sep 3, 2024

It start to complain that it does not find any Abstract classes.
I check if I have abstract and not sealed then the classes shall end with "Base" but it suddenly it does not find anything.

for now I added .WithoutRequiringPositiveResults()

So I at least can release my services :) I will check in to this in more detail later on.

Not sure if there is a cache issue either. I have seen strange things with roslyn etc...

@alexanderlinne
Copy link
Collaborator

This is expected due to a breaking change in 0.11.0 and was documented in the release notes. If this test now fails it means that the predicates never matched any types in your architecture. With the new release, if you don't use .WithoutRequiringPositiveResults(), each test is required to have its conditions checked agains at least one type successfully, otherwise the test is considered failed. This protects against tests that do not actually test anything.

If you suspect a bug here, i.e. that your test should find abstract classes in your architecture, feel free to provide an example that reproduces your issue such that we can further investigate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triage/needs-information Indicates an issue needs more information in order to work on it. waitforfeedback
Projects
None yet
Development

No branches or pull requests

3 participants