Skip to content

Conversation

@nastra
Copy link
Contributor

@nastra nastra commented Oct 7, 2021

While working on #2273 I noticed that assigning a branch/tag to a fresh main without any commits didn't work in 0.9.2, thus adding some tests to make sure this works in the latest Nessie version

@nastra nastra requested a review from snazy October 7, 2021 10:46
@codecov
Copy link

codecov bot commented Oct 7, 2021

Codecov Report

Merging #2294 (36e9144) into main (d835229) will increase coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #2294      +/-   ##
============================================
+ Coverage     83.76%   83.83%   +0.06%     
- Complexity     1843     1847       +4     
============================================
  Files           237      237              
  Lines         10461    10496      +35     
  Branches        734      734              
============================================
+ Hits           8763     8799      +36     
  Misses         1393     1393              
+ Partials        305      304       -1     
Flag Coverage Δ
java 83.44% <100.00%> (+0.07%) ⬆️
javascript 86.61% <ø> (ø)
python 87.13% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...java/org/projectnessie/jaxrs/AbstractTestRest.java 95.15% <100.00%> (+0.08%) ⬆️
...sioned/persist/tests/AbstractVersionStoreTest.java 96.40% <100.00%> (+0.20%) ⬆️
...nessie/versioned/persist/tx/TxDatabaseAdapter.java 75.00% <0.00%> (+0.21%) ⬆️

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 d835229...36e9144. Read the comment docs.

@nastra nastra merged commit 404ff50 into projectnessie:main Oct 7, 2021
@nastra nastra deleted the assign-ref-to-fresh-main branch October 7, 2021 11:11
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

Successfully merging this pull request may close these issues.

2 participants