Skip to content

Update check for election ID in ALL_PRIMARY.#123

Merged
robshakir merged 1 commit intomainfrom
election-id-in-primary
Mar 3, 2022
Merged

Update check for election ID in ALL_PRIMARY.#123
robshakir merged 1 commit intomainfrom
election-id-in-primary

Conversation

@robshakir
Copy link
Copy Markdown
Member

  * (M) compliance/election.go
    - Fixes #119.
    - Previously, we would not send an election ID update to the server
      since the client explicitly doesn't send the ID if initialised
      as ALL_PRIMARY. Change the test to explicitly make the update.

Fixes #119.

  * (M) compliance/election.go
    - Fixes #119.
    - Previously, we would not send an election ID update to the server
      since the client explicitly doesn't send the ID if initialised
      as ALL_PRIMARY. Change the test to explicitly make the update.
@robshakir robshakir merged commit 15d4147 into main Mar 3, 2022
@robshakir robshakir deleted the election-id-in-primary branch March 3, 2022 17:18
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.

TestCompliance/Election_-_Ensure_that_election_ID_is_not_accepted_in_ALL_PRIMARY_mode does not send an election id

2 participants