CUT-4490-ImportUpdate CSV Bug Fix

PowerShell/JumpCloud Module/Docs/JumpCloud.md

@@ -2,7 +2,7 @@
 Module Name: JumpCloud
 Module Guid: 31c023d1-a901-48c4-90a3-082f91b31646
 Download Help Link: https://github.com/TheJumpCloud/support/wiki
-Help Version: 2.15.0
+Help Version: 2.16.0
 Locale: en-Us
 ---
 

PowerShell/JumpCloud Module/JumpCloud.psd1

@@ -8,161 +8,161 @@
 
 @{
 
-# Script module or binary module file associated with this manifest.
-RootModule = 'JumpCloud.psm1'
-
-# Version number of this module.
-ModuleVersion = '2.15.0'
-
-# Supported PSEditions
-# CompatiblePSEditions = @()
-
-# ID used to uniquely identify this module
-GUID = '31c023d1-a901-48c4-90a3-082f91b31646'
-
-# Author of this module
-Author = 'JumpCloud Solutions Architect Team'
-
-# Company or vendor of this module
-CompanyName = 'JumpCloud'
-
-# Copyright statement for this module
-Copyright = '(c) JumpCloud. All rights reserved.'
-
-# Description of the functionality provided by this module
-Description = 'PowerShell functions to manage a JumpCloud Directory-as-a-Service'
-
-# Minimum version of the PowerShell engine required by this module
-PowerShellVersion = '4.0'
-
-# Name of the PowerShell host required by this module
-# PowerShellHostName = ''
-
-# Minimum version of the PowerShell host required by this module
-# PowerShellHostVersion = ''
-
-# Minimum version of Microsoft .NET Framework required by this module. This prerequisite is valid for the PowerShell Desktop edition only.
-# DotNetFrameworkVersion = ''
-
-# Minimum version of the common language runtime (CLR) required by this module. This prerequisite is valid for the PowerShell Desktop edition only.
-# ClrVersion = ''
+    # Script module or binary module file associated with this manifest.
+    RootModule        = 'JumpCloud.psm1'
+
+    # Version number of this module.
+    ModuleVersion     = '2.16.0'
+
+    # Supported PSEditions
+    # CompatiblePSEditions = @()
+
+    # ID used to uniquely identify this module
+    GUID              = '31c023d1-a901-48c4-90a3-082f91b31646'
+
+    # Author of this module
+    Author            = 'JumpCloud Solutions Architect Team'
+
+    # Company or vendor of this module
+    CompanyName       = 'JumpCloud'
+
+    # Copyright statement for this module
+    Copyright         = '(c) JumpCloud. All rights reserved.'
+
+    # Description of the functionality provided by this module
+    Description       = 'PowerShell functions to manage a JumpCloud Directory-as-a-Service'
+
+    # Minimum version of the PowerShell engine required by this module
+    PowerShellVersion = '4.0'
+
+    # Name of the PowerShell host required by this module
+    # PowerShellHostName = ''
+
+    # Minimum version of the PowerShell host required by this module
+    # PowerShellHostVersion = ''
+
+    # Minimum version of Microsoft .NET Framework required by this module. This prerequisite is valid for the PowerShell Desktop edition only.
+    # DotNetFrameworkVersion = ''
+
+    # Minimum version of the common language runtime (CLR) required by this module. This prerequisite is valid for the PowerShell Desktop edition only.
+    # ClrVersion = ''
 
-# Processor architecture (None, X86, Amd64) required by this module
-# ProcessorArchitecture = ''
+    # Processor architecture (None, X86, Amd64) required by this module
+    # ProcessorArchitecture = ''
 
-# Modules that must be imported into the global environment prior to importing this module
-RequiredModules = @('JumpCloud.SDK.DirectoryInsights', 
-               'JumpCloud.SDK.V1', 
-               'JumpCloud.SDK.V2')
+    # Modules that must be imported into the global environment prior to importing this module
+    RequiredModules   = @('JumpCloud.SDK.DirectoryInsights',
+        'JumpCloud.SDK.V1',
+        'JumpCloud.SDK.V2')
 
-# Assemblies that must be loaded prior to importing this module
-# RequiredAssemblies = @()
+    # Assemblies that must be loaded prior to importing this module
+    # RequiredAssemblies = @()
 
-# Script files (.ps1) that are run in the caller's environment prior to importing this module.
-# ScriptsToProcess = @()
+    # Script files (.ps1) that are run in the caller's environment prior to importing this module.
+    # ScriptsToProcess = @()
 
-# Type files (.ps1xml) to be loaded when importing this module
-# TypesToProcess = @()
+    # Type files (.ps1xml) to be loaded when importing this module
+    # TypesToProcess = @()
 
-# Format files (.ps1xml) to be loaded when importing this module
-# FormatsToProcess = @()
+    # Format files (.ps1xml) to be loaded when importing this module
+    # FormatsToProcess = @()
 
-# Modules to import as nested modules of the module specified in RootModule/ModuleToProcess
-# NestedModules = @()
+    # Modules to import as nested modules of the module specified in RootModule/ModuleToProcess
+    # NestedModules = @()
 
-# Functions to export from this module, for best performance, do not use wildcards and do not delete the entry, use an empty array if there are no functions to export.
-FunctionsToExport = 'Add-JCAssociation', 'Add-JCCommandTarget', 'Add-JCGsuiteMember', 
-               'Add-JCOffice365Member', 'Add-JCRadiusReplyAttribute', 
-               'Add-JCSystemGroupMember', 'Add-JCSystemUser', 
-               'Add-JCUserGroupMember', 'Backup-JCOrganization', 'Connect-JCOnline', 
-               'Copy-JCAssociation', 'Get-JCAdmin', 'Get-JCAssociation', 
-               'Get-JCBackup', 'Get-JCCloudDirectory', 'Get-JCCommand', 
-               'Get-JCCommandResult', 'Get-JCCommandTarget', 
-               'Get-JCConfiguredTemplatePolicy', 'Get-JCEvent', 'Get-JCEventCount', 
-               'Get-JCGroup', 'Get-JCOrganization', 'Get-JCPolicy', 
-               'Get-JCPolicyGroup', 'Get-JCPolicyGroupMember', 
-               'Get-JCPolicyGroupTemplate', 'Get-JCPolicyGroupTemplateMember', 
-               'Get-JCPolicyResult', 'Get-JCPolicyTargetGroup', 
-               'Get-JCPolicyTargetSystem', 'Get-JCRadiusReplyAttribute', 
-               'Get-JCRadiusServer', 'Get-JCScheduledUserstate', 'Get-JCSystem', 
-               'Get-JCSystemApp', 'Get-JCSystemGroupMember', 'Get-JCSystemInsights', 
-               'Get-JCSystemKB', 'Get-JCSystemUser', 'Get-JCUser', 
-               'Get-JCUserGroupMember', 'Import-JCCommand', 'Import-JCMSPFromCSV', 
-               'Import-JCUsersFromCSV', 'Invoke-JCCommand', 'Invoke-JCDeployment', 
-               'New-JCCommand', 'New-JCDeploymentTemplate', 'New-JCImportTemplate', 
-               'New-JCMSPImportTemplate', 'New-JCPolicy', 'New-JCPolicyGroup', 
-               'New-JCRadiusServer', 'New-JCSystemGroup', 'New-JCUser', 
-               'New-JCUserGroup', 'Remove-JCAssociation', 'Remove-JCCommand', 
-               'Remove-JCCommandResult', 'Remove-JCCommandTarget', 
-               'Remove-JCGsuiteMember', 'Remove-JCOffice365Member', 
-               'Remove-JCPolicy', 'Remove-JCPolicyGroup', 
-               'Remove-JCPolicyGroupTemplate', 'Remove-JCRadiusReplyAttribute', 
-               'Remove-JCRadiusServer', 'Remove-JCSystem', 'Remove-JCSystemGroup', 
-               'Remove-JCSystemGroupMember', 'Remove-JCSystemUser', 'Remove-JCUser', 
-               'Remove-JCUserGroup', 'Remove-JCUserGroupMember', 
-               'Send-JCPasswordReset', 'Set-JCCloudDirectory', 'Set-JCCommand', 
-               'Set-JCOrganization', 'Set-JCPolicy', 'Set-JCPolicyGroup', 
-               'Set-JCRadiusReplyAttribute', 'Set-JCRadiusServer', 
-               'Set-JCSettingsFile', 'Set-JCSystem', 'Set-JCSystemUser', 'Set-JCUser', 
-               'Set-JCUserGroupLDAP', 'Update-JCModule', 'Update-JCMSPFromCSV', 
-               'Update-JCUsersFromCSV'
+    # Functions to export from this module, for best performance, do not use wildcards and do not delete the entry, use an empty array if there are no functions to export.
+    FunctionsToExport = 'Add-JCAssociation', 'Add-JCCommandTarget', 'Add-JCGsuiteMember',
+    'Add-JCOffice365Member', 'Add-JCRadiusReplyAttribute',
+    'Add-JCSystemGroupMember', 'Add-JCSystemUser',
+    'Add-JCUserGroupMember', 'Backup-JCOrganization', 'Connect-JCOnline',
+    'Copy-JCAssociation', 'Get-JCAdmin', 'Get-JCAssociation',
+    'Get-JCBackup', 'Get-JCCloudDirectory', 'Get-JCCommand',
+    'Get-JCCommandResult', 'Get-JCCommandTarget',
+    'Get-JCConfiguredTemplatePolicy', 'Get-JCEvent', 'Get-JCEventCount',
+    'Get-JCGroup', 'Get-JCOrganization', 'Get-JCPolicy',
+    'Get-JCPolicyGroup', 'Get-JCPolicyGroupMember',
+    'Get-JCPolicyGroupTemplate', 'Get-JCPolicyGroupTemplateMember',
+    'Get-JCPolicyResult', 'Get-JCPolicyTargetGroup',
+    'Get-JCPolicyTargetSystem', 'Get-JCRadiusReplyAttribute',
+    'Get-JCRadiusServer', 'Get-JCScheduledUserstate', 'Get-JCSystem',
+    'Get-JCSystemApp', 'Get-JCSystemGroupMember', 'Get-JCSystemInsights',
+    'Get-JCSystemKB', 'Get-JCSystemUser', 'Get-JCUser',
+    'Get-JCUserGroupMember', 'Import-JCCommand', 'Import-JCMSPFromCSV',
+    'Import-JCUsersFromCSV', 'Invoke-JCCommand', 'Invoke-JCDeployment',
+    'New-JCCommand', 'New-JCDeploymentTemplate', 'New-JCImportTemplate',
+    'New-JCMSPImportTemplate', 'New-JCPolicy', 'New-JCPolicyGroup',
+    'New-JCRadiusServer', 'New-JCSystemGroup', 'New-JCUser',
+    'New-JCUserGroup', 'Remove-JCAssociation', 'Remove-JCCommand',
+    'Remove-JCCommandResult', 'Remove-JCCommandTarget',
+    'Remove-JCGsuiteMember', 'Remove-JCOffice365Member',
+    'Remove-JCPolicy', 'Remove-JCPolicyGroup',
+    'Remove-JCPolicyGroupTemplate', 'Remove-JCRadiusReplyAttribute',
+    'Remove-JCRadiusServer', 'Remove-JCSystem', 'Remove-JCSystemGroup',
+    'Remove-JCSystemGroupMember', 'Remove-JCSystemUser', 'Remove-JCUser',
+    'Remove-JCUserGroup', 'Remove-JCUserGroupMember',
+    'Send-JCPasswordReset', 'Set-JCCloudDirectory', 'Set-JCCommand',
+    'Set-JCOrganization', 'Set-JCPolicy', 'Set-JCPolicyGroup',
+    'Set-JCRadiusReplyAttribute', 'Set-JCRadiusServer',
+    'Set-JCSettingsFile', 'Set-JCSystem', 'Set-JCSystemUser', 'Set-JCUser',
+    'Set-JCUserGroupLDAP', 'Update-JCModule', 'Update-JCMSPFromCSV',
+    'Update-JCUsersFromCSV'
 
-# Cmdlets to export from this module, for best performance, do not use wildcards and do not delete the entry, use an empty array if there are no cmdlets to export.
-CmdletsToExport = @()
+    # Cmdlets to export from this module, for best performance, do not use wildcards and do not delete the entry, use an empty array if there are no cmdlets to export.
+    CmdletsToExport   = @()
 
-# Variables to export from this module
-VariablesToExport = '*'
+    # Variables to export from this module
+    VariablesToExport = '*'
 
-# Aliases to export from this module, for best performance, do not use wildcards and do not delete the entry, use an empty array if there are no aliases to export.
-AliasesToExport = 'New-JCAssociation'
+    # Aliases to export from this module, for best performance, do not use wildcards and do not delete the entry, use an empty array if there are no aliases to export.
+    AliasesToExport   = 'New-JCAssociation'
 
-# DSC resources to export from this module
-# DscResourcesToExport = @()
+    # DSC resources to export from this module
+    # DscResourcesToExport = @()
 
-# List of all modules packaged with this module
-# ModuleList = @()
+    # List of all modules packaged with this module
+    # ModuleList = @()
 
-# List of all files packaged with this module
-# FileList = @()
+    # List of all files packaged with this module
+    # FileList = @()
 
-# Private data to pass to the module specified in RootModule/ModuleToProcess. This may also contain a PSData hashtable with additional module metadata used by PowerShell.
-PrivateData = @{
+    # Private data to pass to the module specified in RootModule/ModuleToProcess. This may also contain a PSData hashtable with additional module metadata used by PowerShell.
+    PrivateData       = @{
 
-    PSData = @{
+        PSData = @{
 
-        # Tags applied to this module. These help with module discovery in online galleries.
-        Tags = 'JumpCloud', 'DaaS', 'Jump', 'Cloud', 'Directory'
+            # Tags applied to this module. These help with module discovery in online galleries.
+            Tags         = 'JumpCloud', 'DaaS', 'Jump', 'Cloud', 'Directory'
 
-        # A URL to the license for this module.
-        LicenseUri = 'https://github.com/TheJumpCloud/support/blob/master/PowerShell/LICENSE'
-
-        # A URL to the main website for this project.
-        ProjectUri = 'https://github.com/TheJumpCloud/support/wiki'
-
-        # A URL to an icon representing this module.
-        IconUri = 'https://avatars1.githubusercontent.com/u/4927461?s=200&v=4'
-
-        # ReleaseNotes of this module
-        ReleaseNotes = 'https://git.io/jc-pwsh-releasenotes'
-
-        # Prerelease string of this module
-        # Prerelease = ''
-
-        # Flag to indicate whether the module requires explicit user acceptance for install/update/save
-        # RequireLicenseAcceptance = $false
-
-        # External dependent modules of this module
-        # ExternalModuleDependencies = @()
-
-    } # End of PSData hashtable
-
-} # End of PrivateData hashtable
-
-# HelpInfo URI of this module
-HelpInfoURI = 'https://github.com/TheJumpCloud/support/wiki'
-
-# Default prefix for commands exported from this module. Override the default prefix using Import-Module -Prefix.
-# DefaultCommandPrefix = ''
+            # A URL to the license for this module.
+            LicenseUri   = 'https://github.com/TheJumpCloud/support/blob/master/PowerShell/LICENSE'
+
+            # A URL to the main website for this project.
+            ProjectUri   = 'https://github.com/TheJumpCloud/support/wiki'
+
+            # A URL to an icon representing this module.
+            IconUri      = 'https://avatars1.githubusercontent.com/u/4927461?s=200&v=4'
+
+            # ReleaseNotes of this module
+            ReleaseNotes = 'https://git.io/jc-pwsh-releasenotes'
+
+            # Prerelease string of this module
+            # Prerelease = ''
+
+            # Flag to indicate whether the module requires explicit user acceptance for install/update/save
+            # RequireLicenseAcceptance = $false
+
+            # External dependent modules of this module
+            # ExternalModuleDependencies = @()
+
+        } # End of PSData hashtable
+
+    } # End of PrivateData hashtable
+
+    # HelpInfo URI of this module
+    HelpInfoURI       = 'https://github.com/TheJumpCloud/support/wiki'
+
+    # Default prefix for commands exported from this module. Override the default prefix using Import-Module -Prefix.
+    # DefaultCommandPrefix = ''
 
 }
 

