NightOwl888 commented on code in PR #838:
URL: https://github.com/apache/lucenenet/pull/838#discussion_r1174500169


##########
src/Lucene.Net.Tests.Facet/Taxonomy/TestTaxonomyCombined.cs:
##########
@@ -784,7 +784,6 @@ public virtual void TestChildrenArraysGrowth()
 
         // Test that getParentArrays is valid when retrieved during refresh
         [Test]
-        [Slow]

Review Comment:
   Let's keep the `[Slow]` attribute since this is almost always over 4 seconds 
on my machine even after the improvement.
   
   This can be used to get a faster run by setting the system property `tests: 
slow` to `false` to exclude the tests that are marked with this attribute. We 
currently only use that for the CLI installer test in GitHub actions, but it is 
also possible to set in Azure DevOps by passing in `RunSlowTests` as `false`.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to