Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

feat: add mock app docs #607

Merged
merged 5 commits into from
Aug 11, 2023
Merged

feat: add mock app docs #607

merged 5 commits into from
Aug 11, 2023

Conversation

ibrizsabin
Copy link
Collaborator

Description:

Commit Message

feat: add  mock app  docs

see the guidelines for commit messages.

Changelog Entry

version: <log entry>

Checklist:

  • I have performed a self-review of my own code
  • I have documented my code in accordance with the documentation guidelines
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the unit tests
  • I only have one commit (if not, squash them into one commit).
  • I have a descriptive commit message that adheres to the commit message guidelines

Please review the CONTRIBUTING.md file for detailed contributing guidelines.

@codecov-commenter
Copy link

codecov-commenter commented Aug 9, 2023 •
edited
Loading

Codecov Report

Merging #607 (9b28bda) into main (a1bbe8d) will decrease coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #607      +/-   ##
============================================
- Coverage     68.32%   68.27%   -0.06%     
+ Complexity      408      402       -6     
============================================
  Files           129      129              
  Lines         12626    12606      -20     
  Branches        294      290       -4     
============================================
- Hits           8627     8607      -20     
  Misses         3843     3843              
  Partials        156      156
Flag Coverage Δ
rust 66.10% <ø> (ø)

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

see 2 files with indirect coverage changes

@ibrizsabin ibrizsabin marked this pull request as ready for review August 9, 2023 09:56
@ibrizsabin ibrizsabin requested a review from redlarva as a code owner August 9, 2023 09:56
@ibrizsabin ibrizsabin linked an issue Aug 10, 2023 that may be closed by this pull request
Add docs for mock ibc app #613
Closed
AntonAndell
AntonAndell requested changes Aug 10, 2023
View reviewed changes
Copy link
Collaborator

@AntonAndell AntonAndell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good but mabye add a link to it with a short explination in a outer readme so you can find it

@ibrizsabin ibrizsabin requested a review from AntonAndell August 10, 2023 10:59
srmaharjan
srmaharjan approved these changes Aug 10, 2023
View reviewed changes
Copy link

