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

Meta: Add Support unit tests #920

Open
3 of 19 tasks
paulirwin opened this issue Feb 25, 2024 · 1 comment
Open
3 of 19 tasks

Meta: Add Support unit tests #920

paulirwin opened this issue Feb 25, 2024 · 1 comment
Labels
is:task A chore to be done pri:normal testability up-for-grabs This issue is open to be worked on by anyone

Comments

@paulirwin
Copy link
Contributor

paulirwin commented Feb 25, 2024

Is there an existing issue for this?

  • I have searched the existing issues

Task description

This is a meta-issue that should be broken out into separate issues.

From #914 via @NightOwl888:

I also noticed we don't have tests for the following types in the Support folder:

@paulirwin paulirwin added the is:task A chore to be done label Feb 25, 2024
@NightOwl888
Copy link
Contributor

@paulirwin - Perhaps eliminating ConcurrentDictionaryWrapper<TKey, TValue> and friends should be a separate issue. Alternatively, it could be rolled into #259. If there is a need for a "ConcurrentLinkedDictionary" in later versions of Lucene, perhaps the implementation should be tagged or put on its own branch so we can recover it later.

@NightOwl888 NightOwl888 added up-for-grabs This issue is open to be worked on by anyone testability pri:normal labels Mar 10, 2024
NightOwl888 added a commit to NightOwl888/lucenenet that referenced this issue Mar 10, 2024
…a ref struct to eliminate the heap allocation. Converted TestControlledRealTimeReopenThread.TestStraightForwardDemonstration() to test using ReferenceContext<T> to verify functionality. See apache#920.
NightOwl888 added a commit that referenced this issue Mar 10, 2024
…a ref struct to eliminate the heap allocation. Converted TestControlledRealTimeReopenThread.TestStraightForwardDemonstration() to test using ReferenceContext<T> to verify functionality. See #920.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
is:task A chore to be done pri:normal testability up-for-grabs This issue is open to be worked on by anyone
Projects
None yet
Development

No branches or pull requests

2 participants