feat!: support users v2 api (#190)

BREAKING CHANGE: all "Q" parameters were renamed to "Query" to ensure consistency in the entire codebase

Signed-off-by: BoxBoxJason <contact@boxboxjason.dev>

Author: BoxBoxJason

README.md

@@ -321,8 +321,8 @@ fmt.Printf("Quality Gate: %s\n", status.ProjectStatus.Status)
 **User management:**
 
 ```go
-users, _, err := client.Users.Search(ctx, &sonar.UsersSearchOption{
- Q: sonar.String("john"),
+users, _, err := client.Users.Search(&sonar.UsersSearchOption{
+ Query: "john",
 })
 for _, user := range users.Users {
  fmt.Printf("User: %s (%s)\n", user.Name, user.Login)

integration_testing/ce_test.go

@@ -110,7 +110,7 @@ var _ = Describe("Ce Service", Ordered, func() {
 
 			It("should search CE tasks by query", func() {
 				result, resp, err := client.Ce.Activity(&sonar.CeActivityOption{
-					Q: testProject.Project.Key,
+					Query: testProject.Project.Key,
 				})
 				Expect(err).NotTo(HaveOccurred())
 				Expect(resp.StatusCode).To(Equal(http.StatusOK))

integration_testing/components_test.go

@@ -480,7 +480,7 @@ var _ = Describe("Components Service", Ordered, func() {
 			Expect(err).NotTo(HaveOccurred())
 			Expect(resp.StatusCode).To(Equal(http.StatusOK))
 			Expect(result.Name).NotTo(BeEmpty())
-			Expect(result.Q).To(Equal("TRK"))
+			Expect(result.Query).To(Equal("TRK"))
 		})
 
 		Context("parameter validation", func() {

integration_testing/helpers/cleanup.go

@@ -165,7 +165,7 @@ func cleanupOrphanedUsers(client *sonar.Client, _ time.Duration) error {
 		LastConnectedAfter:    "",
 		LastConnectedBefore:   "",
 		Managed:               false,
-		Q:                     E2EResourcePrefix,
+		Query:                 E2EResourcePrefix,
 		SlLastConnectedAfter:  "",
 		SlLastConnectedBefore: "",
 	})

integration_testing/users_test.go

@@ -82,7 +82,7 @@ var _ = Describe("Users Service", Ordered, func() {
 			It("should filter users by query", func() {
 				//nolint:staticcheck // Using deprecated API until v2 API is implemented
 				result, resp, err := client.Users.Search(&sonar.UsersSearchOption{
-					Q: "admin",
+					Query: "admin",
 				})
 				Expect(err).NotTo(HaveOccurred())
 				Expect(resp.StatusCode).To(Equal(http.StatusOK))
@@ -101,7 +101,7 @@ var _ = Describe("Users Service", Ordered, func() {
 			It("should return empty list for non-matching query", func() {
 				//nolint:staticcheck // Using deprecated API until v2 API is implemented
 				result, resp, err := client.Users.Search(&sonar.UsersSearchOption{
-					Q: "nonexistentuserxyz123",
+					Query: "nonexistentuserxyz123",
 				})
 				Expect(err).NotTo(HaveOccurred())
 				Expect(resp.StatusCode).To(Equal(http.StatusOK))
@@ -533,7 +533,7 @@ var _ = Describe("Users Service", Ordered, func() {
 			// Verify new login exists
 			//nolint:staticcheck // Using deprecated API until v2 API is implemented
 			result, _, err := client.Users.Search(&sonar.UsersSearchOption{
-				Q: newLogin,
+				Query: newLogin,
 			})
 			Expect(err).NotTo(HaveOccurred())
 			found := false
@@ -601,7 +601,7 @@ var _ = Describe("Users Service", Ordered, func() {
 			//nolint:staticcheck // Using deprecated API until v2 API is implemented
 			result, resp, err := client.Users.Groups(&sonar.UsersGroupsOption{
 				Login: "admin",
-				Q:     "sonar",
+				Query: "sonar",
 			})
 			Expect(err).NotTo(HaveOccurred())
 			Expect(resp.StatusCode).To(Equal(http.StatusOK))
@@ -776,7 +776,7 @@ var _ = Describe("Users Service", Ordered, func() {
 			// Verify user is deactivated
 			//nolint:staticcheck // Using deprecated API until v2 API is implemented
 			searchResult, _, err := client.Users.Search(&sonar.UsersSearchOption{
-				Q:           login,
+				Query:       login,
 				Deactivated: true,
 			})
 			Expect(err).NotTo(HaveOccurred())
@@ -943,7 +943,7 @@ var _ = Describe("Users Service", Ordered, func() {
 			// Step 2: Search and verify
 			//nolint:staticcheck // Using deprecated API until v2 API is implemented
 			searchResult, _, err := client.Users.Search(&sonar.UsersSearchOption{
-				Q: login,
+				Query: login,
 			})
 			Expect(err).NotTo(HaveOccurred())
 			found := false