[On-chain] feat: add `proof_window_open_offset_blocks` as `shared` module parameter

[bryanchriswhite]

Summary

Adds proof_window_open_offset_blocks parameter to the shared module

Issue

• 516

Type of change

Select one or more:

• [x] New feature, functionality or library
• [ ] Bug fix
• [ ] Code health or cleanup
• [ ] Documentation
• [ ] Other (specify)

Testing

Documentation changes (only if making doc changes)

• [ ] make docusaurus_start; only needed if you make doc changes

Local Testing (only if making code changes)

• [x] Unit Tests: make go_develop_and_test
• [x] LocalNet E2E Tests: make test_e2e
• See quickstart guide for instructions

PR Testing (only if making code changes)

• [x] DevNet E2E Tests: Add the devnet-test-e2e label to the PR.
  • THIS IS VERY EXPENSIVE, so only do it after all the reviews are complete.
  • Optionally run make trigger_ci if you want to re-trigger tests without any code changes
  • If tests fail, try re-running failed tests only using the GitHub UI as shown here

Sanity Checklist

• [x] I have tested my changes using the available tooling
• [x] I have commented my code
• [x] I have performed a self-review of my own code; both comments & source code
• [ ] I create and reference any new tickets, if applicable
• [x] I have left TODOs throughout the codebase, if applicable

[github-actions[bot]]

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks.

You may need to run make trigger_ci to submit an empty commit that'll trigger the tests.

GCP workloads (requires changing the namespace to 569)) Grafana network dashboard for devnet-issue-{issue-id}

[bryanchriswhite]

Not a request but a question:

1. Why no Makefile target?

Added in fe30755.

2. Do we have a unit test for failing edge cases elsewhere? If so, we should have it here too

@Olshansk can you elaborate on what you mean here?

The first thing that msgServer#UpdateParams() does is call MsgUpdateParam#ValdiateBasic(), which has almost complete coverage:

Doesn't cover MsgUpdateParam#AsType == nil case.

The rest of the un-covered (sad) paths in msgServer#UpdateParam() are redundant as they would only occur if MsgUpdateParam#ValidateBasic() returned an error, which we're checking for initially:

Doesn't cover redundant sad paths.