Skip to content

test: empty tree#149

Merged
rootulp merged 2 commits intocelestiaorg:mainfrom
andrijamitrovic23:andrija/newTests
Aug 18, 2025
Merged

test: empty tree#149
rootulp merged 2 commits intocelestiaorg:mainfrom
andrijamitrovic23:andrija/newTests

Conversation

@andrijamitrovic23
Copy link
Collaborator

@andrijamitrovic23 andrijamitrovic23 commented Mar 22, 2023

Overview

This is a PR for addition of new test cases that have been noticed to be missing through the audit of the nmt repository. The intention is to increase the number of test cases by adding tests for edge cases and for the issues that have been reported.

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@andrijamitrovic23 andrijamitrovic23 force-pushed the andrija/newTests branch 2 times, most recently from 9e1abe7 to 5f30795 Compare March 24, 2023 13:19
@codecov
Copy link

codecov bot commented Mar 24, 2023

Codecov Report

Merging #149 (5f30795) into master (eb012d0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #149   +/-   ##
=======================================
  Coverage   96.20%   96.20%           
=======================================
  Files           6        6           
  Lines         527      527           
=======================================
  Hits          507      507           
  Misses         12       12           
  Partials        8        8           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rootulp
Copy link
Collaborator

rootulp commented Aug 18, 2025

Test passes so seems safe to merge.

@rootulp rootulp marked this pull request as ready for review August 18, 2025 10:11
@rootulp rootulp enabled auto-merge (squash) August 18, 2025 10:11
@rootulp rootulp requested a review from rach-id August 18, 2025 10:11
@rootulp
Copy link
Collaborator

rootulp commented Aug 18, 2025

golangci-lint is fixed in #294

@rach-id rach-id changed the title Empty Tree test case test: Empty Tree test case Aug 18, 2025
@rootulp rootulp changed the title test: Empty Tree test case test: empty tree Aug 18, 2025
@rootulp rootulp merged commit 9d5d148 into celestiaorg:main Aug 18, 2025
4 of 5 checks passed
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.

3 participants