Skip to content

Commit

Permalink
Migrate RecordPackageManagerVersion to RecordEcosystemVersions
Browse files Browse the repository at this point in the history
This endpoint was renamed in dependabot/dependabot-core#7517
and the accompanying internal API change.

So the mock here needs updating. Also updated the struct.
  • Loading branch information
jeffwidman committed Jul 11, 2023
1 parent b5a4c38 commit 3f6ceef
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
2 changes: 1 addition & 1 deletion internal/model/update.go
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,7 @@ type MarkAsProcessed struct {
BaseCommitSha string `json:"base-commit-sha" yaml:"base-commit-sha"`
}

type RecordPackageManagerVersion struct {
type RecordEcosystemVersions struct {
Ecosystem string `json:"ecosystem" yaml:"ecosystem"`
PackageManagers map[string]any `json:"package-managers" yaml:"package-managers"`
}
Expand Down
12 changes: 6 additions & 6 deletions internal/server/api.go
Original file line number Diff line number Diff line change
Expand Up @@ -223,8 +223,8 @@ func decodeWrapper(kind string, data []byte) (actual *model.UpdateWrapper, err e
actual.Data, err = decode[model.ClosePullRequest](data)
case "mark_as_processed":
actual.Data, err = decode[model.MarkAsProcessed](data)
case "record_package_manager_version":
actual.Data, err = decode[model.RecordPackageManagerVersion](data)
case "record_ecosystem_versions":
actual.Data, err = decode[model.RecordEcosystemVersions](data)
case "record_update_job_error":
actual.Data, err = decode[model.RecordUpdateJobError](data)
case "increment_metric":
Expand Down Expand Up @@ -273,8 +273,8 @@ func compare(expect, actual *model.UpdateWrapper) error {
return compareUpdatePullRequest(v, actual.Data.(model.UpdatePullRequest))
case model.ClosePullRequest:
return compareClosePullRequest(v, actual.Data.(model.ClosePullRequest))
case model.RecordPackageManagerVersion:
return compareRecordPackageManagerVersion(v, actual.Data.(model.RecordPackageManagerVersion))
case model.RecordEcosystemVersions:
return compareRecordPackageManagerVersion(v, actual.Data.(model.RecordEcosystemVersions))
case model.MarkAsProcessed:
return compareMarkAsProcessed(v, actual.Data.(model.MarkAsProcessed))
case model.RecordUpdateJobError:
Expand Down Expand Up @@ -316,11 +316,11 @@ func compareClosePullRequest(expect, actual model.ClosePullRequest) error {
return unexpectedBody("close_pull_request")
}

func compareRecordPackageManagerVersion(expect, actual model.RecordPackageManagerVersion) error {
func compareRecordEcosystemVersions(expect, actual model.RecordEcosystemVersions) error {
if reflect.DeepEqual(expect, actual) {
return nil
}
return unexpectedBody("record_package_manager_version")
return unexpectedBody("record_ecosystem_versions")
}

func compareMarkAsProcessed(expect, actual model.MarkAsProcessed) error {
Expand Down

0 comments on commit 3f6ceef

Please sign in to comment.