Skip to content

Conversation

shubham14bajpai
Copy link
Contributor

I saw this error message with a small spelling mistake while writing tests.

cnanot clone suite after tree has been built

This PR fixes the same.

@blgm
Copy link
Collaborator

blgm commented Jan 5, 2024

Thank you @shubham14bajpai. This is a good improvement, but unfortunately the tests fail because they are expecting the typo!

  [FAILED] Expected
      <*errors.errorString | 0xc0002a81f0>: 
      cannot clone suite after tree has been built
      {
          s: "cannot clone suite after tree has been built",
      }
  to match error
      <string>: cnanot clone suite after tree has been built
  In [It] at: /home/runner/work/ginkgo/ginkgo/internal/suite_test.go:91 @ 01/05/24 11:01:59.47

Do you want to update the test too? If you don't have the time then I'm happy to do it, but wanted to give you the choice.

@shubham14bajpai
Copy link
Contributor Author

Thank you @shubham14bajpai. This is a good improvement, but unfortunately the tests fail because they are expecting the typo!

  [FAILED] Expected
      <*errors.errorString | 0xc0002a81f0>: 
      cannot clone suite after tree has been built
      {
          s: "cannot clone suite after tree has been built",
      }
  to match error
      <string>: cnanot clone suite after tree has been built
  In [It] at: /home/runner/work/ginkgo/ginkgo/internal/suite_test.go:91 @ 01/05/24 11:01:59.47

Do you want to update the test too? If you don't have the time then I'm happy to do it, but wanted to give you the choice.

Sure I will update the test as well 👍

@blgm blgm merged commit beb9507 into onsi:master Jan 5, 2024
@blgm
Copy link
Collaborator

blgm commented Jan 5, 2024

Thanks @shubham14bajpai!

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.

2 participants