Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use standalone EmbeddedSolrServer for Solr integration tests #1368

Conversation

adamyy
Copy link
Contributor

@adamyy adamyy commented Sep 6, 2020

As described in #1367, EmbeddedSolrServerTestBase enforces specific PRNG algorithm to be used (or not used) by Java SecureRandom. This PR uses EmbeddedSolrServer by itself in tests to bypass this check.

@codecov
Copy link

codecov bot commented Sep 6, 2020

Codecov Report

Merging #1368 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1368      +/-   ##
============================================
- Coverage     53.66%   53.57%   -0.10%     
+ Complexity      829      823       -6     
============================================
  Files           154      154              
  Lines          8627     8627              
  Branches       1224     1224              
============================================
- Hits           4630     4622       -8     
- Misses         3618     3627       +9     
+ Partials        379      378       -1     
Impacted Files Coverage Δ Complexity Δ
...java/io/anserini/ltr/feature/CountBigramPairs.java 83.11% <0.00%> (-10.39%) 29.00% <0.00%> (-6.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9459f5f...18b788e. Read the comment docs.

@lintool lintool self-requested a review September 7, 2020 13:55
@lintool lintool merged commit e16b3c1 into castorini:master Sep 7, 2020
@lintool
Copy link
Member

lintool commented Sep 7, 2020

This is great, thanks!

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

Successfully merging this pull request may close these issues.

2 participants