Skip to content

Commit

Permalink
Merge pull request #8 from theseion/more-changes
Browse files Browse the repository at this point in the history
fix: use uint for (no_)expect_ids
  • Loading branch information
theseion authored May 9, 2024
2 parents ba047e4 + d095995 commit 48cda85
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions types/examples.go
Original file line number Diff line number Diff line change
Expand Up @@ -50,8 +50,8 @@ var (
ExpectError: helpers.BoolPtr(true),
}
ExampleLog = Log{
ExpectIds: []int{123456},
NoExpectIds: []int{123456},
ExpectIds: []uint{123456},
NoExpectIds: []uint{123456},
MatchRegex: `id[:\s"]*123456`,
NoMatchRegex: `id[:\s"]*123456`,
}
Expand Down
4 changes: 2 additions & 2 deletions types/types.go
Original file line number Diff line number Diff line change
Expand Up @@ -341,13 +341,13 @@ type Log struct {
// Expect the given IDs to be contained in the log output.
// examples:
// -value: ExampleLog.ExpectIds
ExpectIds []int `yaml:"expect_ids,omitempty"`
ExpectIds []uint `yaml:"expect_ids,omitempty"`

// description: |
// Expect the given IDs _not_ to be contained in the log output.
// examples:
// - value: ExampleLog.NoExpectIds
NoExpectIds []int `yaml:"no_expect_ids,omitempty"`
NoExpectIds []uint `yaml:"no_expect_ids,omitempty"`

// description: |
// Expect the regular expression to match log content for the current types.
Expand Down

0 comments on commit 48cda85

Please sign in to comment.