Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

test: add dcc sign bytes test #71

Merged
merged 2 commits into from
Jan 27, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Jan 26, 2023

Overview

Closes: #52

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

@rach-id rach-id requested a review from rahulghangas January 26, 2023 13:11
@rach-id rach-id self-assigned this Jan 26, 2023
@rach-id rach-id requested a review from evan-forbes as a code owner January 26, 2023 13:11
rahulghangas
rahulghangas previously approved these changes Jan 27, 2023
types/data_commitment_confirm_test.go Outdated Show resolved Hide resolved
evan-forbes
evan-forbes previously approved these changes Jan 27, 2023
@codecov-commenter
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (main@74c53e2). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #71   +/-   ##
=======================================
  Coverage        ?   17.52%           
=======================================
  Files           ?        2           
  Lines           ?       97           
  Branches        ?        0           
=======================================
  Hits            ?       17           
  Misses          ?       79           
  Partials        ?        1           

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

@rach-id
Copy link
Member Author

rach-id commented Jan 27, 2023

Merging even if linter is not passing as it will be fixed in this PR #80

@rach-id rach-id merged commit 16eb6c5 into celestiaorg:main Jan 27, 2023
@rach-id rach-id deleted the add_dcc_sign_bytes_test branch January 27, 2023 10:44
rach-id added a commit to rach-id/orchestrator-relayer that referenced this pull request Jan 31, 2023
* test: add dcc sign bytes test

* chore: rename byt to buf as suggested by @rahulghangas
rach-id added a commit that referenced this pull request Jan 31, 2023
* feat: add base DHT implementation and test node

* chore: rename test network to test DHT network

* Update p2p/dht.go

* Update p2p/dht.go

* chore: rename DHTNetwork to TestDHTNetwork

* ci: add markdown lint and run lint on PRs (#67)

* ci: add markdown lint

* ci: update lint to run on pull request

* test: add dcc sign bytes test (#71)

* test: add dcc sign bytes test

* chore: rename byt to buf as suggested by @rahulghangas

* ci: use go version 1.19 in lint (#68)

* chore: changes for golang ci lint to pass (#80)

* chore: chore add nolint to parameters that will be used later

* chore: chore add nolint to parameters that will be used later

* chore: comment unused code for CI to pass

* chore: rename commands for revive to pass

* chore(deps): bump goreleaser/goreleaser-action from 4.1.0 to 4.1.1 (#83)

Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 4.1.0 to 4.1.1.
- [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
- [Commits](goreleaser/goreleaser-action@v4.1.0...v4.1.1)

---
updated-dependencies:
- dependency-name: goreleaser/goreleaser-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* ci: update golang ci lint version (#82)

* ci: only run `split-test-files` if go files change (#77)

* feat: add copied relayer implementation (#84)

* feat: add relayer implementation

* chore: refactor relayer to have start and stop methods

* chore(deps): bump actions/cache from 3.2.3 to 3.2.4 (#86)

Bumps [actions/cache](https://github.com/actions/cache) from 3.2.3 to 3.2.4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](actions/cache@v3.2.3...v3.2.4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): bump goreleaser/goreleaser-action from 4.1.1 to 4.2.0 (#87)

Bumps [goreleaser/goreleaser-action](https://github.com/goreleaser/goreleaser-action) from 4.1.1 to 4.2.0.
- [Release notes](https://github.com/goreleaser/goreleaser-action/releases)
- [Commits](goreleaser/goreleaser-action@v4.1.1...v4.2.0)

---
updated-dependencies:
- dependency-name: goreleaser/goreleaser-action
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat: update confirms types and adds marshallers/unmarshallers (#70)

* feat: update confirms types and adds marshallers/unmarshallers

* chore: fix conflicts

* chore: go mod tidy

* chore: increase the DHT sleep to 10ms

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add unit tests for the attestations confirms
4 participants