@srmaharjan srmaharjan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@ibrizsabin ibrizsabin merged commit c71475f into main Aug 11, 2023
@ibrizsabin ibrizsabin deleted the feat/mock-dapp-docs branch August 11, 2023 04:23
redlarva added a commit that referenced this pull request Aug 17, 2023
@debendraoli @ibrizsabin
@sabinchitrakar @AntonAndell @viveksharmapoudel @sdpisreddevil @nightowl121 @redlarva @DeepakBomjan
Merge branch 'main' into fix/merge-conflicts (#632)
0f617a2 
* feat: add range queries (#595)

* fix: add packet heights

* feat: add packet receipt

* fix: add prev consensus state query

* chore: build pass

* fix: return type as map

* chore: fix lint

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>

* feat: Add Range query for packet heights and receipts (#594)

For each packet sent store the height at it which it was sent and clear it as packets are acknowledged. Add range query to get all missing packets receipts in a range.

Co-authored-by: viveksharmapoudel <viveksharmapoudel@gmail.com>

* fix: Use string keys for return map in getPacketHeights (#608)

* feat: e2e testing on package size (#606)

* Add tests for MaxSizeLimit from Chain A to ChainB and vice-versa

* return error message(icon) on `ExecuteContract()`

* docs: add test badges for java and rust (#602)

Signed-off-by: Night Owl <nightowl121@protonmail.com>

* feat: updated code owner lists (#610)

* docs: add cicd documentation (#603)

* docs: add cicd documentation

* docs: rename file

* feat: add  mock app  docs (#607)

* feat: add docs

* fix: add write-ack

* fix: add write-ack

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>

* chore: ibc system architecture docs (#615)

* chore: add ibc design docs.

* chore: remove relay.md

* chore: more about ibc architecture

* feat: update docs (#614)

* feat: update docs

* feat: add java build commands

* feat: add guidelines

* fix: topic order

* fix: update license name

* Update README.md

Co-authored-by: AntonAndell <anton@andell.eu>

* fix: remove integration test block

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>
Co-authored-by: AntonAndell <anton@andell.eu>

* feat(e2e): relayer tests framework (#578)

* feat: support expanding config value, convert json to yaml

* fix: invalid basepath, fix: correct goloop docker image

* WIP: initial relayer e2e

* fix: client state test

* add: relayer crash and recover

* fix(e2e): test results

* add(e2e): connections test

* fix(e2e): connection and client state tests

* fix(e2e): connection setup

* add(e2e): channel tests

* refactor: integration test setup (#616)

refactor: test setup

---------

Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* docs: add docs for java smart contracts (#622)

Signed-off-by: Night Owl <nightowl121@protonmail.com>

* ci: add cw-xcall-ibc-connection to Codecov (#620)

* rf: linting

---------

Signed-off-by: Night Owl <nightowl121@protonmail.com>
Co-authored-by: ibrizsabin <101165234+ibrizsabin@users.noreply.github.com>
Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>
Co-authored-by: AntonAndell <anton@andell.eu>
Co-authored-by: viveksharmapoudel <viveksharmapoudel@gmail.com>
Co-authored-by: Sudeep Bhandari <bhandari.sudeep68@gmail.com>
Co-authored-by: Night Owl <91905208+nightowl121@users.noreply.github.com>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>
Co-authored-by: DeepakBomjan <44976635+DeepakBomjan@users.noreply.github.com>
ibrizsabin added a commit that referenced this pull request Aug 18, 2023
@debendraoli @ibrizsabin
@sabinchitrakar @AntonAndell @viveksharmapoudel @sdpisreddevil @nightowl121 @redlarva @DeepakBomjan
feat: crash tests (#621)
f6d2e8f 
* feat: support expanding config value, convert json to yaml

* fix: invalid basepath, fix: correct goloop docker image

* WIP: initial relayer e2e

* fix: client state test

* add: relayer crash and recover

* fix(e2e): test results

* add(e2e): connections test

* fix(e2e): connection and client state tests

* fix(e2e): connection setup

* add(e2e): channel tests

* feat: WIP crash tests

* improve: crash tests

* rf: concurrent call

* chore: workpoint

* rf: linting

* rm: concurrent wallet build due to xcall issue

* feat: stop relay without cleaning for reuse purpose

* rf: interface calls for better reusablity

* fix: comarision of sent packet

* feat: ability to save blockheight on relayer stop

* chore: workpoint

* fix: donot crash chain

* Merge branch 'main' into fix/merge-conflicts (#632)

* feat: add range queries (#595)

* fix: add packet heights

* feat: add packet receipt

* fix: add prev consensus state query

* chore: build pass

* fix: return type as map

* chore: fix lint

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>

* feat: Add Range query for packet heights and receipts (#594)

For each packet sent store the height at it which it was sent and clear it as packets are acknowledged. Add range query to get all missing packets receipts in a range.

Co-authored-by: viveksharmapoudel <viveksharmapoudel@gmail.com>

* fix: Use string keys for return map in getPacketHeights (#608)

* feat: e2e testing on package size (#606)

* Add tests for MaxSizeLimit from Chain A to ChainB and vice-versa

* return error message(icon) on `ExecuteContract()`

* docs: add test badges for java and rust (#602)

Signed-off-by: Night Owl <nightowl121@protonmail.com>

* feat: updated code owner lists (#610)

* docs: add cicd documentation (#603)

* docs: add cicd documentation

* docs: rename file

* feat: add  mock app  docs (#607)

* feat: add docs

* fix: add write-ack

* fix: add write-ack

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>

* chore: ibc system architecture docs (#615)

* chore: add ibc design docs.

* chore: remove relay.md

* chore: more about ibc architecture

* feat: update docs (#614)

* feat: update docs

* feat: add java build commands

* feat: add guidelines

* fix: topic order

* fix: update license name

* Update README.md

Co-authored-by: AntonAndell <anton@andell.eu>

* fix: remove integration test block

---------

Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>
Co-authored-by: AntonAndell <anton@andell.eu>

* feat(e2e): relayer tests framework (#578)

* feat: support expanding config value, convert json to yaml

* fix: invalid basepath, fix: correct goloop docker image

* WIP: initial relayer e2e

* fix: client state test

* add: relayer crash and recover

* fix(e2e): test results

* add(e2e): connections test

* fix(e2e): connection and client state tests

* fix(e2e): connection setup

* add(e2e): channel tests

* refactor: integration test setup (#616)

refactor: test setup

---------

Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>

* docs: add docs for java smart contracts (#622)

Signed-off-by: Night Owl <nightowl121@protonmail.com>

* ci: add cw-xcall-ibc-connection to Codecov (#620)

* rf: linting

---------

Signed-off-by: Night Owl <nightowl121@protonmail.com>
Co-authored-by: ibrizsabin <101165234+ibrizsabin@users.noreply.github.com>
Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>
Co-authored-by: AntonAndell <anton@andell.eu>
Co-authored-by: viveksharmapoudel <viveksharmapoudel@gmail.com>
Co-authored-by: Sudeep Bhandari <bhandari.sudeep68@gmail.com>
Co-authored-by: Night Owl <91905208+nightowl121@users.noreply.github.com>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>
Co-authored-by: DeepakBomjan <44976635+DeepakBomjan@users.noreply.github.com>

* mv: tests to the e2e directory

* fix: code structure

* refactor: clean up and setup

* fix: remove unused storage for mock dapp

* fix: clean up and setup channel integration test

* fix: fix lint

---------

Signed-off-by: Night Owl <nightowl121@protonmail.com>
Co-authored-by: ibrizsabin <101165234+ibrizsabin@users.noreply.github.com>
Co-authored-by: sabinchitrakar <immortal.infidel@gmail.com>
Co-authored-by: AntonAndell <anton@andell.eu>
Co-authored-by: viveksharmapoudel <viveksharmapoudel@gmail.com>
Co-authored-by: Sudeep Bhandari <bhandari.sudeep68@gmail.com>
Co-authored-by: Night Owl <91905208+nightowl121@users.noreply.github.com>
Co-authored-by: redlarva <91685111+redlarva@users.noreply.github.com>
Co-authored-by: DeepakBomjan <44976635+DeepakBomjan@users.noreply.github.com>
Co-authored-by: red__larva <red.x.larva@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@viveksharmapoudel viveksharmapoudel viveksharmapoudel approved these changes

@AntonAndell AntonAndell AntonAndell approved these changes

@srmaharjan srmaharjan srmaharjan approved these changes

@redlarva redlarva Awaiting requested review from redlarva

@nightowl121 nightowl121 Awaiting requested review from nightowl121 nightowl121 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add docs for mock ibc app
6 participants
@ibrizsabin @codecov-commenter @srmaharjan @AntonAndell @viveksharmapoudel @sabinchitrakar