Skip to content

Commit

Permalink
Updated tests
Browse files Browse the repository at this point in the history
  • Loading branch information
jbristowe committed Nov 19, 2020
1 parent 75fe91d commit 7837ddb
Show file tree
Hide file tree
Showing 7 changed files with 8 additions and 12 deletions.
2 changes: 1 addition & 1 deletion integration/account_service_test.go
Expand Up @@ -246,7 +246,7 @@ func TestAccountServiceGetByID(t *testing.T) {

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

Expand Down
4 changes: 2 additions & 2 deletions integration/community_action_template_service_test.go
Expand Up @@ -38,12 +38,12 @@ func TestCommunityActionTemplateServiceGetBy(t *testing.T) {

invalidID := getRandomName()
communityActionTemplate, err := octopusClient.CommunityActionTemplates.GetByID(invalidID)
require.Equal(t, err, createResourceNotFoundError(octopusdeploy.ServiceCommunityActionTemplateService, "ID", invalidID))
require.Error(t, err)
require.Nil(t, communityActionTemplate)

invalidName := getRandomName()
communityActionTemplate, err = octopusClient.CommunityActionTemplates.GetByName(invalidName)
require.Equal(t, err, createResourceNotFoundError(octopusdeploy.ServiceCommunityActionTemplateService, "name", invalidName))
require.Error(t, err)
require.Nil(t, communityActionTemplate)

communityActionTemplates, err := octopusClient.CommunityActionTemplates.GetAll()
Expand Down
2 changes: 1 addition & 1 deletion octopusdeploy/deployment_process_service_test.go
Expand Up @@ -57,7 +57,7 @@ func TestDeploymentProcessServiceParameters(t *testing.T) {
require.Nil(t, resource)
} else {
resource, err := service.GetByID(tc.parameter)
require.Equal(t, err, createResourceNotFoundError(ServiceDeploymentProcessesService, "ID", tc.parameter))
require.Error(t, err)
require.Nil(t, resource)
}
})
Expand Down
3 changes: 1 addition & 2 deletions octopusdeploy/library_variable_set_service_test.go
Expand Up @@ -210,12 +210,11 @@ func TestLibraryVariableSetServiceParameters(t *testing.T) {
require.Equal(t, err, createInvalidParameterError(OperationDeleteByID, ParameterID))
} else {
resource, err := service.GetByID(tc.parameter)
require.Equal(t, err, createResourceNotFoundError(ServiceLibraryVariableSetService, "ID", tc.parameter))
require.Error(t, err)
require.Nil(t, resource)

err = service.DeleteByID(tc.parameter)
require.Error(t, err)
require.Equal(t, err, createResourceNotFoundError(ServiceLibraryVariableSetService, "ID", tc.parameter))
}
})
}
Expand Down
3 changes: 1 addition & 2 deletions octopusdeploy/machine_policy_service_test.go
Expand Up @@ -240,12 +240,11 @@ func TestMachinePolicyServiceParameters(t *testing.T) {
require.Equal(t, err, createInvalidParameterError(OperationDeleteByID, ParameterID))
} else {
resource, err := service.GetByID(tc.parameter)
require.Equal(t, err, createResourceNotFoundError(ServiceMachinePolicyService, "ID", tc.parameter))
require.Error(t, err)
require.Nil(t, resource)

err = service.DeleteByID(tc.parameter)
require.Error(t, err)
require.Equal(t, err, createResourceNotFoundError(ServiceMachinePolicyService, "ID", tc.parameter))
}
})
}
Expand Down
3 changes: 1 addition & 2 deletions octopusdeploy/space_service_test.go
Expand Up @@ -67,12 +67,11 @@ func TestSpaceServiceParameters(t *testing.T) {
require.Equal(t, err, createInvalidParameterError(OperationDeleteByID, ParameterID))
} else {
resource, err := service.GetByID(tc.parameter)
require.Equal(t, err, createResourceNotFoundError(ServiceSpaceService, "ID", tc.parameter))
require.Error(t, err)
require.Nil(t, resource)

err = service.DeleteByID(tc.parameter)
require.Error(t, err)
require.Equal(t, err, createResourceNotFoundError(ServiceSpaceService, "ID", tc.parameter))
}
})
}
Expand Down
3 changes: 1 addition & 2 deletions octopusdeploy/team_service_test.go
Expand Up @@ -67,12 +67,11 @@ func TestTeamServiceParameters(t *testing.T) {
require.Equal(t, err, createInvalidParameterError(OperationDeleteByID, ParameterID))
} else {
resource, err := service.GetByID(tc.parameter)
require.Equal(t, err, createResourceNotFoundError(ServiceTeamService, "ID", tc.parameter))
require.Error(t, err)
require.Nil(t, resource)

err = service.DeleteByID(tc.parameter)
require.Error(t, err)
require.Equal(t, err, createResourceNotFoundError(ServiceTeamService, "ID", tc.parameter))
}
})
}
Expand Down

0 comments on commit 7837ddb

Please sign in to comment.