PowerShell/JumpCloud Module/Public/Utilities/CSV_Import/Import-JCUsersFromCSV.ps1

@@ -369,7 +369,13 @@ Function Import-JCUsersFromCSV () {
             $FormatGroupOutput = $Null
             $CustomGroupArrayList = $Null
 
-            $CustomAttributes = $UserAdd | Get-Member | Where-Object Name -Like "*Attribute*" | Where-Object { $_.Definition -NotLike "*=" -and $_.Definition -NotLike "*null" } | Select-Object
+            # Get all the custom attributes that are not null
+            $CustomAttributes = $UserAdd | Get-Member | Where-Object Name -Like "*Attribute*" | Where-Object { $_.Definition -NotLike "*=" -and $_.Definition -NotLike "*null" }
+
+            # Sort the attributes by number and name
+            $CustomAttributes = $CustomAttributes | Sort-Object {
+                [int]([regex]::Match($_.Name, '\d+').Value) },
+            { $_.Name }
 
             Write-Verbose $CustomAttributes.name.count
 

PowerShell/JumpCloud Module/Public/Utilities/CSV_Import/Update-JCUsersFromCSV.ps1

@@ -341,7 +341,13 @@ Function Update-JCUsersFromCSV () {
             $FormatGroupOutput = $Null
             $CustomGroupArrayList = $Null
 
-            $CustomAttributes = $UserUpdate | Get-Member | Where-Object Name -Like "*Attribute*" | Where-Object { $_.Definition -NotLike "*=" -and $_.Definition -NotLike "*null" } | Select-Object
+            # Get all the custom attributes that are not null
+            $CustomAttributes = $UserUpdate | Get-Member | Where-Object Name -Like "*Attribute*" | Where-Object { $_.Definition -NotLike "*=" -and $_.Definition -NotLike "*null" }
+
+            # Sort the attributes by number and name
+            $CustomAttributes = $CustomAttributes | Sort-Object {
+                [int]([regex]::Match($_.Name, '\d+').Value) },
+            { $_.Name }
 
             if ($CustomAttributes.name.count -gt 1) {
                 try {

PowerShell/JumpCloud Module/Tests/Public/Utilities/CSV_Import/Import-JCUsersFromCSV.Tests.ps1

@@ -636,6 +636,47 @@ Describe -Tag:('JCUsersFromCSV') "Import-JCUsersFromCSV 2.14.2" {
         }
     }
 }
+Describe -Tag:('JCUsersFromCSV') "Import-JCUsersFromCSV 2.15.1" {
+    Context "Import-JCUsersFromCSV with 10 or more custom attributes" {
+        It "When there are 10 or more custom attributes, the API should not return an error message in the status field and continue to import the user" {
+            $user = New-RandomUser -Domain "ImportCSVUser.$(New-RandomString -NumberOfChars 5)"
+            $today = Get-Date
+            $EnrollmentDays = 14
+            $CSVDATA = @{
+                Username          = $user.username
+                LastName          = $user.LastName
+                FirstName         = $user.FirstName
+                Email             = $user.Email
+                Attribute1_name   = "Name"
+                Attribute1_value  = "Value"
+                Attribute2_name   = "Name1"
+                Attribute2_value  = "Value1"
+                Attribute3_name   = "Name2"
+                Attribute3_value  = "Value2"
+                Attribute4_name   = "Name3"
+                Attribute4_value  = "Value3"
+                Attribute5_name   = "Name4"
+                Attribute5_value  = "Value4"
+                Attribute6_name   = "Name5"
+                Attribute6_value  = "Value5"
+                Attribute7_name   = "Name6"
+                Attribute7_value  = "Value6"
+                Attribute8_name   = "Name7"
+                Attribute8_value  = "Value7"
+                Attribute9_name   = "Name8"
+                Attribute9_value  = "Value8"
+                Attribute10_name  = "Name9"
+                Attribute10_value = "Value9"
+                Attribute11_name  = "Name10"
+                Attribute11_value = "Value10"
+            }
+            $CSVFILE = $CSVDATA | Export-Csv "$PesterParams_ImportPath/custom_attribute.csv" -Force
+            $importResults = Import-JCUsersFromCSV -CSVFilePath "$PesterParams_ImportPath/custom_attribute.csv" -force
+            # ImportResults should not be error
+            $importResults[0].Status | Should -Match "User Created"
+        }
+    }
+}
 
 AfterAll {
     Get-JCUser | Where-Object Email -like *testimportcsvuser* | Remove-JCUser -force