refactoring: group host inventory job test. Makefile Adjustments

Author: arrestle

.gitignore

@@ -6,7 +6,6 @@ terraform-provider-aap
 *.tfstate
 *.tfstate.*
 *.tfrc
-*.txt
 
 # Crash log files
 crash.log

Makefile

@@ -13,9 +13,13 @@ test: ## Execute all unit tests with verbose output."
 	@echo "==> Running unit tests..."
 	go test -v ./...
 
-testacc: ## Run acceptance tests against local aap-dev instance (https://localhost:9080)."
+testacc: ## Run acceptance tests against local aap-dev instance (https://localhost:8043)."
 	@echo "==> Running acceptance tests..."
-	TF_ACC=1 AAP_HOST="http://localhost:9080" go test -count=1 -v ./...
+	TF_ACC=1 AAP_HOST="http://localhost:8043" go test -count=1 -v ./...
+
+testacc-aapdev: ## Run acceptance tests against local aap-dev instance (EXPORT AAP_HOST="http://localhost:9080")
+	@echo "==> Running acceptance tests..."
+	TF_ACC=1 go test -count=1 -v ./...
 
 generatedocs: ## Format example Terraform configurations and generate plugin documentation."
 	@echo "==> Formatting examples and generating docs..."

README.md

@@ -42,11 +42,14 @@ Run `make test`
 
 Acceptance tests apply test terraform configurations to a running AAP instance and make changes to resources in that instance, use with caution!
 
-To run acceptance tests locally, start a local AAP instance following the [The AAP development environment](https://github.com/ansible/aap-dev/blob/main/README.md). Create an admin user for the AAP instance and save the credentials to these environment variables:
+To run acceptance tests locally, you will need a running instance of Ansible Automation Platform (AAP). You can either use an existing instance or deploy a local test environment using any supported method (e.g., containerized or VM-based deployment from official Red Hat resources).
+
+Create an admin user for the AAP instance and set the following environment variables:
 
 ```bash
 export AAP_USERNAME=<your admin username>
 export AAP_PASSWORD=<your admin password>
+export AAP_HOST="http://localhost:9080" # if using aap-dev (Note: Subject to change)
 ```
 
 In order to run the acceptance tests for the job resource, you must have a working job template already in your AAP instance. The job template must be set to require an inventory on launch. Export the id of this job template:

internal/provider/utils_test.go

@@ -66,11 +66,12 @@ func checkBasicInventoryAttributes(t *testing.T, name, expectedName string) reso
 	t.Helper()
 	return resource.ComposeAggregateTestCheckFunc(
 		resource.TestCheckResourceAttr(name, "name", expectedName),
-		resource.TestCheckResourceAttr(name, "organization_id", "1"),
+		resource.TestCheckResourceAttr(name, "organization", "1"),
 		resource.TestCheckResourceAttr(name, "organization_name", "Default"),
 		resource.TestMatchResourceAttr(name, "url", reInventoryURLPattern),
-		resource.TestCheckResourceAttrSet(name, "id"),
 		resource.TestCheckResourceAttr(name, "named_url", fmt.Sprintf("/api/controller/v2/inventories/%s++%s/", expectedName, "Default")),
+		resource.TestCheckResourceAttrSet(name, "id"),
+		resource.TestCheckResourceAttrSet("aap_inventory.test", "url"),
 	)
 }