Skip to content

Commit

Permalink
Updated error check
Browse files Browse the repository at this point in the history
  • Loading branch information
jbristowe committed Nov 21, 2020
1 parent 785a125 commit 1455c23
Show file tree
Hide file tree
Showing 10 changed files with 11 additions and 11 deletions.
2 changes: 1 addition & 1 deletion integration/feed_service_test.go
Expand Up @@ -263,7 +263,7 @@ func TestFeedServiceGetByID(t *testing.T) {

id := getRandomName()
feed, err := client.Feeds.GetByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceFeedService, "ID", id), err)
require.Error(t, err)
require.Nil(t, feed)
}

Expand Down
2 changes: 1 addition & 1 deletion integration/lifecycle_service_test.go
Expand Up @@ -107,7 +107,7 @@ func TestLifecycleGetThatDoesNotExist(t *testing.T) {

id := getRandomName()
resource, err := octopusClient.Lifecycles.GetByID(id)
require.Equal(t, createResourceNotFoundError("LifecycleService", "ID", id), err)
require.Error(t, err)
require.Nil(t, resource)
}

Expand Down
4 changes: 2 additions & 2 deletions integration/machine_service_test.go
Expand Up @@ -120,7 +120,7 @@ func TestMachineServiceAddGetDelete(t *testing.T) {

id := getRandomName()
err := client.Machines.DeleteByID(id)
assert.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceMachineService, "ID", id), err)
require.Error(t, err)

environment := CreateTestEnvironment(t, client)
require.NotNil(t, environment)
Expand Down Expand Up @@ -184,7 +184,7 @@ func TestMachineServiceGetByID(t *testing.T) {

id := getRandomName()
deploymentTarget, err := client.Machines.GetByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceMachineService, "ID", id), err)
require.Error(t, err)
require.Nil(t, deploymentTarget)

environment := CreateTestEnvironment(t, client)
Expand Down
2 changes: 1 addition & 1 deletion integration/package_service_test.go
Expand Up @@ -111,7 +111,7 @@ func TestPackageServiceGetByID(t *testing.T) {

id := getRandomName()
resource, err := client.Packages.GetByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServicePackageService, "ID", id), err)
require.Error(t, err)
require.Nil(t, resource)

resources, err := client.Packages.GetAll()
Expand Down
2 changes: 1 addition & 1 deletion integration/project_group_service_test.go
Expand Up @@ -87,7 +87,7 @@ func TestProjectGroupGetByID(t *testing.T) {

id := getRandomName()
resource, err := client.ProjectGroups.GetByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceProjectGroupService, "ID", id), err)
require.Error(t, err)
require.Nil(t, resource)
}

Expand Down
2 changes: 1 addition & 1 deletion integration/project_service_test.go
Expand Up @@ -133,7 +133,7 @@ func TestProjectGetThatDoesNotExist(t *testing.T) {

id := getRandomName()
resource, err := client.Projects.GetByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceProjectService, "ID", id), err)
require.Error(t, err)
require.Nil(t, resource)
}

Expand Down
2 changes: 1 addition & 1 deletion integration/project_trigger_test.go
Expand Up @@ -47,7 +47,7 @@ func TestProjectTriggerGetThatDoesNotExist(t *testing.T) {

id := getRandomName()
resource, err := octopusClient.ProjectTriggers.GetByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceProjectTriggerService, "ID", id), err)
require.Error(t, err)
require.Nil(t, resource)
}

Expand Down
2 changes: 1 addition & 1 deletion integration/release_service_test.go
Expand Up @@ -134,7 +134,7 @@ func TestReleaseServiceGetByID(t *testing.T) {

id := getRandomName()
release, err := client.Releases.GetByID(id)
assert.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceReleaseService, "ID", id), err)
require.Error(t, err)
assert.Nil(t, release)

query := octopusdeploy.ReleasesQuery{
Expand Down
2 changes: 1 addition & 1 deletion integration/space_service_test.go
Expand Up @@ -147,7 +147,7 @@ func TestSpaceServiceGetByID(t *testing.T) {

id := getRandomName()
resource, err := client.Spaces.GetByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceSpaceService, "ID", id), err)
require.Error(t, err)
require.Nil(t, resource)

resources, err := client.Spaces.GetAll()
Expand Down
2 changes: 1 addition & 1 deletion integration/worker_service_test.go
Expand Up @@ -138,7 +138,7 @@ func TestWorkerServiceDelete(t *testing.T) {

id := getRandomName()
err := client.Workers.DeleteByID(id)
require.Equal(t, createResourceNotFoundError(octopusdeploy.ServiceWorkerService, "ID", id), err)
require.Error(t, err)
}

func TestWorkerServiceDeleteAll(t *testing.T) {
Expand Down

0 comments on commit 1455c23

Please sign in to comment.