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.

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

chore(rln-relay): Laundry list for tests #1358

Closed
3 tasks done
rymnc opened this issue Nov 8, 2022 · 1 comment
Closed
3 tasks done

chore(rln-relay): Laundry list for tests #1358

rymnc opened this issue Nov 8, 2022 · 1 comment
Labels
track:rln RLN Track (Secure Messaging/Applied ZK), e.g. relay and applications

Comments

@rymnc
Copy link
Contributor

rymnc commented Nov 8, 2022

Background

There are a few instances of code in

  • test_waku_rln_relay.nim
  • test_waku_rln_relay_onchain.nim
    that need to be refactored according to the Status and/or nwaku style guide.

Details

Address the issues and adhere to the style guides.

Acceptance criteria

  • Replace var with let
  • Move checks to the end of the test, or replace with require
  • Use parentheses for isOk, isErr and associated procs

cc: @LNSD

@rymnc rymnc added track:rln RLN Track (Secure Messaging/Applied ZK), e.g. relay and applications track:maintenance labels Nov 8, 2022
@oskarth oskarth added this to Waku and Vac Research Nov 8, 2022
@jm-clius jm-clius removed this from Waku Nov 16, 2022
@rymnc rymnc added this to the Release 0.14.0 milestone Dec 7, 2022
@rymnc
Copy link
Contributor Author

rymnc commented Dec 12, 2022

Completed in #1452

@rymnc rymnc closed this as completed Dec 12, 2022
@rymnc rymnc moved this to Done in Vac Research Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
track:rln RLN Track (Secure Messaging/Applied ZK), e.g. relay and applications
Projects
None yet
Development

No branches or pull requests

1 participant