willBeRetried not part of TestStepResult

[christian-bromann]

Describe the bug
The latest version of @cucumber/messages seems to mismatch with the current master branch. The willBeRetried property is in master part of the TestStepResult class, e.g.:

export class TestStepResult {

  @Type(() => Duration)
  duration: Duration = new Duration()

  message?: string

  status: TestStepResultStatus = TestStepResultStatus.UNKNOWN

  willBeRetried: boolean = false
}

However if you download latest package version it is part of TestCaseFinished for some reason:

export class TestCaseFinished {

  testCaseStartedId: string = ''

  @Type(() => Timestamp)
  timestamp: Timestamp = new Timestamp()

  willBeRetried: boolean = false
}

To Reproduce
Steps to reproduce the behavior:

1. Go to '...'
2. Click on '....'
3. Scroll down to '....'
4. See error

Expected behavior
willBeRetried should be part of TestStepResult.

Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

Smartphone (please complete the following information):

• Device: [e.g. iPhone6]
• OS: [e.g. iOS8.1]
• Browser [e.g. stock browser, safari]
• Version [e.g. 22]

Additional context
Add any other context about the problem here.

[luke-hill]

https://github.com/cucumber/common/blob/main/messages/CHANGELOG.md#1700---2021-07-08

It was moved as part of the latest breaking changes.

[mpkorstanje]

@luke-hill this was orginally posted in cucumber/messages-javascript.

The last sync to the subrepo was on May 20th. So I think there is another underlying cause.

@mattwynne With #1550 in mind I think some sign posts need to be updated.

[luke-hill]

Ah oki, rsync playing up then.

[mpkorstanje]

Mmh. The master branch is out of date. The main branch is up to date.

@mattwynne same problem but because of #1561 instead.

[luke-hill]

Actually I think it's the branch name. main seems correct: https://github.com/cucumber/messages-javascript/blob/main/src/messages.ts#L618

[aslakhellesoy]

I've updated the default branch of https://github.com/cucumber/messages-javascript to be main. Does that fix this issue @christian-bromann ?

[christian-bromann]

@aslakhellesoy I updated the package, all looks good now, thank you team!