Merge pull request #2 from pepabo/feature/add-github-actions

Add GitHub Actions workflow for Go tests

Author: buty4649

.github/workflows/test.yml

@@ -0,0 +1,23 @@
+name: Go Test
+
+on:
+  pull_request:
+
+jobs:
+  test:
+    name: Test
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Set up Go
+      uses: actions/setup-go@v5
+      with:
+        go-version: '1.24'
+        cache: true
+
+    - name: Install dependencies
+      run: go mod download
+
+    - name: Run tests
+      run: go test -v ./...

onelogin/user.go

@@ -86,6 +86,7 @@ func (o *Onelogin) CreateUser(user models.User) (models.User, error) {
 	createdUser := models.User{
 		ID:        int32(createdUserMap["id"].(float64)),
 		Email:     createdUserMap["email"].(string),
+		Username:  createdUserMap["username"].(string),
 		Firstname: createdUserMap["firstname"].(string),
 		Lastname:  createdUserMap["lastname"].(string),
 	}

onelogin/user_test.go

@@ -1,6 +1,7 @@
 package onelogin
 
 import (
+	"fmt"
 	"strconv"
 	"testing"
 
@@ -173,7 +174,7 @@ func TestCreateUser(t *testing.T) {
 				Lastname:  "User",
 			},
 			mockResponse: map[string]interface{}{
-				"id":        3,
+				"id":        float64(3),
 				"email":     "newuser@example.com",
 				"username":  "newuser",
 				"firstname": "New",
@@ -196,7 +197,7 @@ func TestCreateUser(t *testing.T) {
 				Lastname:  "User",
 			},
 			mockError:     assert.AnError,
-			expectedError: assert.AnError,
+			expectedError: fmt.Errorf("error creating user: %v", assert.AnError),
 		},
 	}
 
@@ -209,6 +210,11 @@ func TestCreateUser(t *testing.T) {
 
 			mockClient.On("CreateUser", tt.inputUser).Return(tt.mockResponse, tt.mockError)
 
+			// Add expectation for UpdateUser call in SetUserState
+			if tt.expectedError == nil {
+				mockClient.On("UpdateUser", 3, mock.AnythingOfType("models.User")).Return(nil, nil)
+			}
+
 			createdUser, err := o.CreateUser(tt.inputUser)
 
 			if tt.expectedError != nil {